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

Support deploying multiple instances to the same cluster #2835

Open
antdking opened this issue Sep 6, 2023 · 1 comment
Open

Support deploying multiple instances to the same cluster #2835

antdking opened this issue Sep 6, 2023 · 1 comment

Comments

@antdking
Copy link
Contributor

antdking commented Sep 6, 2023

Is your feature request related to a problem? Please describe.

We deploy multiple instances of Fission; however parts of the Chart don't support this pattern out the box.
The main problem is with the Webhook configurations, which aren't namespaced + use static names while being cluster-scoped objects.
This causes conflicts while deploying, and problems with multiple instances trying to handle the same request.

Describe the solution you'd like

Apply namespace scoping + unique names to the Webhook Configuration objects.
We have a patch prepared for our usecase as an example, and are happy to open a PR in the near future.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@thoma01
Copy link

thoma01 commented Feb 1, 2024

@antdking This patch is working for me as well! Do you think you could open a PR and we can work to get this merged into the next release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants