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

ObservedGeneration is lost on two Ready state updates #657

Closed
burmanm opened this issue May 16, 2024 · 0 comments · Fixed by #654
Closed

ObservedGeneration is lost on two Ready state updates #657

burmanm opened this issue May 16, 2024 · 0 comments · Fixed by #654
Assignees
Labels
bug Something isn't working done Issues in the state 'done'

Comments

@burmanm
Copy link
Contributor

burmanm commented May 16, 2024

What happened?

If an update to Spec did not modify the StatefulSet (and as such causing the Updating status to be triggered) the ObservedGeneration is never updated.

The ObservedGeneration is only updated if State is changed, but Ready -> Ready even if generation has changed will not trigger this. And as such, we will remain in the wrong ObservedGeneration. This could cause unintentional restart on updates also.

What did you expect to happen?

ObservedGeneration is updated when we have finished the reconcile.

How can we reproduce it (as minimally and precisely as possible)?

Make a change that doesn't fix StatefulSet, such as abuse a bug in Canary upgrades.

cass-operator version

1.20

Kubernetes version

1.28

Method of installation

No response

Anything else we need to know?

No response

@burmanm burmanm added the bug Something isn't working label May 16, 2024
@burmanm burmanm self-assigned this May 16, 2024
@adejanovski adejanovski added the in-progress Issues in the state 'in-progress' label May 16, 2024
@adejanovski adejanovski added ready-for-review Issues in the state 'ready-for-review' and removed in-progress Issues in the state 'in-progress' labels May 17, 2024
@adejanovski adejanovski added done Issues in the state 'done' and removed ready-for-review Issues in the state 'ready-for-review' labels May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working done Issues in the state 'done'
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants