Skip to content

Conversation

@tchap
Copy link
Contributor

@tchap tchap commented Oct 15, 2025

We need to include restricted-v3 scc in the list of rules for
authenticated users, but iff the associated ClusterRoleBinding exists.

This is a temporary and conditional fix to make tests passing for

openshift/cluster-kube-apiserver-operator#1944

A subsequent PR is gonna be opened once the PR above is merged.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 15, 2025
@openshift-ci-robot
Copy link

@tchap: This pull request explicitly references no jira issue.

In response to this:

Add "use restricted-v3 scc" to the list of expected rules for authenticated users.

Needed for openshift/cluster-kube-apiserver-operator#1935

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.

@tchap
Copy link
Contributor Author

tchap commented Oct 15, 2025

/payload-job-with-prs pull-ci-openshift-cluster-kube-apiserver-operator-main-e2e-aws openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 15, 2025

@tchap: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

@tchap
Copy link
Contributor Author

tchap commented Oct 15, 2025

/payload-job-with-prs pull-ci-openshift-cluster-kube-apiserver-operator-main-e2e-aws openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 15, 2025

@tchap: trigger 0 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

@tchap
Copy link
Contributor Author

tchap commented Oct 15, 2025

/payload-job-with-prs ci/prow/e2e-aws-ovn openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 15, 2025

@tchap: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/payload-job-with-prs pull-ci-openshift-origin-main-e2e-vsphere-ovn-upi openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: trigger 0 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/payload-with-prs openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/payload-with-prs openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/payload-with-prs 4.21 ci openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/payload-with-prs 4.21 ci all openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: An error was encountered. No known errors were detected, please see the full error message for details.

Full error message. could not resolve jobs for 4.21 ci all: failed to request latest release: server responded with 400: error: jobType must be one of '', 'informing', 'blocking', 'disabled' or 'periodic'

Please contact an administrator to resolve this issue.

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/payload-with-prs 4.21 ci all informing openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: An error was encountered. No known errors were detected, please see the full error message for details.

Full error message. could not resolve jobs for 4.21 ci all: failed to request latest release: server responded with 400: error: jobType must be one of '', 'informing', 'blocking', 'disabled' or 'periodic'

Please contact an administrator to resolve this issue.

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/payload-with-prs 4.21 ci informing openshift/cluster-kube-apiserver-operator#1935

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: trigger 1 job(s) of type informing for the ci release of OCP 4.21

  • periodic-ci-openshift-release-master-ci-4.21-upgrade-from-stable-4.20-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/d6b1e380-aa85-11f0-95a7-712e44658b77-0

@tchap
Copy link
Contributor Author

tchap commented Oct 16, 2025

/retest

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 16, 2025

@tchap: This PR was included in a payload test run from openshift/cluster-kube-apiserver-operator#1935
trigger 1 job(s) of type informing for the ci release of OCP 4.21

  • periodic-ci-openshift-release-master-ci-4.21-upgrade-from-stable-4.20-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/33e53000-aaa1-11f0-86f9-7907cfa77396-0

@tchap tchap force-pushed the test-restricted-v3 branch from 09aba2a to e744d03 Compare October 21, 2025 14:58
@tchap
Copy link
Contributor Author

tchap commented Oct 22, 2025

/retest

@tchap tchap force-pushed the test-restricted-v3 branch from e744d03 to 1435371 Compare October 22, 2025 08:38
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 22, 2025

@tchap: This pull request references CNTRLPLANE-1544 which is a valid jira issue.

In response to this:

We need to include restricted-v3 scc in the list of rules for
authenticated users, but iff the associated ClusterRoleBinding exists.

This is a temporary and conditional fix to make tests passing for

openshift/cluster-kube-apiserver-operator#1944

A subsequent PR is gonna be opened once the PR above is merged.

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.

We need to include restricted-v3 scc in the list of rules for
authenticated users, but iff the associated ClusterRoleBinding exists.

This is a temporary and conditional fix to make tests passing for

openshift/cluster-kube-apiserver-operator#1944

A subsequent PR is gonna be opened once the PR above is merged.
@tchap tchap force-pushed the test-restricted-v3 branch from 1435371 to b39fa71 Compare October 22, 2025 08:41
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 22, 2025

@tchap: This PR was included in a payload test run from openshift/cluster-kube-apiserver-operator#1944
trigger 1 job(s) of type informing for the ci release of OCP 4.21

  • periodic-ci-openshift-release-master-ci-4.21-upgrade-from-stable-4.20-e2e-aws-ovn-upgrade

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b2df3500-af37-11f0-812a-d5b1ae1849ed-0

@tchap
Copy link
Contributor Author

tchap commented Oct 22, 2025

/retest-required

@liouk
Copy link
Member

liouk commented Oct 23, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 23, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 23, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: liouk, tchap

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 23, 2025
@tchap
Copy link
Contributor Author

tchap commented Oct 23, 2025

/verified by "[sig-auth][Feature:OpenShiftAuthorization] The default cluster RBAC policy should have correct RBAC rules"

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Oct 23, 2025
@openshift-ci-robot
Copy link

@tchap: This PR has been marked as verified by "[sig-auth][Feature:OpenShiftAuthorization] The default cluster RBAC policy should have correct RBAC rules".

In response to this:

/verified by "[sig-auth][Feature:OpenShiftAuthorization] The default cluster RBAC policy should have correct RBAC rules"

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-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 0734d3a and 2 for PR HEAD b39fa71 in total

@openshift-trt
Copy link

openshift-trt bot commented Oct 23, 2025

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: b39fa71

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.

New tests seen in this PR at sha: b39fa71

  • "Import the release payload "nightly-arm64" from an external source" [Total: 4, Pass: 4, Fail: 0, Flake: 0]

1 similar comment
@openshift-trt
Copy link

openshift-trt bot commented Oct 23, 2025

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: b39fa71

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.

New tests seen in this PR at sha: b39fa71

  • "Import the release payload "nightly-arm64" from an external source" [Total: 4, Pass: 4, Fail: 0, Flake: 0]

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD d629cf1 and 1 for PR HEAD b39fa71 in total

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 24, 2025

@tchap: 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/okd-scos-e2e-aws-ovn b39fa71 link false /test okd-scos-e2e-aws-ovn

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.

@openshift-trt
Copy link

openshift-trt bot commented Oct 24, 2025

Job Failure Risk Analysis for sha: b39fa71

Job Name Failure Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-serial-2of2 IncompleteTests
Tests for this run (25) are below the historical average (1668): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: b39fa71

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.

New tests seen in this PR at sha: b39fa71

  • "Import the release payload "nightly-arm64" from an external source" [Total: 4, Pass: 4, Fail: 0, Flake: 0]

@openshift-trt
Copy link

openshift-trt bot commented Oct 24, 2025

Job Failure Risk Analysis for sha: b39fa71

Job Name Failure Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-serial-2of2 IncompleteTests
Tests for this run (25) are below the historical average (1666): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: b39fa71

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.

New tests seen in this PR at sha: b39fa71

  • "Import the release payload "nightly-arm64" from an external source" [Total: 4, Pass: 4, Fail: 0, Flake: 0]

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 450d258 and 0 for PR HEAD b39fa71 in total

@openshift-merge-bot openshift-merge-bot bot merged commit 367ab40 into openshift:main Oct 24, 2025
20 of 21 checks passed
@tchap tchap deleted the test-restricted-v3 branch October 24, 2025 12:31
@haircommander
Copy link
Member

/cherry-pick release-4.20

@openshift-cherrypick-robot

@haircommander: new pull request created: #30502

In response to this:

/cherry-pick release-4.20

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.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants