Skip to content

test: add auto cases for feature - config to not create ClusterRoleBi…#972

Open
lihongyan1 wants to merge 2 commits intorhobs:mainfrom
lihongyan1:automate_clusterrolebinding
Open

test: add auto cases for feature - config to not create ClusterRoleBi…#972
lihongyan1 wants to merge 2 commits intorhobs:mainfrom
lihongyan1:automate_clusterrolebinding

Conversation

@lihongyan1
Copy link
Contributor

@lihongyan1 lihongyan1 commented Jan 13, 2026

…nding

current case "ClusterRoleBinding NOT created with NoClusterRoleBindings policy" failed for product bug #COO-1493

@openshift-ci
Copy link

openshift-ci bot commented Jan 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lihongyan1

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:

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

…nding

case "ClusterRoleBinding NOT created with NoClusterRoleBindings policy" failed now
because product bug #COO-1493
@lihongyan1 lihongyan1 force-pushed the automate_clusterrolebinding branch from c05ee4d to 09b09d1 Compare January 13, 2026 07:43
@lihongyan1 lihongyan1 force-pushed the automate_clusterrolebinding branch from e875308 to f181017 Compare January 14, 2026 06:13
@lihongyan1
Copy link
Contributor Author

/retest

@lihongyan1
Copy link
Contributor Author

The PR is ready for review, pre-merge e2e testing prow job has only one expected failure for product bug COO-1493

<testcase name="TestMonitoringStackController/ClusterRoleBinding_NOT_created_with_NoClusterRoleBindings_policy" classname="" time="85.140">
<failure message="Failed">
<![CDATA[ assertions.go:670: MonitoringStack e2e-tests/crb-no-policy was not available - err: %!w(<nil>) | [{ResourceDiscovery 1 2026-01-14 07:20:22 +0000 UTC None Resource discovery is operational True} {Available 0 2026-01-14 07:20:22 +0000 UTC PrometheusNotAvailable False} {Reconciled 1 2026-01-14 07:20:22 +0000 UTC MonitoringStackReconciled Monitoring Stack is successfully reconciled True}] ]]>
</failure>
</testcase>

@jan--f
Copy link
Collaborator

jan--f commented Jan 16, 2026

Great, thanks a lot!
/lgtm

@jan--f
Copy link
Collaborator

jan--f commented Jan 16, 2026

/retest

@openshift-ci
Copy link

openshift-ci bot commented Jan 16, 2026

@lihongyan1: 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/observability-operator-e2e f181017 link true /test observability-operator-e2e

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.

@lihongyan1
Copy link
Contributor Author

lihongyan1 commented Jan 19, 2026

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.

2 participants