ofagent incorrectly uses OF1.0 VLAN semantics

Bug #1301144 reported by YAMAMOTO Takashi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
YAMAMOTO Takashi
Icehouse
Fix Released
Undecided
Unassigned

Bug Description

while ofagent uses OF1.3, it incorrectly uses OF1.0 terms in some places.

namely, _local_vlan_for_flat uses 0xffff to mean "no VLAN".
it should use OFPVID_NONE and pop_vlan/push_vlan appropriately.

the same problem exists for reclaim_local_vlan.

tags: added: icehouse-rc-potential
tags: added: icehouse-backport-potential openflowagent
removed: icehouse-rc-potential
Changed in neutron:
importance: Undecided → Medium
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/85649

Changed in neutron:
assignee: nobody → YAMAMOTO Takashi (yamamoto)
status: New → In Progress
tags: added: ml2
Kyle Mestery (mestery)
Changed in neutron:
milestone: none → juno-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/85649
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=35b0e003e59232d63b8dbd0da511ee8e3877335e
Submitter: Jenkins
Branch: master

commit 35b0e003e59232d63b8dbd0da511ee8e3877335e
Author: YAMAMOTO Takashi <email address hidden>
Date: Wed Apr 2 15:41:38 2014 +0900

    ofagent: Fix VLAN usage for TYPE_FLAT and TYPE_VLAN

    while ofagent uses OF1.3, the current coding incorrectly uses
    OF1.0 terms in some places. namely, _local_vlan_for_flat uses
    0xffff to mean "no VLAN". it should use OFPVID_NONE and
    pop_vlan/push_vlan appropriately. the same problem exists for
    reclaim_local_vlan.

    Closes-Bug: 1301144
    Change-Id: I3df821fd72471f8bd84366e3b5a1cc7e3489156c

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

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/100380

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

Reviewed: https://review.openstack.org/100380
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=1da7abde2e5354c2bcd088d2de25254742ea01fe
Submitter: Jenkins
Branch: stable/icehouse

commit 1da7abde2e5354c2bcd088d2de25254742ea01fe
Author: YAMAMOTO Takashi <email address hidden>
Date: Wed Apr 2 15:41:38 2014 +0900

    ofagent: Fix VLAN usage for TYPE_FLAT and TYPE_VLAN

    while ofagent uses OF1.3, the current coding incorrectly uses
    OF1.0 terms in some places. namely, _local_vlan_for_flat uses
    0xffff to mean "no VLAN". it should use OFPVID_NONE and
    pop_vlan/push_vlan appropriately. the same problem exists for
    reclaim_local_vlan.

    Closes-Bug: 1301144
    Change-Id: I3df821fd72471f8bd84366e3b5a1cc7e3489156c
    (cherry picked from commit 35b0e003e59232d63b8dbd0da511ee8e3877335e)

tags: added: in-stable-icehouse
Chuck Short (zulcss)
tags: removed: icehouse-backport-potential
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-1 → 2014.2
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.