nec plugin deletes and recreate a OFC port unexpectedly when update_port from dhcp-agent.

Bug #1278349 reported by Akihiro Motoki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Akihiro Motoki

Bug Description

nec plugin deletes and recreate a OFC port unexpectedly when update_port from dhcp-agent
even though there is no actual changes in OpenFlow port information (datapath_id and port number).

It is caused by normalization of datapath_id is different between when plugin.update_port is called and when portinfo upadte is notified from nec-plugin-agent. As a result, nec-plugin will mis-recognize datapath_id is changed and recreate a port on OpenFlow controller. This leads to unexpected temporary outage of a dhcp port.
The same normalization method should be used for both case.

Tags: nec
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/72870

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

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

commit d772fbe5ce12c2bea77c5ec2abc10ca4ae17318b
Author: Akihiro Motoki <email address hidden>
Date: Wed Feb 12 16:38:10 2014 +0900

    nec plugin: Compare OFS datapath_id as hex int

    Previously NEC plugin compares old and new datapath_ids as
    a string and zero padding in hex notation is not taken into
    account when compared. This causes unintended deletion and
    recreation of a port on OpenFlow controller. This patch fixes
    this issue by comparing datapath_ids as hex int.

    Change-Id: I6aa0a041e98c9bc489af89bb642ec5f86eaecce5
    Closes-Bug: 1278349

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: icehouse-3 → 2014.1
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.