[DNM] [multiple] Remove unnecessary registries from defaults#4013
[DNM] [multiple] Remove unnecessary registries from defaults#4013Valkyrie00 wants to merge 1 commit into
Conversation
Remove registries that are not needed in upstream defaults. These are now supplied via cifmw_openshift_setup_allowed_extra_registries at the job level. Also cleaned up related references from comments and documentation. Signed-off-by: Vito Castellano <vcastell@redhat.com>
|
Skipping CI for Draft Pull Request. |
|
[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 |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 26m 06s |
1 participant
Summary
Remove registries that are not needed in upstream defaults from the
openshift_setuprole.A mechanism already exists to supply additional registries at the job level via
cifmw_openshift_setup_allowed_extra_registries, which is consumed inroles/openshift_setup/tasks/configure_registries.ymland merged into the finalallowedRegistrieslist.Note
A hardcoded registry reference was found in
hooks/playbooks/config_cluster_for_disconnected_deployment.yml. This needs to be investigated and removed in a dedicated follow-up task.