Error: in /etc/cni/net.d/87-podman-bridge.conflist: error parsing configuration list: unexpected end of JSON input

Bug #1889510 reported by Cédric Jeanneret
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Medium
Cédric Jeanneret

Bug Description

First report here: https://bugzilla.redhat.com/show_bug.cgi?id=1861848

We override the /etc/cni/net.d/87-podman-bridge.conflist with an invalid JSON content in https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_podman/tasks/tripleo_podman_install.yml#L46

This breaks `podman network ls' command (and probably other network related things).

tags: added: train-backport-potential ussuri-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-ansible (master)

Fix proposed to branch: master
Review: https://review.opendev.org/743911

Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-ansible (master)

Reviewed: https://review.opendev.org/743911
Committed: https://git.openstack.org/cgit/openstack/tripleo-ansible/commit/?id=dad6b77f0e0cde530f44c514da2bcb214abd9acf
Submitter: Zuul
Branch: master

commit dad6b77f0e0cde530f44c514da2bcb214abd9acf
Author: Cédric Jeanneret <email address hidden>
Date: Thu Jul 30 07:54:40 2020 +0200

    Ensure we produce a valid JSON while emptying the file

    It was detected this overrides breaks, at least, `podman network ls'
    CLI.

    Change-Id: I96382e655a03306f233963edb2aaa72953c7a531
    Closes-Bug: #1889510

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

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/744041

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-ansible (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/744042

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-ansible (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/744046

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

Reviewed: https://review.opendev.org/744042
Committed: https://git.openstack.org/cgit/openstack/tripleo-ansible/commit/?id=47e82a634349e33a941fbc039873c6445b5dd73a
Submitter: Zuul
Branch: stable/train

commit 47e82a634349e33a941fbc039873c6445b5dd73a
Author: Cédric Jeanneret <email address hidden>
Date: Thu Jul 30 07:54:40 2020 +0200

    Ensure we produce a valid JSON while emptying the file

    It was detected this overrides breaks, at least, `podman network ls'
    CLI.

    Change-Id: I96382e655a03306f233963edb2aaa72953c7a531
    Closes-Bug: #1889510
    (cherry picked from commit dad6b77f0e0cde530f44c514da2bcb214abd9acf)

tags: added: in-stable-train
tags: added: in-stable-ussuri
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-ansible (stable/ussuri)

Reviewed: https://review.opendev.org/744041
Committed: https://git.openstack.org/cgit/openstack/tripleo-ansible/commit/?id=74494f6070e7aa4cfecd1f4758f29d0bc5264026
Submitter: Zuul
Branch: stable/ussuri

commit 74494f6070e7aa4cfecd1f4758f29d0bc5264026
Author: Cédric Jeanneret <email address hidden>
Date: Thu Jul 30 07:54:40 2020 +0200

    Ensure we produce a valid JSON while emptying the file

    It was detected this overrides breaks, at least, `podman network ls'
    CLI.

    Change-Id: I96382e655a03306f233963edb2aaa72953c7a531
    Closes-Bug: #1889510
    (cherry picked from commit dad6b77f0e0cde530f44c514da2bcb214abd9acf)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-ansible (master)

Change abandoned by Emilien Macchi (<email address hidden>) on branch: master
Review: https://review.opendev.org/744046
Reason: podman network ls won't be supported in the context of TripleO, we don't use the CNI networks/bridges, therefore this command won't work.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-ansible (master)

Fix proposed to branch: master
Review: https://review.opendev.org/750780

Changed in tripleo:
milestone: none → victoria-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ansible-role-collect-logs (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/756504

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-ansible (master)

Reviewed: https://review.opendev.org/750780
Committed: https://git.openstack.org/cgit/openstack/tripleo-ansible/commit/?id=4446cbcd19e386bfb457dddf5e208a8184b8f166
Submitter: Zuul
Branch: master

commit 4446cbcd19e386bfb457dddf5e208a8184b8f166
Author: Alex Schultz <email address hidden>
Date: Wed Sep 9 12:19:01 2020 -0600

    Configure the default podman network

    Instead of blanking out the file, we need to configure it. An empty
    file causes the `podman network` to not function and prevents us from
    using containers without --net=host.

    While we don't use this in tripleo today, it is beneficial to not leave
    podman in a broken state on the undercloud or standalone if we actually
    wanted to use podman networking for an external container. This change
    won't remove the network if the interface has been configured and is in
    use as to not break 'podman network'

    It should be noted that the default interface was changed in later
    versions from cni0 to cni-podman0 see podman 3755 (podman > 1.6).

    Change-Id: Id3ba3a531b8f86d411c843683c94a9d632cbf217
    Closes-Bug: #1889510
    Closes-Bug: #1898120

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ansible-role-collect-logs (master)

Change abandoned by wes hayutin (<email address hidden>) on branch: master
Review: https://review.opendev.org/756504

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ansible-role-collect-logs (master)

Reviewed: https://review.opendev.org/756504
Committed: https://git.openstack.org/cgit/openstack/ansible-role-collect-logs/commit/?id=12723823110d6a8070778ddb8615a47f3add8de9
Submitter: Zuul
Branch: master

commit 12723823110d6a8070778ddb8615a47f3add8de9
Author: Bogdan Dobrelya <email address hidden>
Date: Wed Oct 7 14:42:29 2020 +0200

    Also collect info on podman networks

    Change-Id: Ie3a3ef258ece1b8b7d15b3b8eb71b402d39b360d
    Related-Bug: #1889510
    Related-Bug: #1898120
    Signed-off-by: Bogdan Dobrelya <email address hidden>

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-ansible 1.5.2

This issue was fixed in the openstack/tripleo-ansible 1.5.2 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-ansible 0.6.0

This issue was fixed in the openstack/tripleo-ansible 0.6.0 release.

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.