yahoo-eng-team team mailing list archive

[OpenStack Infra <1323975@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/377711
Committed: https://git.openstack.org/cgit/openstack/castellan/commit/?id=02144d04dd44b6eab33af900dd6a1ded336c6c80
Submitter: Jenkins
Branch:    master

commit 02144d04dd44b6eab33af900dd6a1ded336c6c80
Author: Jiong Liu <liujiong@xxxxxxxxxxxxx>
Date:   Tue Sep 27 21:52:53 2016 +0800

    Remove default=None when set value in Config
    
    By default oslo.cfg sets the default values as None [1], there is no need
    to explicitly do this.
    
    [1] https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L829
    
    Change-Id: I97c70439c76fd17712f79371dd0ebc0ab12caabf
    Closes-bug: #1323975


** Changed in: castellan
       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/1323975

Title:
  do not use default=None for config options

Status in Aodh:
  Fix Released
Status in Barbican:
  Fix Released
Status in castellan:
  Fix Released
Status in Ceilometer:
  Fix Released
Status in Cinder:
  Fix Released
Status in congress:
  Fix Released
Status in Designate:
  Fix Released
Status in Freezer:
  In Progress
Status in Glance:
  Fix Released
Status in glance_store:
  Fix Released
Status in Gnocchi:
  Fix Released
Status in heat:
  Fix Released
Status in OpenStack Identity (keystone):
  Fix Released
Status in Magnum:
  Fix Released
Status in Manila:
  Fix Released
Status in Mistral:
  Fix Released
Status in Murano:
  Fix Released
Status in neutron:
  Fix Released
Status in OpenStack Compute (nova):
  Fix Released
Status in oslo-incubator:
  Fix Released
Status in oslo.messaging:
  Fix Released
Status in Rally:
  Fix Released
Status in Sahara:
  Fix Released
Status in OpenStack DBaaS (Trove):
  In Progress
Status in zaqar:
  Fix Released

Bug description:
  In the cfg module default=None is set as the default value. It's not
  necessary to set it again when defining config options.

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