-
Notifications
You must be signed in to change notification settings - Fork 607
Move to apiserver_request_sli_
metric
#874
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
Move to apiserver_request_sli_
metric
#874
Conversation
The older `apiserver_request_sli_.*` metrics have been name changed starting in Kubernetes v1.26. kubernetes/kubernetes#112679
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
I'm not merging as I'm not sure that we only target Kube v1.26 and above on main now. If someone knows this to be true, feel free to merge. |
I guess we'll see and figure out. |
@metalmatze should we update the compatibility matrix? |
Yes, please send a PR. |
yes the current OpenShift release (4.13) uses 1.26. So that works for us. |
The older
apiserver_request_sli_.*
metrics have been name changed starting in Kubernetes v1.26.kubernetes/kubernetes#112679