yahoo-eng-team team mailing list archive

[Launchpad Bug Tracker <1104001@xxxxxxxxxxxxxxxxxx>]

[Expired for neutron because there has been no activity for 60 days.]

** Changed in: neutron
       Status: Incomplete => Expired

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

Title:
  Wrong default values when creating HealthMonitor

Status in neutron:
  Expired

Bug description:
  in quantum/extensions/loadbalancer.py we set default values to 3 attributes of HealthMonitor.
  Code:
  health_monitors': {
        ...
          'http_method': {'allow_post': True, 'allow_put': True,
                          'validate': {'type:string': None},
                          'default': 'GET',
                          'is_visible': True},
          'url_path': {'allow_post': True, 'allow_put': True,
                       'validate': {'type:string': None},
                       'default': '/',
                       'is_visible': True},
          'expected_codes': {'allow_post': True, 'allow_put': True,
                             'validate': {
                                 'type:regex':
                                 '^(\d{3}(\s*,\s*\d{3})*)$|^(\d{3}-\d{3})$'},
                             'default': '200',
                             'is_visible': True},
          ...

  Those default values are OK only the the 'type' attribute is HTTP/S. 
  So it looks like there should be no defaults values and the values in the DB should be null unless the user specified the values.

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