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

More resilience to initContainers overrides #571

Closed
burmanm opened this issue Sep 26, 2023 · 0 comments · Fixed by #572
Closed

More resilience to initContainers overrides #571

burmanm opened this issue Sep 26, 2023 · 0 comments · Fixed by #572
Labels
done Issues in the state 'done' enhancement New feature or request

Comments

@burmanm
Copy link
Contributor

burmanm commented Sep 26, 2023

What is missing?

Currently, overriding the "server-config-init" caused the "server-base-config" init container to be absent as well, making users unaware that the upgrade from 4.0 -> 4.1 would cause issues in starting the container. This caused also the k8ssandra-operator to fail when using 4.1.x and medusa.

We should allow automated correction to always include both server-config-init and server-base-config, but also allow user to add extra initContainers before and after these configs. "server-config-init" should be appended as last one, "server-base-config" before it.

Why is this needed?

Users might not be aware that all the required initContainers must be defined if one is overridden.

@burmanm burmanm added the enhancement New feature or request label Sep 26, 2023
@adejanovski adejanovski added the done Issues in the state 'done' label Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Issues in the state 'done' enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants