-
Notifications
You must be signed in to change notification settings - Fork 579
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
k8s: Restart redpanda after configmap changes #3262
k8s: Restart redpanda after configmap changes #3262
Conversation
9ff4b75
to
36b6545
Compare
0a40923
to
0c5eaf6
Compare
This pull request has been linked to Shortcut Story #6518: Restart node after configmap change. |
configMapHashGetter := func() (string, error) { | ||
return configMapResource.GetConfigHash(ctx) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why Getter does not takes context as an input? Couldn't we pass configMapResource.GetConfigHash to statefulset resource?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. One nit
0c5eaf6
to
176f826
Compare
176f826
to
6e555d5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Cover letter
After a user changes config in Cluster CR, the configmap is updated but we don't trigger restart of the redpanda pods so the new configuration does not get loaded until the pods are not restarted for some other reason. This adds an immediate restart to all redpanda nodes after the configmap changes. We achieve that by adding annotation with configmap hash to redpanda pods.
Fixes #2619
[ch-6518]
Release notes
Improvements
k8s: After cluster config is changed in CR, the redpanda pods will get restarted for the config to get used.
Beware that updating to this version of operator will always trigger one restart of your managed redpanda clusters after it's updated.