yahoo-eng-team team mailing list archive

[Kevin Benton <1331249@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

Both the Big Switch Plugin and ML2 driver allow a server_timeout param
in the initialization methods that can never be set by a user so it
doesn't serve a purpose [1][2]. With those removed, the entire __init__
method of the base class can be removed as well.


1. https://github.com/openstack/neutron/blob/d379170109982a53544d01566ba9231d66b24ed4/neutron/plugins/bigswitch/plugin.py#L171 
2. https://github.com/openstack/neutron/blob/1a116d24a955c9e45fa8a29998d09da0350be4ab/neutron/plugins/ml2/drivers/mech_bigswitch/driver.py#L46

** Affects: neutron
     Importance: Undecided
         Status: New

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

Title:
  Big Switch: Uneccesary init method and server_timeout param

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  Both the Big Switch Plugin and ML2 driver allow a server_timeout param
  in the initialization methods that can never be set by a user so it
  doesn't serve a purpose [1][2]. With those removed, the entire
  __init__ method of the base class can be removed as well.

  
  1. https://github.com/openstack/neutron/blob/d379170109982a53544d01566ba9231d66b24ed4/neutron/plugins/bigswitch/plugin.py#L171 
  2. https://github.com/openstack/neutron/blob/1a116d24a955c9e45fa8a29998d09da0350be4ab/neutron/plugins/ml2/drivers/mech_bigswitch/driver.py#L46

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