Skip to content

Fix ansible 2.19 jinja incompatibilities#1547

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
rabi:ansible_2.19
Aug 11, 2025
Merged

Fix ansible 2.19 jinja incompatibilities#1547
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
rabi:ansible_2.19

Conversation

@rabi

@rabi rabi commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

With ansible 2.19 it prints the return value of the mtu_list.append() None and the network config is not parseable with os-net-config and it fails with,

os_net_config.main No interfaces defined in config: /etc/os-net-config/config.yaml"]}

jira: OSPRH-18609

@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/5adac10636ed4f10b8e099279ce0eacd

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 01m 42s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 13m 28s
cifmw-crc-podified-edpm-baremetal FAILURE in 1h 45m 27s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 33m 36s
✔️ openstack-operator-docs-preview SUCCESS in 2m 33s

With ansible 2.19 it prints the return value of the mtu_list.append()
`None` and the network config is not parseable with os-net-config and it
fails with,

os_net_config.main No interfaces defined in config: /etc/os-net-config/config.yaml"]}

jira: OSPRH-18609
Signed-off-by: rabi <ramishra@redhat.com>
@rabi

rabi commented Jul 29, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

@rabi

rabi commented Jul 29, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-tempest

rabi added a commit to rabi/edpm-ansible that referenced this pull request Jul 30, 2025
rabi added a commit to rabi/edpm-ansible that referenced this pull request Jul 30, 2025
@rabi

rabi commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

rabi added a commit to rabi/edpm-ansible that referenced this pull request Jul 31, 2025
Non boolean conditionals are not allowed in ansible 2.19.
Fix private var access from namespace not allowed

Depends-On: openstack-k8s-operators/openstack-operator#1547
Depends-On: openstack-k8s-operators/ci-framework#3168
Signed-off-by: rabi <ramishra@redhat.com>
rabi added a commit to rabi/edpm-ansible that referenced this pull request Aug 1, 2025
Non boolean conditionals are not allowed in ansible 2.19.
Fix private var access from namespace not allowed

Depends-On: openstack-k8s-operators/openstack-operator#1547
Depends-On: openstack-k8s-operators/ci-framework#3168
Signed-off-by: rabi <ramishra@redhat.com>
rabi added a commit to rabi/edpm-ansible that referenced this pull request Aug 1, 2025
Non boolean conditionals are not allowed in ansible 2.19.
Fix private var access from namespace not allowed

Depends-On: openstack-k8s-operators/openstack-operator#1547
Depends-On: openstack-k8s-operators/ci-framework#3168
Signed-off-by: rabi <ramishra@redhat.com>
@stuggi

stuggi commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

/retest

@rabi rabi requested a review from abays August 11, 2025 11:55

@abays abays left a comment

Copy link
Copy Markdown
Contributor

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 commented Aug 11, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, rabi

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

The pull request process is described 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

@openshift-merge-bot openshift-merge-bot Bot merged commit 6d676c6 into openstack-k8s-operators:main Aug 11, 2025
12 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