← Back to team overview

yahoo-eng-team team mailing list archive

  1. yahoo-eng-team team
  2. Mailing list archive
  3. Message #81400

[Bug 1860560] [NEW] [ovn] lsp_set_address Exception possible when passed empty list of addresses

  Thread Previous • Date Previous • • Thread Next 
Public bug reported:

It is possible for maintenance.check_for_port_security_unknown_address()
to pass addresses=[] to lsp_set_addresses() which will fail the regex
check for it being a correct address. This matches the behavior of ovn-
nbctl's lsp-set-addresses. We should ensure that we don't pass [] to
lsp_set_addresses.

Example:

2020-01-17 16:52:32.786 53 ERROR futurist.periodics [req-0187e45b-682f-4c1d-ab26-20bc89c223e5 - - - - -] Failed to call periodic 'networking_ovn.common.maintenance.DBInconsistenciesPeriodics.check_for_port_security_unknown_address' (it runs every 600.00 seconds): TypeError: address must be router/unknown/dynamic/ethaddr ipaddr...
2020-01-17 16:52:32.786 53 ERROR futurist.periodics Traceback (most recent call last):
2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/futurist/periodics.py", line 290, in run
2020-01-17 16:52:32.786 53 ERROR futurist.periodics     work()
2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/futurist/periodics.py", line 64, in __call__
2020-01-17 16:52:32.786 53 ERROR futurist.periodics     return self.callback(*self.args, **self.kwargs)
2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/futurist/periodics.py", line 178, in decorator
2020-01-17 16:52:32.786 53 ERROR futurist.periodics     return f(*args, **kwargs)
2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/networking_ovn/common/maintenance.py", line 447, in check_for_port_security_unknown_address
2020-01-17 16:52:32.786 53 ERROR futurist.periodics     port.name, addresses=addresses).execute(check_error=True)
2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/ovsdbapp/schema/ovn_northbound/impl_idl.py", line 89, in lsp_set_addresses
2020-01-17 16:52:32.786 53 ERROR futurist.periodics     return cmd.LspSetAddressesCommand(self, port, addresses)
2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/ovsdbapp/schema/ovn_northbound/commands.py", line 305, in __init__
2020-01-17 16:52:32.786 53 ERROR futurist.periodics     "address must be router/unknown/dynamic/ethaddr ipaddr...")
2020-01-17 16:52:32.786 53 ERROR futurist.periodics TypeError: address must be router/unknown/dynamic/ethaddr ipaddr...

** Affects: neutron
     Importance: Undecided
     Assignee: Terry Wilson (otherwiseguy)
         Status: In Progress

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

Title:
  [ovn] lsp_set_address Exception possible when passed empty list of
  addresses

Status in neutron:
  In Progress

Bug description:
  It is possible for
  maintenance.check_for_port_security_unknown_address() to pass
  addresses=[] to lsp_set_addresses() which will fail the regex check
  for it being a correct address. This matches the behavior of ovn-
  nbctl's lsp-set-addresses. We should ensure that we don't pass [] to
  lsp_set_addresses.

  Example:

  2020-01-17 16:52:32.786 53 ERROR futurist.periodics [req-0187e45b-682f-4c1d-ab26-20bc89c223e5 - - - - -] Failed to call periodic 'networking_ovn.common.maintenance.DBInconsistenciesPeriodics.check_for_port_security_unknown_address' (it runs every 600.00 seconds): TypeError: address must be router/unknown/dynamic/ethaddr ipaddr...
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics Traceback (most recent call last):
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/futurist/periodics.py", line 290, in run
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics     work()
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/futurist/periodics.py", line 64, in __call__
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics     return self.callback(*self.args, **self.kwargs)
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/futurist/periodics.py", line 178, in decorator
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics     return f(*args, **kwargs)
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/networking_ovn/common/maintenance.py", line 447, in check_for_port_security_unknown_address
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics     port.name, addresses=addresses).execute(check_error=True)
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/ovsdbapp/schema/ovn_northbound/impl_idl.py", line 89, in lsp_set_addresses
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics     return cmd.LspSetAddressesCommand(self, port, addresses)
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics   File "/usr/lib/python2.7/site-packages/ovsdbapp/schema/ovn_northbound/commands.py", line 305, in __init__
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics     "address must be router/unknown/dynamic/ethaddr ipaddr...")
  2020-01-17 16:52:32.786 53 ERROR futurist.periodics TypeError: address must be router/unknown/dynamic/ethaddr ipaddr...

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

Follow ups

  Thread Previous • Date Previous • • Thread Next