network interface changes don't initiate a restart

Bug #1551256 reported by Andy McCrae
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
High
Andy McCrae
Kilo
Fix Released
High
Andy McCrae
Liberty
Fix Released
High
Andy McCrae

Bug Description

When changing network interface options (e.g. adding static_routes) the task to make changes on the container doesn't initiate a restart of the interface or of the container itself.

This means the static_routes are never actually adopted unless you go afterward to restart the container/networking manually, or are lucky enough to have a task that fires a restart.

Due to the nature of the task itself, and being a flat "container_command" we can't use notify so will have to perform some type of check in the command itself.

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

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

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

Reviewed: https://review.openstack.org/286078
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-lxc_container_create/commit/?id=c2f43ebb7dcee8b800df9f4b8c638ae58d6476d9
Submitter: Jenkins
Branch: master

commit c2f43ebb7dcee8b800df9f4b8c638ae58d6476d9
Author: Andy McCrae <email address hidden>
Date: Mon Feb 29 13:57:25 2016 +0000

    Restart containers on network configuration change

    Changes to the network configuration need to cause a restart of the
    interface for the changes to take place.

    We can check the md5sum of the network interface config before and after
    the change, and ifup/down when appropriate.

    Change-Id: Ib6b2c167c1718710342fdf61c40fa3fe875f2293
    Closes-Bug: #1551256

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

Fix proposed to branch: liberty
Review: https://review.openstack.org/289904

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

Fix proposed to branch: kilo
Review: https://review.openstack.org/289905

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

Reviewed: https://review.openstack.org/289904
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=56a4b416150ffbcf3d0ff7c55fabd2a55a20ef8f
Submitter: Jenkins
Branch: liberty

commit 56a4b416150ffbcf3d0ff7c55fabd2a55a20ef8f
Author: Andy McCrae <email address hidden>
Date: Mon Feb 29 13:57:25 2016 +0000

    Restart containers on network configuration change

    Changes to the network configuration need to cause a restart of the
    interface for the changes to take place.

    We can check the md5sum of the network interface config before and after
    the change, and ifup/down when appropriate.

    Cherry-pick of https://review.openstack.org/#/c/286078/

    Change-Id: Ib6b2c167c1718710342fdf61c40fa3fe875f2293
    Closes-Bug: #1551256

tags: added: in-liberty
tags: added: in-kilo
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible (kilo)

Reviewed: https://review.openstack.org/289905
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=2b45722327ae28bd1e8beeec8fab1e79196c3c78
Submitter: Jenkins
Branch: kilo

commit 2b45722327ae28bd1e8beeec8fab1e79196c3c78
Author: Andy McCrae <email address hidden>
Date: Mon Feb 29 13:57:25 2016 +0000

    Restart containers on network configuration change

    Changes to the network configuration need to cause a restart of the
    interface for the changes to take place.

    We can check the md5sum of the network interface config before and after
    the change, and ifup/down when appropriate.

    Cherry-pick of https://review.openstack.org/#/c/286078/

    Change-Id: Ib6b2c167c1718710342fdf61c40fa3fe875f2293
    Closes-Bug: #1551256

Changed in openstack-ansible:
milestone: none → 13.0.0
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 11.2.11

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 12.0.8

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 11.2.12

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 12.0.9

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

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 12.0.11

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

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 11.2.14

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 11.2.15

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

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

This issue was fixed in the openstack/openstack-ansible 12.0.8 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.