Public bug reported:

The error message is seen on each ovs agent resync on compute node where
there are dvr serviced ports. Resync can be triggered by any error -
this is unrelated to this bug.

The error message appears on processing distributed router port for a
subnet which is already in local_dvr_map of the agent, see
_bind_distributed_router_interface_port in ovs_dvr_neutron_agent.py:

 if subnet_uuid in self.local_dvr_map:
     ldm = self.local_dvr_map[subnet_uuid]
     csnat_ofport = ldm.get_csnat_ofport()
     if csnat_ofport == constants.OFPORT_INVALID:
         LOG.error(_LE("DVR: Duplicate DVR router interface detected "
                       "for subnet %s"), subnet_uuid)
         return

where csnat_ofport = OFPORT_INVALID by default and can only change when
the agent processes csnat port of the router - this will never happen on
compute node and we'll see the misleading log.

The proposal would be to delete the condition and the log as they're
useless.

** Affects: neutron
     Importance: Medium
     Assignee: Oleg Bondarev (obondarev)
         Status: Confirmed


** Tags: l3-dvr-backlog

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

Title:
  Misleading "DVR: Duplicate DVR router interface detected for subnet"

Status in neutron:
  Confirmed

Bug description:
  The error message is seen on each ovs agent resync on compute node
  where there are dvr serviced ports. Resync can be triggered by any
  error - this is unrelated to this bug.

  The error message appears on processing distributed router port for a
  subnet which is already in local_dvr_map of the agent, see
  _bind_distributed_router_interface_port in ovs_dvr_neutron_agent.py:

   if subnet_uuid in self.local_dvr_map:
       ldm = self.local_dvr_map[subnet_uuid]
       csnat_ofport = ldm.get_csnat_ofport()
       if csnat_ofport == constants.OFPORT_INVALID:
           LOG.error(_LE("DVR: Duplicate DVR router interface detected "
                         "for subnet %s"), subnet_uuid)
           return

  where csnat_ofport = OFPORT_INVALID by default and can only change
  when the agent processes csnat port of the router - this will never
  happen on compute node and we'll see the misleading log.

  The proposal would be to delete the condition and the log as they're
  useless.

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

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to