Closed flavianmissi closed 3 months ago
@flavianmissi: This pull request references Jira Issue OCPBUGS-35335, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
/jira refresh
@flavianmissi: This pull request references Jira Issue OCPBUGS-35335, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @xiuwang
/assign @dmage
so far the test failures look unrelated to the changes.. /retest
test failures are unrelated to the changes in this PR /retest
/label qe-approved /label cherry-pick-approved
Launch a hypershift hosted cluster using payload image which built from this code. And configure external oidc auth for HCP cluster. Run the failed image registry cases in 4.15-amd64-nightly-aws-rosa-hcp-capi-stage-f7 , all suitable cases are passed.
@flavianmissi: This pull request references Jira Issue OCPBUGS-35335, which is valid.
Requesting review from QA contact: /cc @xiuwang
@flavianmissi: all tests passed!
Full PR test history. Your PR dashboard.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dmage, flavianmissi
The full list of commands accepted by this bot can be found here.
The pull request process is described here
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dmage, flavianmissi
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@flavianmissi: Jira Issue OCPBUGS-35335: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-35335 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
This PR has been included in build openshift-enterprise-registry-container-v4.17.0-202406171342.p0.gf6ef41b.assembly.stream.el9 for distgit openshift-enterprise-registry. All builds following this will include this PR.
This fixes cases where openshift is running with external users (and the user api is unavailable).
Note that the SelfAccessReview is new on k8s 1.28, and we had vendored 1.27 so I had to upgrade to the latest. Because of this api bump, backports will have to be made manually, bumping the openshift and k8s api corresponding to the openshift version (1.29 for 4.16, 1.28 for 4.15).