Skip to content
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

[ACM-9321] Stop firing IngressWithoutClassName alert #1312

Conversation

subbarao-meduri
Copy link
Collaborator

Per https://issues.redhat.com/browse/OCPBUGS-18537, define .spec.ingressClassName so IngressWithoutClassName alert does not fire.

@bjoydeep
Copy link
Contributor

bjoydeep commented Feb 6, 2024

/lgtm

@subbarao-meduri
Copy link
Collaborator Author

/test e2e-kind

@subbarao-meduri subbarao-meduri force-pushed the fix-alert-ingresswithout-classname branch from 63d3da8 to 4cd6195 Compare February 6, 2024 23:06
@openshift-ci openshift-ci bot removed the lgtm label Feb 6, 2024
@saswatamcode
Copy link
Member

/retest

Copy link
Contributor

@douglascamata douglascamata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Copy link

openshift-ci bot commented Feb 9, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bjoydeep, douglascamata, subbarao-meduri

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

The pull request process is described here

Needs approval from an approver in each of these files:

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

Copy link

sonarqubecloud bot commented Feb 9, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@openshift-merge-bot openshift-merge-bot bot merged commit 4f57180 into stolostron:main Feb 9, 2024
15 checks passed
coleenquadros pushed a commit to coleenquadros/multicluster-observability-operator that referenced this pull request Feb 10, 2024
philipgough pushed a commit to philipgough/multicluster-observability-operator that referenced this pull request Feb 13, 2024
openshift-merge-bot bot pushed a commit that referenced this pull request Feb 13, 2024
…rt (#1312) (#1327)

Signed-off-by: Subbarao Meduri <[email protected]>
Co-authored-by: Subbarao Meduri <[email protected]>
@subbarao-meduri subbarao-meduri deleted the fix-alert-ingresswithout-classname branch March 13, 2024 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants