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

Increment desired nodes version on each call #6037

Merged
merged 2 commits into from
Sep 27, 2022

Conversation

pebrc
Copy link
Collaborator

@pebrc pebrc commented Sep 23, 2022

Fixes #5979

Update the version of the desired nodes history on each call by fetching the latest version before updating.

The optimisation outlined in #5979 (comment) is in a separate commit and can be backed out if we think it is too much.

@botelastic botelastic bot added the triage label Sep 23, 2022
@pebrc pebrc added >bug Something isn't working v2.5.0 labels Sep 23, 2022
@botelastic botelastic bot removed the triage label Sep 23, 2022
Copy link
Contributor

@barkbay barkbay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@pebrc pebrc merged commit d4224ae into elastic:main Sep 27, 2022
fantapsody pushed a commit to fantapsody/cloud-on-k8s that referenced this pull request Feb 7, 2023
Update the version of the desired nodes history on each call by fetching the latest version before updating.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug Something isn't working v2.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrading from ECK 2.4.0 to latest version may fail
2 participants