Skip to content

Add include_file and include_dir helpers#3406

Merged
danpawlik merged 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:include-files-helper
Oct 14, 2025
Merged

Add include_file and include_dir helpers#3406
danpawlik merged 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:include-files-helper

Conversation

@danpawlik
Copy link
Copy Markdown
Contributor

The future group_vars directory, might not contain "clear" variables in the yaml file - instead of each variable has own type like string, integer, bool etc. it would be jinja2 variable, which is not "translated" in Ansible during setting facts.

This helper would be useful for parsing group_vars variables, that might be based on jinja2 vars.

@danpawlik danpawlik requested a review from a team as a code owner October 13, 2025 15:39
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 13, 2025

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

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
Copy Markdown

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/bc0c7702e7a543e08f301540a55455cf

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 41m 10s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 15m 30s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 26m 23s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 44s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 42s
cifmw-pod-pre-commit FAILURE in 7m 58s
✔️ build-push-container-cifmw-client SUCCESS in 23m 19s
✔️ cifmw-molecule-cifmw_helpers SUCCESS in 1m 53s

danpawlik pushed a commit to danpawlik/ci-framework that referenced this pull request Oct 13, 2025
Depends-On: openstack-k8s-operators#3404
Depends-On: openstack-k8s-operators#3406

Co-Authored-By: Daniel Pawlik <dpawlik@redhat.com>

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@danpawlik danpawlik force-pushed the include-files-helper branch from 3b7e5d8 to 7c86154 Compare October 14, 2025 06:43
amartyasinha
amartyasinha previously approved these changes Oct 14, 2025
The future group_vars directory, might not contain "clear"
variables in the yaml file - instead of each variable has
own type like string, integer, bool etc. it would be jinja2 variable,
which is not "translated" in Ansible during setting facts.

This helper would be useful for parsing group_vars variables, that
might be based on jinja2 vars.

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@danpawlik
Copy link
Copy Markdown
Contributor Author

it does not like 'raw' format in markdown

@danpawlik danpawlik enabled auto-merge (rebase) October 14, 2025 14:39
@danpawlik danpawlik merged commit 0d6c58a into openstack-k8s-operators:main Oct 14, 2025
5 checks passed
@danpawlik danpawlik deleted the include-files-helper branch October 14, 2025 14:58
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