NO-ISSUE: switch default CI registry to quay-proxy.ci.openshift.org#1933
NO-ISSUE: switch default CI registry to quay-proxy.ci.openshift.org#1933andfasano wants to merge 1 commit into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/hold Fine tuning the non-CI access as per the latest CI docs recommendations |
f7e6631 to
4030051
Compare
4030051 to
a21bbb1
Compare
|
/hold cancel |
|
@andfasano: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
When I test this locally I don't see the |
|
/hold With this patch the |
In get_latest_ci_image() it does but there is only a registry.ci. pull-spec, not one for quay-proxy at https://amd64.ocp.releases.ci.openshift.org/api/v1/releasestream/5.0.0-0.nightly/latest So recommend adding the quay-proxy url instead of replacing registry.ci.openshift.org similar to https://github.com/openshift-metal3/dev-scripts/compare/65de2b30af827b8a5271722d276ae80dc385c620..e6cd719f0675227a509719b0e9fd11599b4708bf#diff-35a54395325de0227bbdd613dbf61c3265b5f6ab205cf32348e3538399cf1b9eL1029-L1034 |
As per the latest CI change described here https://docs.ci.openshift.org/how-tos/use-registries-in-build-farm/#container-image-data-flows, the CI default registry is now quay-proxy.ci.openshift.org