Skip to content

Conversation

@danpawlik
Copy link
Contributor

Some of the CI jobs, even when ansible-playbook execution is done from ci-framework dir, does not read group_vars/all.yml variable file. That's because the playbook execution require to have symlink to the group_vars in the playbook dir.
Until we still have nested ansible execution available in ci-framework, we should read the all.yml file using ansible extra vars parameters instead of relay that there should be symlink done in this dir.

Some of the CI jobs, even when ansible-playbook execution is done from
ci-framework dir, does not read group_vars/all.yml variable file.
That's because the playbook execution require to have symlink to the
group_vars in the playbook dir.
Until we still have nested ansible execution available in ci-framework,
we should read the all.yml file using ansible extra vars parameters
instead of relay that there should be symlink done in this dir.

Signed-off-by: Daniel Pawlik <[email protected]>
@danpawlik danpawlik requested a review from a team as a code owner October 28, 2025 16:34
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 28, 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.

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

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 39m 16s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 15m 49s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 25m 32s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 48s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 55s
✔️ cifmw-content-provider-build-images SUCCESS in 16m 41s
✔️ cifmw-edpm-build-images SUCCESS in 14m 36s
cifmw-multinode-kuttl TIMED_OUT in 2h 39m 19s
✔️ cifmw-tcib SUCCESS in 39m 23s
✔️ ci-framework-openstack-meta-content-provider SUCCESS in 14m 03s
✔️ build-push-container-cifmw-client SUCCESS in 19m 23s

@danpawlik
Copy link
Contributor Author

recheck

@stuggi
Copy link
Contributor

stuggi commented Oct 29, 2025

/lgtm with a depends-on to this PR the previously kuttl job in openstack-k8s-operators/nova-operator#1025 passed.

Copy link
Contributor

@stuggi stuggi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Oct 30, 2025
@danpawlik danpawlik merged commit b3778e5 into openstack-k8s-operators:main Oct 30, 2025
5 checks passed
@danpawlik danpawlik deleted the read-group-vars branch October 30, 2025 07:31
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