NO-JIRA: issue-336: Sets openshift.io/cluster-monitoring as label in namespace

[bharath-b-rh]

Fixes #336

The PR is for updating the openshift.io/cluster-monitoring as a label instead of annotation in the cert-manager namespace.

[openshift-ci-robot]

@bharath-b-rh: This pull request explicitly references no jira issue.

Details

In response to this:

Fixes #336

The PR is for updating the openshift.io/cluster-monitoring as a label instead of annotation in the cert-manager namespace.

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.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bharath-b-rh

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [bharath-b-rh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e235afce-8b92-416e-94ec-754b61464c8f

📥 Commits

Reviewing files that changed from the base of the PR and between 8eb5d9e and c1f2c85.

📒 Files selected for processing (2)

• bindata/cert-manager-deployment/cert-manager-namespace.yaml
• pkg/operator/assets/bindata.go

---

Walkthrough

The changes move the openshift.io/cluster-monitoring: "true" configuration from the metadata annotations field to the metadata labels field in the cert-manager Namespace manifest across two files, aligning with OpenShift documentation requirements.

Changes

Cohort / File(s)	Summary
Cert-Manager Namespace Manifest bindata/cert-manager-deployment/cert-manager-namespace.yaml, pkg/operator/assets/bindata.go	Migrated openshift.io/cluster-monitoring: "true" from metadata.annotations to metadata.labels in the Namespace resource definition.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Assessment against linked issues

Objective	Addressed	Explanation
Move openshift.io/cluster-monitoring: "true" from annotations to labels [#336]	✅

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[bharath-b-rh]

/test e2e-operator

[openshift-ci]

@bharath-b-rh: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-operator-tech-preview	c1f2c85	link	false	/test e2e-operator-tech-preview

Full PR test history. Your PR dashboard.

Details

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 kubernetes-sigs/prow repository. I understand the commands that are listed here.