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

feat: Bump thanos kapps version #490

Merged
merged 1 commit into from
Jul 19, 2022
Merged

feat: Bump thanos kapps version #490

merged 1 commit into from
Jul 19, 2022

Conversation

gracedo
Copy link
Contributor

@gracedo gracedo commented Jul 19, 2022

What problem does this PR solve?:
Arbitrary kapps version bump of thanos, it doesn't look like the thanos chart is maintained anymore so I wasn't able to get an appversion bump in the upstream chart merged in time. We need a version bump to avoid the app getting automatically upgraded in all clusters when kommander (bootstrap gitrepo) is upgraded

Which issue(s) does this PR fix?:

https://jira.d2iq.com/browse/D2IQ-91678

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


If the PR adds a version bump, does it add a breaking change in License:

  • No License Change (or NA).

@gracedo gracedo added ready-for-review ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Jul 19, 2022
@gracedo gracedo self-assigned this Jul 19, 2022
@d2iq-mergebot
Copy link
Contributor

This repo has @mesosphere-mergebot integration. You can perform the following commands by submitting a comment. Submit a comment with content "@mesosphere-mergebot help" to view more detailed help text and examples. Be sure the have a look at the mergebot documentation, too.

@mesosphere-mergebot backport  
@mesosphere-mergebot autotest-on-backports-and-trains  
@mesosphere-mergebot override-status pr-status-check jira-url 
@mesosphere-mergebot ready-for-review-label  
@mesosphere-mergebot autotrigger-tests-on-ready-for-review  
@mesosphere-mergebot test  

@d2iq-mergebot
Copy link
Contributor

@mesosphere-mergebot test all

@d2iq-mergebot
Copy link
Contributor

Successfully triggered checks:

kommander-applications/upgrade-e2e-tests
kommander-applications/install-e2e-tests
kommander-applications/pre-commit
kommander-applications/manifest-validation

@gracedo gracedo requested review from takirala, bcm820, cbuto and cwyl02 July 19, 2022 21:03
@gracedo gracedo force-pushed the gracedo/thanos047 branch from ec29271 to 9893a5c Compare July 19, 2022 22:06
@d2iq-mergebot
Copy link
Contributor

@mesosphere-mergebot test all

@d2iq-mergebot
Copy link
Contributor

Successfully triggered checks:

kommander-applications/upgrade-e2e-tests
kommander-applications/install-e2e-tests
kommander-applications/pre-commit
kommander-applications/manifest-validation

@gracedo gracedo enabled auto-merge (squash) July 19, 2022 22:59
@gracedo gracedo merged commit 5d8ffc7 into main Jul 19, 2022
@gracedo gracedo deleted the gracedo/thanos047 branch July 19, 2022 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Signals mergebot that CI checks are ready to be kicked off ready-for-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants