Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add warning banner or disable license upload when running in Kubernetes #1479

Open
chrisseto opened this issue Oct 17, 2024 · 3 comments
Open
Labels
backend feature New feature or request frontend

Comments

@chrisseto
Copy link

Thanks to @JakeSCahill for bringing this to my attention.

Console's new (?) ability to directly upload a license to redpanda is great but could lead to some accidental foot-guns if deployed in Kubernetes via the helm chart or operator.

  1. Console will require a manual restart. This would require a user to manually delete the console Pods after uploading the console.
  2. The uploaded license may get overridden if the redpanda license is also managed via helm or the operator.

Neither of these are particularly nasty but 2. could certainly lead to some head scratching or perhaps some unexpected downtime if redpanda doesn't reject expired licenses.

I'd be ideal if console could either disable the ability to upload licenses or perhaps display a warning with a link to our docs with an explanation of the warning. As for the restart... console could just crash itself and rely on Kubernetes to restart it but that's likely not the most pleasant experience.

Console could either garner it's environment via some heuristics or we could have the chart / operator set either a config value or environment variable to enable such functionalities.

cc @david-yu for your thoughts as well.

@weeco
Copy link
Contributor

weeco commented Oct 20, 2024

I'd probably vote for adding some config that informs Console it's managed by our Kubernetes operator. Based on that we could do further adaptions that are specific to operator managed environments. Maybe that suffices versus the approach of making it controllable per feature (i.e. disable license upload because operator) until we need additional flexibility in the future? Both would be fine for me though.

@weeco weeco added feature New feature or request frontend backend labels Oct 20, 2024
@chrisseto
Copy link
Author

No strong preference from my end! Either way we'll just be making changes to the config file.

@david-yu
Copy link

@weeco That works. I've seen it work the same in a previous job just as you described.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend feature New feature or request frontend
Projects
None yet
Development

No branches or pull requests

3 participants