[OVS] VLAN tag should be set in the Port register, not the Interface

Bug #1860329 reported by Rodolfo Alonso
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
os-vif
Fix Released
Critical
Rodolfo Alonso
Queens
New
Undecided
Unassigned
Rocky
New
Undecided
Unassigned
Stein
New
Undecided
Unassigned
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-vif (master)

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

Changed in os-vif:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
status: New → In Progress
Changed in os-vif:
assignee: Rodolfo Alonso (rodolfo-alonso-hernandez) → sean mooney (sean-k-mooney)
Revision history for this message
sean mooney (sean-k-mooney) wrote :

once this is merged ill start submitting the backport but we should bring this back to all releases where we have the security feature.

Changed in os-vif:
assignee: sean mooney (sean-k-mooney) → Rodolfo Alonso (rodolfo-alonso-hernandez)
importance: Undecided → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-vif (master)

Reviewed: https://review.opendev.org/703373
Committed: https://git.openstack.org/cgit/openstack/os-vif/commit/?id=d5b61d10655b3b7a9d32378b5f7bcdb06479e15d
Submitter: Zuul
Branch: master

commit d5b61d10655b3b7a9d32378b5f7bcdb06479e15d
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Jan 20 11:41:24 2020 +0000

    [OVS] VLAN tag should be set in the Port register

    In OVS, the VLAN tag for a device is set in the Port register,
    not the Interface [1][2]. Method "BaseOVS.create_ovs_vif_port"
    should implement it.

    [1] http://docs.openvswitch.org/en/latest/faq/configuration/
    [2] https://github.com/openstack/neutron/blob/1d354f75776fca6bf12beae70f7e3206459c84c4/neutron/agent/common/ovs_lib.py#L346-L347

    Change-Id: Iaebd42af6d5b8e3165cf10e269addae0ff3665fb
    Closes-Bug: #1860329

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

This issue was fixed in the openstack/os-vif 2.0.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.