cni_ds_init unable to distinguish additional containers in kuryr-cni pod

Bug #1808969 reported by Michal Dulko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kuryr-kubernetes
Fix Released
High
Michal Dulko

Bug Description

If there are additional containers in kuryr-cni pod, cni_ds_init is prone to error when choosing on which container `runc exec` or `docker exec` should be executed.

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

Fix proposed to branch: master
Review: https://review.openstack.org/625900

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

Reviewed: https://review.openstack.org/625900
Committed: https://git.openstack.org/cgit/openstack/kuryr-kubernetes/commit/?id=65f86ae58e91d272592f1092f247f4bf956e4f77
Submitter: Zuul
Branch: master

commit 65f86ae58e91d272592f1092f247f4bf956e4f77
Author: Michał Dulko <email address hidden>
Date: Tue Dec 18 13:17:05 2018 +0100

    Lookup container name in cni_ds_init

    If your kuryr-cni pod definition consists of multiple containers (e.g.
    you've added an initContainer), cni_ds_init script is prone to error
    when looking up the container running kuryr-daemon. This is becasue it's
    only looking at pod name and namespace. This commit extends it by adding
    container.name=kuryr-cni to the conditions.

    Change-Id: Ib45f6f9f366a62d373c6f37def07e7dcc862726d
    Closes-Bug: 1808969

Changed in kuryr-kubernetes:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kuryr-kubernetes 0.6.1

This issue was fixed in the openstack/kuryr-kubernetes 0.6.1 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.