[release-4.20] OCPBUGS-105211: Apply MetricsSet relabel configs to KAS ServiceMonitor - #9235
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
@redhat-chai-bot: This pull request references Jira Issue OCPBUGS-105211, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
/jira refresh |
|
@wanghaoran1988: This pull request references Jira Issue OCPBUGS-105211, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
…onitor The KAS adaptServiceMonitor() function was missing the line that applies MetricRelabelConfigs from the configured MetricsSet (Telemetry/SRE/All). Every other component (etcd, KCM, CVO, scheduler, OAPI, OCM, routecm, NTO, OLM, catalog-operator) correctly calls its corresponding metrics.XxxRelabelConfigs(cpContext.MetricsSet), but KAS did not. This caused the KAS ServiceMonitor to always use the hardcoded Telemetry default from the YAML asset (3 metrics: apiserver_storage_objects, apiserver_request_total, apiserver_current_inflight_requests), ignoring the sre-metric-set ConfigMap entirely when METRICS_SET=SRE. The fix adds the missing call to metrics.KASRelabelConfigs() and a unit test matching the pattern used by KCM (kcm/servicemonitor_test.go). (cherry picked from commit 707e294)
575b7f4 to
70ac5d7
Compare
|
/retest AI-generated. Review for accuracy. |
|
/test e2e-azure-aks-external-oidc AI-generated. Review for accuracy. |
|
Pipeline controller notification No second-stage tests were triggered for this PR. This can happen when:
Use |
|
/cc @bryan-cox |
|
/verified by @csrwng |
|
@csrwng: This PR has been marked as verified by DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
/jira refresh |
|
@wanghaoran1988: This pull request references Jira Issue OCPBUGS-105211, which is invalid:
Comment DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mehabhalodiya, muraee, redhat-chai-bot The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/jira refresh |
|
@wanghaoran1988: This pull request references Jira Issue OCPBUGS-105211, which is valid. The bug has been moved to the POST state. 7 validation(s) were run on this bug
DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
/retest-required AI-generated. Review for accuracy. |
|
/test e2e-kubevirt-aws-ovn-reduced AI-generated. Review for accuracy. |
|
/test e2e-aws AI-generated. Review for accuracy. |
|
/test e2e-aws AI-generated. Review for accuracy. |
|
/test e2e-kubevirt-aws-ovn-reduced AI-generated. Review for accuracy. |
|
/test e2e-aws AI-generated. Review for accuracy. |
|
/test e2e-kubevirt-aws-ovn-reduced AI-generated. Review for accuracy. |
1 similar comment
|
/test e2e-kubevirt-aws-ovn-reduced AI-generated. Review for accuracy. |
|
@redhat-chai-bot: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
3ddd15f
into
openshift:release-4.20
|
@redhat-chai-bot: Jira Issue Verification Checks: Jira Issue OCPBUGS-105211 Jira Issue OCPBUGS-105211 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
Fix included in release 4.20.0-0.nightly-2026-08-14-105317 |
Cherry-pick of #8715 to release-4.20.
Backport of the fix for OCPBUGS-105211 / OCPBUGS-91997.
The KAS (kube-apiserver) ServiceMonitor was not applying MetricsSet relabel configs. This cherry-pick brings the fix to the 4.20 release branch.
Conflict resolution: The GCP testdata fixture (
testdata/kube-apiserver/GCP/zz_fixture_TestControlPlaneComponents_kube_apiserver_servicemonitor.yaml) was deleted during cherry-pick because it does not exist onrelease-4.20(the broader GCP fixture set was added in main-only commits). All other fixtures applied cleanly. Build and test verification passed./jira-link OCPBUGS-105211
AI-generated. Review for accuracy.
@wanghaoran1988 requested in Slack thread