Skip to content

[cifmw_helpers] Improve cifmw_helpers/include_file tasks#3647

Merged
evallesp merged 1 commit into
openstack-k8s-operators:mainfrom
danpawlik:workaround-helpers
Feb 5, 2026
Merged

[cifmw_helpers] Improve cifmw_helpers/include_file tasks#3647
evallesp merged 1 commit into
openstack-k8s-operators:mainfrom
danpawlik:workaround-helpers

Conversation

@danpawlik

Copy link
Copy Markdown
Contributor

In some cases, tasks can be done as become_user, which would not work, when Ansible host would be Zuul executor (it would only work, when nested Ansible is done).

@openshift-ci

openshift-ci Bot commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

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

openshift-ci Bot commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

[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 fultonj 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

In some cases, tasks can be done as become_user, which would not work,
when Ansible host would be Zuul executor (it would only work,
when nested Ansible is done).

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@danpawlik danpawlik changed the title Improve cifmw_helpers/include_file tasks [cifmw_helpers] Improve cifmw_helpers/include_file tasks Jan 30, 2026
@danpawlik danpawlik marked this pull request as ready for review February 2, 2026 07:13
@evallesp evallesp merged commit 2633e1d into openstack-k8s-operators:main Feb 5, 2026
9 of 11 checks passed
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.

3 participants