yahoo-eng-team team mailing list archive

[Kevin Benton <1289192@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

The BigSwitch plugin has helper methods for writing certificates to the
file system that are incorrectly defined.

They are missing the self argument that will be passed in. 
https://github.com/openstack/neutron/blob/7255e056092f034daaeb4246a812900645d46911/neutron/plugins/bigswitch/servermanager.py#L368
https://github.com/openstack/neutron/blob/7255e056092f034daaeb4246a812900645d46911/neutron/plugins/bigswitch/servermanager.py#L319

The unit tests were also incorrect in this case since they were
refactored at right before the merge to avoid any file-system writes
during unit tests.

** Affects: neutron
     Importance: Undecided
     Assignee: Kevin Benton (kevinbenton)
         Status: New

** Changed in: neutron
     Assignee: (unassigned) => Kevin Benton (kevinbenton)

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

Title:
  BigSwitch: certificate file helper functions incorrect

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  The BigSwitch plugin has helper methods for writing certificates to
  the file system that are incorrectly defined.

  They are missing the self argument that will be passed in. 
  https://github.com/openstack/neutron/blob/7255e056092f034daaeb4246a812900645d46911/neutron/plugins/bigswitch/servermanager.py#L368
  https://github.com/openstack/neutron/blob/7255e056092f034daaeb4246a812900645d46911/neutron/plugins/bigswitch/servermanager.py#L319

  The unit tests were also incorrect in this case since they were
  refactored at right before the merge to avoid any file-system writes
  during unit tests.

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