-
Notifications
You must be signed in to change notification settings - Fork 61
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
Bumping Prometheus Operator version #856
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.
This won't affect the prometheus that is deployed - but should be done in tandem with updating the prometheus on the cluster right?
there's no prometheus deployed on the cluster - we removed it a while ago because it wasn't cleaning up properly |
While we disabled the prometheus instances by default, the "operator" is still installed and at a much earlier version |
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.
Commented
d31d21d
to
51beb0d
Compare
Is prometheus updated in the clusters now? |
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.
Is prometheus updated in the clusters now?
We should take a look at the version we deploy in the kubebuild script too
We need to see that we have the correct operator versions deployed. Also, "We should take a look at the version we deploy in the kubebuild script too" from @psav can we get this looked at too? |
Looking. I would also propose to research user workload monitoring. This all could be moot if it works well for us |
/retest |
2 similar comments
/retest |
/retest |
80f080b
to
39632da
Compare
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
/retest |
No description provided.