yahoo-eng-team team mailing list archive

[ZongKai LI <lzklibj@xxxxxxxxxx>]

Public bug reported:

In tunnel_type.py, method tunnel_sync, endpoint will be queried by
passed host and tunnel ip, and previous checking in tunnel_sync will
check queried endpoint with passed host and tunnel ip in multiple cases
, like whether local_ip and host has changed or upgrade.

But for case which local_ip and host are not changed will not be
checked, this will happened when ovs-agent and ovs restarted. And for
that, previous logic will try to add_endpoint again for passed tunnel ip
and host, and raise endpoint already exists warning.

It doesn't make sense, since local ip and host are not changed, and we
have queried DB twice, we don't need do another DB operation and raise a
warning.

** Affects: neutron
     Importance: Undecided
     Assignee: ZongKai LI (lzklibj)
         Status: New

** Changed in: neutron
     Assignee: (unassigned) => ZongKai LI (lzklibj)

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

Title:
  Warning Endpoint with ip already exists should be avoidable

Status in neutron:
  New

Bug description:
  In tunnel_type.py, method tunnel_sync, endpoint will be queried by
  passed host and tunnel ip, and previous checking in tunnel_sync will
  check queried endpoint with passed host and tunnel ip in multiple
  cases , like whether local_ip and host has changed or upgrade.

  But for case which local_ip and host are not changed will not be
  checked, this will happened when ovs-agent and ovs restarted. And for
  that, previous logic will try to add_endpoint again for passed tunnel
  ip and host, and raise endpoint already exists warning.

  It doesn't make sense, since local ip and host are not changed, and we
  have queried DB twice, we don't need do another DB operation and raise
  a warning.

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