-
Notifications
You must be signed in to change notification settings - Fork 183
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
Update prometheus operator helm chart #667
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we spot some suspicious changes? Thats a lot of versions 🙀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sumo-drosiek I think the request from Frank was to verify and bump up all of our dependency chart versions to latest if possible, LGTM assuming we sanity checked - we can run more extensive verification pre-release
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@samjsong That makes sense. LGTM then :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@sumo-drosiek I did a sanity check of the release changes in the versions and I did not spot anything that looked suspicious. However, we can bump, allow e2e to run, and then sanity check the apps as well to confirm. |
Update prometheus operator helm chart
d052360
to
3293cd5
Compare
Description
Fill in your description here.
Testing performed