Fix infinite loop in seed container deployment

The deploy.yml playbook previously called deploy-container.yml, but this
was changed to call itself when podman support was added [1]. It would
then recursively loop until failure. This change reverts that, so
deploy-container.yml is called instead.

Add a test to seed jobs, to catch any regression in CI.

[1] https://review.opendev.org/c/openstack/kayobe/+/909686

Closes-Bug: #2114845
Change-Id: I825ab7331fbe8f28c17b87ff79a436144426a78c
This commit is contained in:
Alex-Welsh
2025-06-18 10:09:56 +01:00
committed by Pierre Riteau
parent 9e18a6e02a
commit 7bd7929ea7
3 changed files with 19 additions and 1 deletions

View File

@@ -10,7 +10,7 @@
become: "{{ container_engine == 'podman' }}"
- name: Deploy containers (loop)
include_tasks: deploy.yml
include_tasks: deploy-container.yml
vars:
container_name: "{{ item.key }}"
container_config: "{{ item.value }}"

View File

@@ -57,3 +57,16 @@ kolla_base_distro: "{% raw %}{{ 'rocky' if os_distribution == 'centos' else os_d
# Support overriding container_engine
container_engine: "{{ container_engine }}"
# Test deployment of custom seed containers
seed_containers:
node_exporter:
image: "quay.io/prometheus/node-exporter"
tag: "latest"
{% if ansible_os_family == "Debian" %}
# TODO(priteau): Required for running some containers on Ubuntu + Podman.
# Remove once installed by ansible-collection-kolla.
dev_tools_packages_extra:
- "catatonit"
{% endif %}

View File

@@ -0,0 +1,5 @@
---
fixes:
- |
Fixes an infinite loop when deploying seed containers.
`LP#2114845 <https://bugs.launchpad.net/kayobe/+bug/2114845>`__