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

Support annotation to disable downgrade validation on all resources #5531

Closed
pebrc opened this issue Mar 29, 2022 · 0 comments · Fixed by #5680
Closed

Support annotation to disable downgrade validation on all resources #5531

pebrc opened this issue Mar 29, 2022 · 0 comments · Fixed by #5680
Assignees
Labels
>enhancement Enhancement of existing functionality good first issue Good for newcomers

Comments

@pebrc
Copy link
Collaborator

pebrc commented Mar 29, 2022

eck.k8s.elastic.co/disable-downgrade-validation=true currently only supports Elasticsearch. This is documented here https://www.elastic.co/guide/en/cloud-on-k8s/master/k8s-common-problems.html#k8s-common-problems-version-downgrade

However if a user made a mistake an upgrade to the wrong version or a downgrade is made necessary for other reasons (bug!) it is to be expected that all related resources have already been upgraded to the new version.

Currently this make the downgrade process hard because you have to scale down the operator or disable the validating webhook to force the downgrade on the non-Elasticsearch resources.

Also inconsistent behaviour between different managed applications is never good. Therefore we should support the annotation on all kinds of Elastic stack resources.

@botelastic botelastic bot added the triage label Mar 29, 2022
@pebrc pebrc added >enhancement Enhancement of existing functionality good first issue Good for newcomers and removed triage labels Mar 29, 2022
@pebrc pebrc self-assigned this May 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement Enhancement of existing functionality good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant