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

[bitnami/thanos] Revert #29673 #29763

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[bitnami/thanos] Revert #29673 #29763

wants to merge 2 commits into from

Conversation

migruiz4
Copy link
Member

@migruiz4 migruiz4 commented Oct 3, 2024

Description of the change

Reverts #29673

Additional information

Fixes #29749

Although the fix is correct, modifying the Statefulset immutable fields is problematic and should not have been included in a patch release.

Reverting the change so users do not have to face issues while upgrading, we may include this in the next thanos major.

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

Signed-off-by: Miguel Ruiz <[email protected]>
@github-actions github-actions bot added thanos verify Execute verification workflow for these changes bitnami labels Oct 3, 2024
@github-actions github-actions bot requested a review from juan131 October 3, 2024 13:19
Signed-off-by: Bitnami Containers <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitnami thanos verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bitnami/thanos] PR #29673 is a breaking change for existing deployments
3 participants