OVS metadata for trunk ports are not handled correctly

Bug #1625875 reported by Armando Migliaccio
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Armando Migliaccio

Bug Description

Caught during code review, there's issue when we add/remove subports. This makes the use of metadata totally unreliable. The error may lead to stale
OVS ports on the host when the trunk is removed.

Changed in neutron:
milestone: none → newton-rc2
tags: added: newton-rc-potential
Changed in neutron:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Armando Migliaccio (armando-migliaccio)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: Confirmed → In Progress
description: updated
tags: added: trunk
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/373600
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=29457631ad0b77b06dee40cb5114f5f4aced129a
Submitter: Jenkins
Branch: master

commit 29457631ad0b77b06dee40cb5114f5f4aced129a
Author: Armando Migliaccio <email address hidden>
Date: Tue Sep 20 19:34:32 2016 -0700

    Update trunk metadata during wire/unwire operations

    Before this patch metadata on the OVS trunk bridge were
    not updated during unwiring operations and upon subsequent
    wire operations old subports were getting wiped from the
    external_ids field of the OVS parent port. This could lead
    to OVS ports left behind on the host on trunk removal.

    This patch ensures that the subports stored as metadata
    reflect the logical state of the trunk.

    Closes-bug: #1625875

    Change-Id: I9d885b10d84fad43c527d3ca7bc9a37439197fc9

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

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/374938

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/newton)

Reviewed: https://review.openstack.org/374938
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=a7be741e018c54bca0610a5881085690fc6667c5
Submitter: Jenkins
Branch: stable/newton

commit a7be741e018c54bca0610a5881085690fc6667c5
Author: Armando Migliaccio <email address hidden>
Date: Tue Sep 20 19:34:32 2016 -0700

    Update trunk metadata during wire/unwire operations

    Before this patch metadata on the OVS trunk bridge were
    not updated during unwiring operations and upon subsequent
    wire operations old subports were getting wiped from the
    external_ids field of the OVS parent port. This could lead
    to OVS ports left behind on the host on trunk removal.

    This patch ensures that the subports stored as metadata
    reflect the logical state of the trunk.

    Closes-bug: #1625875

    Change-Id: I9d885b10d84fad43c527d3ca7bc9a37439197fc9
    (cherry picked from commit 29457631ad0b77b06dee40cb5114f5f4aced129a)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 9.0.0.0rc2

This issue was fixed in the openstack/neutron 9.0.0.0rc2 release candidate.

tags: removed: newton-rc-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 10.0.0.0b1

This issue was fixed in the openstack/neutron 10.0.0.0b1 development milestone.

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.