podman part of kolla_container_facts does not return list of all container facts

Bug #2058492 reported by Martin Hiner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kolla-ansible
Status tracked in Caracal
Bobcat
Fix Committed
Undecided
Unassigned
Caracal
Fix Released
Undecided
Martin Hiner

Bug Description

**Bug Report**

What happened:
  Module returned an empty list when using Podman as a container engine.

What you expected to happen:
  Module should return list with container facts of all containers when name option is left empty.

How to reproduce it (minimal and precise):
  Create a task using kolla_container_facts, set Podman as container engine, leave name option empty.

Martin Hiner (hinermar)
Changed in kolla-ansible:
assignee: nobody → Martin Hiner (hinermar)
assignee: Martin Hiner (hinermar) → nobody
status: New → Fix Committed
Martin Hiner (hinermar)
Changed in kolla-ansible:
assignee: nobody → Martin Hiner (hinermar)
status: Fix Committed → In Progress
Revision history for this message
Martin Hiner (hinermar) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (master)

Reviewed: https://review.opendev.org/c/openstack/kolla-ansible/+/912521
Committed: https://opendev.org/openstack/kolla-ansible/commit/a7c8bcda2e8b3c9d218555349d0440a9c9232fa1
Submitter: "Zuul (22348)"
Branch: master

commit a7c8bcda2e8b3c9d218555349d0440a9c9232fa1
Author: Martin Hiner <email address hidden>
Date: Tue Mar 12 11:07:09 2024 +0100

    Fix incorrect condition in kolla_container_facts

    Incorrect condition in Podman part prevented the retrieval
    of facts of all the containers when no names were provided.

    Closes-Bug: #2058492
    Change-Id: I6d7f7ca0523eb17c7d9a9b93d2037bf77f2c2a47
    Signed-off-by: Martin Hiner <email address hidden>

Changed in kolla-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/2023.2)

Fix proposed to branch: stable/2023.2
Review: https://review.opendev.org/c/openstack/kolla-ansible/+/916098

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/2023.2)

Reviewed: https://review.opendev.org/c/openstack/kolla-ansible/+/916098
Committed: https://opendev.org/openstack/kolla-ansible/commit/adc109d1b0fc6e21fd5c99677577c5c9f8a7551c
Submitter: "Zuul (22348)"
Branch: stable/2023.2

commit adc109d1b0fc6e21fd5c99677577c5c9f8a7551c
Author: Martin Hiner <email address hidden>
Date: Tue Mar 12 11:07:09 2024 +0100

    Fix incorrect condition in kolla_container_facts

    Incorrect condition in Podman part prevented the retrieval
    of facts of all the containers when no names were provided.

    Closes-Bug: #2058492
    Change-Id: I6d7f7ca0523eb17c7d9a9b93d2037bf77f2c2a47
    Signed-off-by: Martin Hiner <email address hidden>
    (cherry picked from commit a7c8bcda2e8b3c9d218555349d0440a9c9232fa1)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.