Uploaded image for project: 'Red Hat Advanced Cluster Management'
  1. Red Hat Advanced Cluster Management
  2. ACM-7631

Improve PrometheusRules, ServiceMonitors creates for ACM Hub deploy on managed OpenShift

XMLWordPrintable

    • Improve PrometheusRules creates for ACM Hub deploy on managed OpenShift
    • False
    • None
    • False
    • Green
    • To Do
    • 0% To Do, 0% In Progress, 100% Done

      Epic Goal

      Avoid the proliferation of PrometheusRules into openshift-monitoring namespace. Use a label in the user-defined ACM hub cluster namespace: the label `openshift.io/cluster-monitoring: "true"` will cause CMO to scrape a namespace

      Why is this important?

      SRE-P governs openshift-* namespaces and has explicit tooling to control what can/cannot be created in openshift-monitoring namespace

      Scenarios

      A monitoring expert can validate the label `openshift.io/cluster-monitoring: "true"` will cause CMO (platform prometheus) to scrape a namespace without needing to enable user-workload-monitoring.

      A user can deploy ACM Hub on ROSA/OSD/ARO/ROIC successfully.

      A user can deploy ACM Hub on managed OpenShift and have the same level of alerting/monitoring as an ACM Hub on self-managed OpenShift.

      A user should not have to perform any additional/different steps during their ACM hub on OpenShift anywhere.

      Acceptance Criteria

      Ensure an ACM Hub can be deployed on managed OpenShift with the SRE managed-cluster-validating-webhooks running at full speed.

      Dependencies (internal and external)

      1. managed-cluster-validating-webhooks: https://github.com/openshift/managed-cluster-validating-webhooks/
      2. Obs, GRC, Installer - any component that introduces prom rules

      Previous Work (Optional):

      1. https://docs.google.com/document/d/1gR4L5WmUW-yB7YB3dVfDHafVEkzmyExYFnaqNMHxFsQ/edit?usp=sharing

      Open questions:

      1. Migration of existing PrometheusRules from the openshift-monitoring namespace- need to ensure those are gone and not duplicative to the new label path.

      Done Checklist

      • CI - CI is running, tests are automated and merged.
      • Release Enablement <link to Feature Enablement Presentation>
      • DEV - Upstream code and tests merged: <link to meaningful PR or GitHub
        Issue>
      • DEV - Upstream documentation merged: <link to meaningful PR or GitHub
        Issue>
      • DEV - Downstream build attached to advisory: <link to errata>
      • QE - Test plans in Polarion: <link or reference to Polarion>
      • QE - Automated tests merged: <link or reference to automated tests>
      • DOC - Downstream documentation merged: <link to meaningful PR>

            mprahl Matthew Prahl
            sberens@redhat.com Scott Berens
            Nelson Jean Nelson Jean
            Christopher Dawson, Disaiah Bennett, Joe Gdaniec (Inactive), Jorge Padilla, Le Yang, Roke Jung, Subbarao Meduri, Trevor Nierman, Yuanyuan He
            Hui Chen Hui Chen
            Scott Berens Scott Berens
            Votes:
            0 Vote for this issue
            Watchers:
            11 Start watching this issue

              Created:
              Updated:
              Resolved: