[multiple] Remove unnecessary registries from defaults#4013
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. |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 26m 06s |
|
recheck |
evallesp
changed the title
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nemarjan The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-merge-bot
Bot
merged commit e5cd0c5
into
openstack-k8s-operators:main
5 participants
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.