openstack/oslo.messaging
Code Issues Proposed changes

Config: no need to set default=None

Browse Source 
By default oslo.cfg sets the default value as None. There is no
need to explicitly do this.

TrivialFix

Change-Id: I57fa9651ea56a180a910d6cbf7b7f385da8af239
This commit is contained in:
ChangBo Guo(gcb) 2016-06-28 15:47:39 +08:00
parent 5dfb1d8ef9
commit 61aae0ffac
2 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
oslo_messaging/_drivers/amqp1_driver/opts.py
View File

@ -32,7 +32,6 @@ amqp1_opts = [
help="address prefix when sending to any server in group"), help="address prefix when sending to any server in group"),
cfg.StrOpt('container_name', cfg.StrOpt('container_name',
default=None,
deprecated_group='amqp1', deprecated_group='amqp1',
help='Name for the AMQP container'), help='Name for the AMQP container'),
@ -62,7 +61,6 @@ amqp1_opts = [
help='Private key PEM file used to sign cert_file certificate'), help='Private key PEM file used to sign cert_file certificate'),
cfg.StrOpt('ssl_key_password', cfg.StrOpt('ssl_key_password',
default=None,
deprecated_group='amqp1', deprecated_group='amqp1',
secret=True, secret=True,
help='Password for decrypting ssl_key_file (if encrypted)'), help='Password for decrypting ssl_key_file (if encrypted)'),

8
oslo_messaging/_drivers/pika_driver/pika_connection_factory.py
View File

@ -36,15 +36,15 @@ HOST_CONNECTION_LAST_TRY_TIME = "last_try_time"
HOST_CONNECTION_LAST_SUCCESS_TRY_TIME = "last_success_try_time" HOST_CONNECTION_LAST_SUCCESS_TRY_TIME = "last_success_try_time"
pika_opts = [ pika_opts = [
cfg.IntOpt('channel_max', default=None, cfg.IntOpt('channel_max',
help='Maximum number of channels to allow'), help='Maximum number of channels to allow'),
cfg.IntOpt('frame_max', default=None, cfg.IntOpt('frame_max',
help='The maximum byte size for an AMQP frame'), help='The maximum byte size for an AMQP frame'),
cfg.IntOpt('heartbeat_interval', default=3, cfg.IntOpt('heartbeat_interval', default=3,
help="How often to send heartbeats for consumer's connections"), help="How often to send heartbeats for consumer's connections"),
cfg.BoolOpt('ssl', default=None, cfg.BoolOpt('ssl',
help='Enable SSL'), help='Enable SSL'),
cfg.DictOpt('ssl_options', default=None, cfg.DictOpt('ssl_options',
help='Arguments passed to ssl.wrap_socket'), help='Arguments passed to ssl.wrap_socket'),
cfg.FloatOpt('socket_timeout', default=0.25, cfg.FloatOpt('socket_timeout', default=0.25,
help="Set socket timeout in seconds for connection's socket"), help="Set socket timeout in seconds for connection's socket"),