Skip to content

Conversation

@JessicaJHee
Copy link
Member

Description

Adds tests for the new policyDecisionPrecedence config

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@openshift-ci
Copy link

openshift-ci bot commented Oct 17, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@JessicaJHee
Copy link
Member Author

/test e2e-tests

@qodo-merge-pro
Copy link

You are above your monthly Qodo Merge usage quota. For more information, please visit here.

@openshift-ci
Copy link

openshift-ci bot commented Oct 17, 2025

@JessicaJHee: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test e2e-ocp-helm

The following commands are available to trigger optional jobs:

/test cleanup-mapt-destroy-orphaned-aks-clusters
/test cleanup-mapt-destroy-orphaned-eks-clusters
/test e2e-aks-helm-nightly
/test e2e-aks-operator-nightly
/test e2e-eks-helm-nightly
/test e2e-eks-operator-nightly
/test e2e-gke-helm-nightly
/test e2e-gke-operator-nightly
/test e2e-ocp-helm-nightly
/test e2e-ocp-helm-sealights-nightly
/test e2e-ocp-helm-upgrade-nightly
/test e2e-ocp-operator-auth-providers-nightly
/test e2e-ocp-operator-nightly
/test e2e-ocp-v4-17-helm-nightly
/test e2e-ocp-v4-19-helm-nightly
/test e2e-osd-gcp-helm-nightly
/test e2e-osd-gcp-operator-nightly

Use /test all to run the following jobs that were automatically triggered:

pull-ci-redhat-developer-rhdh-main-e2e-ocp-helm

In response to this:

/test e2e-tests

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.

@JessicaJHee
Copy link
Member Author

/test all

@qodo-merge-pro
Copy link

You are above your monthly Qodo Merge usage quota. For more information, please visit here.

@github-actions
Copy link
Contributor

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from 8697b1e to 663e3a5 Compare October 20, 2025 19:27
@openshift-ci
Copy link

openshift-ci bot commented Oct 20, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign gustavolira for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@JessicaJHee
Copy link
Member Author

/test all

@qodo-merge-pro
Copy link

You are above your monthly Qodo Merge usage quota. For more information, please visit here.

@JessicaJHee
Copy link
Member Author

/retest

@JessicaJHee
Copy link
Member Author

/retest

@github-actions
Copy link
Contributor

@JessicaJHee
Copy link
Member Author

/retest

@JessicaJHee
Copy link
Member Author

/retest

@JessicaJHee
Copy link
Member Author

/retest

@JessicaJHee
Copy link
Member Author

/retest

@JessicaJHee JessicaJHee marked this pull request as ready for review October 22, 2025 15:37
@openshift-ci openshift-ci bot requested review from gustavolira and psrna October 22, 2025 15:37
@JessicaJHee
Copy link
Member Author

/retest

6 similar comments
@JessicaJHee
Copy link
Member Author

/retest

@JessicaJHee
Copy link
Member Author

/retest

@JessicaJHee
Copy link
Member Author

/retest

@albarbaro
Copy link
Member

/retest

@albarbaro
Copy link
Member

/retest

@JessicaJHee
Copy link
Member Author

/retest

@openshift-ci
Copy link

openshift-ci bot commented Oct 27, 2025

@JessicaJHee: 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-ocp-helm 783d0b5 link true /test e2e-ocp-helm

Full PR test history. Your PR dashboard.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants