yahoo-eng-team team mailing list archive

[Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>]

** Changed in: neutron
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1187249

Title:
  nec plugin: portinfo entries are not removed

Status in OpenStack Neutron (virtual network service):
  Fix Released
Status in neutron grizzly series:
  Fix Committed

Bug description:
  After commit a4e2442f25b63c976490882b21979a1a44963618 (the fix of bug #1160193),
  portinfo entries in the database are not removed.

  "is not" in L.674 update_ports in nec_plugin.py should be "!=".

  quantum/plugins/nec/nec_plugin.py:
      641     def update_ports(self, rpc_context, **kwargs):

      667         for id in kwargs.get('port_removed', []):
      668             portinfo = ndb.get_portinfo(session, id)
      669             if not portinfo:
      670                 LOG.debug(_("update_ports(): ignore port_removed message "
      671                             "due to portinfo for port_id=%s was not "
      672                             "registered"), id)
      673                 continue
      674             if portinfo.datapath_id is not datapath_id:   <----------- XXXXXX
      675                 LOG.debug(_("update_ports(): ignore port_removed message "
      676                             "received from different host "
      677                             "(registered_datapath_id=%(registered)s, "
      678                             "received_datapath_id=%(received)s)."),
      679                           {'registered': portinfo.datapath_id,
      680                            'received': datapath_id})

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1187249/+subscriptions