Skip to content

[DNM] [openshift_setup] Refactor registries patch to use Ansible modules instead of ansible.builtin.shell with oc patch.#3767

Draft
nemarjan wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
nemarjan:refactor/allowed-registries-json-patch
Draft

[DNM] [openshift_setup] Refactor registries patch to use Ansible modules instead of ansible.builtin.shell with oc patch.#3767
nemarjan wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
nemarjan:refactor/allowed-registries-json-patch

Conversation

@nemarjan
Copy link

  • Refactoring task "Add additional allowed registries" as specified here ( https://github.com/openstack-k8s-operators/ci-framework/pull/3712/changes#r2858651309*) and using Ansible module to patch.

  • Merging tasks "Patch the image.config.openshift.io resource to include insecure registry" and "Patch the image.config.openshift.io resource to allow registries" in to a single task, replacing use of ansible.builtin.shell with use of Ansible module.

…stead of ansible.builtin.shell with oc patch.

- Refactoring task "Add additional allowed registries" as specified here ( https://github.com/openstack-k8s-operators/ci-framework/pull/3712/changes#r2858651309*) and using Ansible module to patch.

- Merging tasks "Patch the image.config.openshift.io resource to include insecure registry" and "Patch the image.config.openshift.io resource to allow registries" in to a single task,
  replacing use of ansible.builtin.shell with use of Ansible module.

Signed-off-by: nemarjan <nemarjan@redhat.com>
@nemarjan nemarjan requested review from a team, Valkyrie00 and evallesp March 16, 2026 11:02
@nemarjan nemarjan self-assigned this Mar 16, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 16, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 16, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign valkyrie00 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details 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

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ca2ae5211d0d421085a5d6fc89d4eb1e

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 05m 08s
podified-multinode-edpm-deployment-crc RETRY_LIMIT in 15m 03s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 23m 11s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 52m 19s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 15s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 34s

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant