← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1561152] Re: neutron-sanity-check generates invalid bridge names

 

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

commit 8ad9c902c20397f5ee643990ef1847e7f2ab799b
Author: Terry Wilson <twilson@xxxxxxxxxx>
Date:   Fri Mar 18 02:16:01 2016 -0500

    Ensure bridge names are shorter than max device name len
    
    Use test.base's get_rand_name function to ensure that bridge name
    lengths don't exceed the maximum device name length. For uses that
    require multiple bridges/ports with the same random characters
    appended, add get_related_rand_names and get_related_rand_device_names
    functions.
    
    Change-Id: Ib03653f3ca2d2c3d2ea7be1dff4ab0e4e77df51e
    Closes-Bug: #1561152


** Changed in: neutron
       Status: In Progress => 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/1561152

Title:
  neutron-sanity-check generates invalid bridge names

Status in neutron:
  Fix Released

Bug description:
  Instead of using neutron.tests.base.get_rand_device_name(), sanity
  check tests have been generating their own prefix name and appending a
  random string with utils.get_rand_name(). Many of the strings
  generated were too long to be device names, so ovs-vswitchd would fail
  to create the devices.

  For example:

  2016-03-18T05:40:41.950Z|07166|dpif|WARN|system@ovs-system: failed to query port patchtest-b76adc: Invalid argument
  2016-03-18T05:40:41.950Z|07167|dpif|WARN|system@ovs-system: failed to add patchtest-b76adc as port: Invalid argument

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


References