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

Watch zilla configuration for changes and apply automatically #138

Closed
jfallows opened this issue Dec 21, 2022 · 1 comment · Fixed by #141
Closed

Watch zilla configuration for changes and apply automatically #138

jfallows opened this issue Dec 21, 2022 · 1 comment · Fixed by #141
Assignees
Labels
enhancement New feature or request

Comments

@jfallows
Copy link
Contributor

jfallows commented Dec 21, 2022

After zilla is configured and running successfully, support ability to change zilla.yaml and have zilla notice to apply the changes.

Typically this will involve tearing down an existing configuration namespace, and applying the new configuration namespace. This requires tearing down any existing streams for bindings in the old namespace before applying the new namespace configuration.

Note that the new zilla.yaml contents may be invalid syntax, and should then be ignored but with a suitable error message to indicate that the attempted configuration changes were unsuccessful, so that the failure can be detected in the logs.

@jfallows jfallows added the enhancement New feature or request label Dec 21, 2022
@bmaidics bmaidics self-assigned this Jan 2, 2023
@bmaidics bmaidics mentioned this issue Jan 4, 2023
@bmaidics bmaidics linked a pull request Jan 4, 2023 that will close this issue
@jfallows jfallows changed the title Watch zilla.json configuration for changes and apply automatically Watch zilla configuration for changes and apply automatically Feb 3, 2023
@bmaidics
Copy link
Contributor

bmaidics commented Feb 7, 2023

Edit config file using kubectl:
kubectl edit configmap zilla -n <kubernetes namespace>

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

Successfully merging a pull request may close this issue.

2 participants