]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Don't use default=None for config options
authorNate Potter <nathaniel.potter@intel.com>
Tue, 27 Oct 2015 16:09:40 +0000 (16:09 +0000)
committerSean McGinnis <sean.mcginnis@gmail.com>
Sat, 7 Nov 2015 23:20:39 +0000 (23:20 +0000)
In the config module default=None is set as the default value,
so it isn't necessary to set it again when doing config options.

Change-Id: I09989ab679d249a9f7dea6af5d53c14dd1726e09
Closes-Bug: #1323975

48 files changed:
cinder/api/common.py
cinder/api/views/versions.py
cinder/backup/drivers/glusterfs.py
cinder/backup/drivers/nfs.py
cinder/backup/drivers/swift.py
cinder/cmd/volume_usage_audit.py
cinder/common/config.py
cinder/compute/nova.py
cinder/context.py
cinder/tests/unit/test_service.py
cinder/tests/unit/volume/drivers/emc/scaleio/mocks.py
cinder/volume/driver.py
cinder/volume/drivers/blockbridge.py
cinder/volume/drivers/cloudbyte/options.py
cinder/volume/drivers/datera.py
cinder/volume/drivers/dothill/dothill_common.py
cinder/volume/drivers/emc/emc_vnx_cli.py
cinder/volume/drivers/emc/scaleio.py
cinder/volume/drivers/hitachi/hbsd_common.py
cinder/volume/drivers/hitachi/hbsd_horcm.py
cinder/volume/drivers/hitachi/hbsd_iscsi.py
cinder/volume/drivers/huawei/huawei_driver.py
cinder/volume/drivers/ibm/gpfs.py
cinder/volume/drivers/ibm/storwize_svc/__init__.py
cinder/volume/drivers/lenovo/lenovo_common.py
cinder/volume/drivers/netapp/options.py
cinder/volume/drivers/nfs.py
cinder/volume/drivers/pure.py
cinder/volume/drivers/quobyte.py
cinder/volume/drivers/rbd.py
cinder/volume/drivers/remotefs.py
cinder/volume/drivers/san/hp/hp_lefthand_rest_proxy.py
cinder/volume/drivers/san/hp/hp_xp_opts.py
cinder/volume/drivers/san/hp/hpmsa_common.py
cinder/volume/drivers/scality.py
cinder/volume/drivers/solidfire.py
cinder/volume/drivers/srb.py
cinder/volume/drivers/tintri.py
cinder/volume/drivers/violin/v6000_common.py
cinder/volume/drivers/vmware/vmdk.py
cinder/volume/drivers/vzstorage.py
cinder/volume/manager.py
cinder/wsgi/eventlet_server.py
cinder/zonemanager/drivers/brocade/brcd_fabric_opts.py
cinder/zonemanager/drivers/brocade/brcd_fc_zone_driver.py
cinder/zonemanager/drivers/cisco/cisco_fabric_opts.py
cinder/zonemanager/drivers/cisco/cisco_fc_zone_driver.py
cinder/zonemanager/fc_zone_manager.py

index 793da39aab5079c73bcc4407cca6aecbed9032d5..0352ddd4a2dc113b058053c1934e3b5bb27a34c3 100644 (file)
@@ -35,7 +35,6 @@ api_common_opts = [
                help='The maximum number of items that a collection '
                     'resource returns in a single response'),
     cfg.StrOpt('osapi_volume_base_URL',
-               default=None,
                help='Base URL that will be presented to users in links '
                     'to the OpenStack Volume API',
                deprecated_name='osapi_compute_link_prefix'),
index d2471925bb80d13892d855c966ed6007f7c183e3..1129f3470b7fa6297e372a6ee46e93e0c2b8a9a0 100644 (file)
@@ -20,7 +20,7 @@ from oslo_config import cfg
 
 
 versions_opts = [
-    cfg.StrOpt('public_endpoint', default=None,
+    cfg.StrOpt('public_endpoint',
                help="Public url to use for versions endpoint. The default "
                     "is None, which will use the request's host_url "
                     "attribute to populate the URL base. If Cinder is "
index 4da838fe6c260a12f2af401b23cbb44623802c6e..7e274b5057c2d0be42720148fe0a204541fdeea6 100644 (file)
@@ -32,7 +32,6 @@ glusterfsbackup_service_opts = [
                default='$state_path/backup_mount',
                help='Base dir containing mount point for gluster share.'),
     cfg.StrOpt('glusterfs_backup_share',
-               default=None,
                help='GlusterFS share in '
                     '<hostname|ipv4addr|ipv6addr>:<gluster_vol_name> format. '
                     'Eg: 1.2.3.4:backup_vol'),
index 3a884b29ab514e499f496ffb455022bc6f6df3a9..b3d4d36512a6c988d942ed26c8ac1ab8a581a741 100644 (file)
@@ -33,11 +33,9 @@ nfsbackup_service_opts = [
                default='$state_path/backup_mount',
                help='Base dir containing mount point for NFS share.'),
     cfg.StrOpt('backup_share',
-               default=None,
                help='NFS share in hostname:path, ipv4addr:path, '
                     'or "[ipv6addr]:path" format.'),
     cfg.StrOpt('backup_mount_options',
-               default=None,
                help=('Mount options passed to the NFS client. See NFS '
                      'man page for details.')),
 ]
index 1525e791e85f713fc7fc19f2199fe940c827daa2..eb00133b7cb4085da6b2048f00895c4e583e5493 100644 (file)
@@ -61,10 +61,8 @@ LOG = logging.getLogger(__name__)
 
 swiftbackup_service_opts = [
     cfg.StrOpt('backup_swift_url',
-               default=None,
                help='The URL of the Swift endpoint'),
     cfg.StrOpt('backup_swift_auth_url',
-               default=None,
                help='The URL of the Keystone endpoint'),
     cfg.StrOpt('swift_catalog_info',
                default='object-store:swift:publicURL',
@@ -86,14 +84,11 @@ swiftbackup_service_opts = [
                help='Swift authentication version. Specify "1" for auth 1.0'
                     ', or "2" for auth 2.0'),
     cfg.StrOpt('backup_swift_tenant',
-               default=None,
                help='Swift tenant/account name. Required when connecting'
                     ' to an auth 2.0 system'),
     cfg.StrOpt('backup_swift_user',
-               default=None,
                help='Swift user name'),
     cfg.StrOpt('backup_swift_key',
-               default=None,
                help='Swift key for authentication'),
     cfg.StrOpt('backup_swift_container',
                default='volumebackups',
@@ -119,7 +114,6 @@ swiftbackup_service_opts = [
                      'up the volume to the Swift backend storage. The '
                      'default value is True to enable the timer.'),
     cfg.StrOpt('backup_swift_ca_cert_file',
-               default=None,
                help='Location of the CA certificate file to use for swift '
                     'client requests.'),
     cfg.BoolOpt('backup_swift_auth_insecure',
index b5123e508b32175fd1407e317a836e4327a2221e..23b9a4a1cfaaa2a88f38617d03036428a491d7fc 100644 (file)
@@ -55,12 +55,10 @@ import cinder.volume.utils
 CONF = cfg.CONF
 script_opts = [
     cfg.StrOpt('start_time',
-               default=None,
                help="If this option is specified then the start time "
                     "specified is used instead of the start time of the "
                     "last completed audit period."),
     cfg.StrOpt('end_time',
-               default=None,
                help="If this option is specified then the end time "
                     "specified is used instead of the end time of the "
                     "last completed audit period."),
index de45905fa70a53397161a53f16040423df088101..79033bc03cda3351f1f95fa1c533800b95d4299c 100644 (file)
@@ -88,7 +88,6 @@ global_opts = [
                help='Location of ca certificates file to use for glance '
                     'client requests.'),
     cfg.IntOpt('glance_request_timeout',
-               default=None,
                help='http/https timeout value for glance operations. If no '
                     'value (None) is supplied here, the glanceclient default '
                     'value is used.'),
@@ -136,7 +135,6 @@ global_opts = [
                default='nova',
                help='Availability zone of this node'),
     cfg.StrOpt('default_availability_zone',
-               default=None,
                help='Default availability zone for new volumes. If not set, '
                     'the storage_availability_zone option value is used as '
                     'the default for new volumes.'),
@@ -147,7 +145,6 @@ global_opts = [
                      'default_availability_zone, then '
                      'storage_availability_zone, instead of failing.'),
     cfg.StrOpt('default_volume_type',
-               default=None,
                help='Default volume type to use'),
     cfg.StrOpt('volume_usage_audit_period',
                default='month',
@@ -178,7 +175,6 @@ global_opts = [
                choices=['noauth', 'keystone'],
                help='The strategy to use for auth. Supports noauth or keystone.'),
     cfg.ListOpt('enabled_backends',
-                default=None,
                 help='A list of backend names to use. These backend names '
                      'should be backed by a unique [CONFIG] group '
                      'with its options'),
@@ -195,21 +191,17 @@ global_opts = [
                default='cinder.consistencygroup.api.API',
                help='The full class name of the consistencygroup API class'),
     cfg.StrOpt('os_privileged_user_name',
-               default=None,
                help='OpenStack privileged account username. Used for requests '
                     'to other services (such as Nova) that require an account '
                     'with special rights.'),
     cfg.StrOpt('os_privileged_user_password',
-               default=None,
                help='Password associated with the OpenStack privileged '
                     'account.',
                secret=True),
     cfg.StrOpt('os_privileged_user_tenant',
-               default=None,
                help='Tenant name associated with the OpenStack privileged '
                     'account.'),
     cfg.StrOpt('os_privileged_user_auth_url',
-               default=None,
                help='Auth URL associated with the OpenStack privileged '
                     'account.'),
 ]
index 58a349da5f32458cbc12e47966a8de083ec4173b..600e823d528437fe8b8f7f70e9e786b53f1594a5 100644 (file)
@@ -39,17 +39,13 @@ nova_opts = [
                default='compute:Compute Service:adminURL',
                help='Same as nova_catalog_info, but for admin endpoint.'),
     cfg.StrOpt('nova_endpoint_template',
-               default=None,
                help='Override service catalog lookup with template for nova '
                     'endpoint e.g. http://localhost:8774/v2/%(project_id)s'),
     cfg.StrOpt('nova_endpoint_admin_template',
-               default=None,
                help='Same as nova_endpoint_template, but for admin endpoint.'),
     cfg.StrOpt('os_region_name',
-               default=None,
                help='Region name of this node'),
     cfg.StrOpt('nova_ca_certificates_file',
-               default=None,
                help='Location of ca certificates file to use for nova client '
                     'requests.'),
     cfg.BoolOpt('nova_api_insecure',
index 08274fe76fed58cee5c74f2daa08c8aba178c2df..8c43d98843553fa3cf5f5fe9020418df6a9f1ecc 100644 (file)
@@ -30,11 +30,9 @@ from cinder import policy
 
 context_opts = [
     cfg.StrOpt('cinder_internal_tenant_project_id',
-               default=None,
                help='ID of the project which will be used as the Cinder '
                     'internal tenant.'),
     cfg.StrOpt('cinder_internal_tenant_user_id',
-               default=None,
                help='ID of the user to be used in volume operations as the '
                     'Cinder internal tenant.'),
 ]
index 5303f893807141eae48d2f0db26284cfb1ecb0ac..fee6b60b800145ec1978506b99efa5a564ad32e9 100644 (file)
@@ -40,7 +40,6 @@ test_service_opts = [
                default="cinder.tests.unit.test_service.FakeManager",
                help="Manager for testing"),
     cfg.StrOpt("test_service_listen",
-               default=None,
                help="Host to bind test service to"),
     cfg.IntOpt("test_service_listen_port",
                default=0,
index 5b8c630f699e30435a7bb2d9049f67d77dbabeb2..660ff2287f52016ac665dd915ba91caba484a72b 100644 (file)
@@ -29,7 +29,7 @@ class ScaleIODriver(scaleio.ScaleIODriver):
     def __init__(self, *args, **kwargs):
         configuration = conf.Configuration(
             [
-                cfg.StrOpt('fake', default=None),
+                cfg.StrOpt('fake'),
             ],
             None
         )
index 1b6f438930d7c1ad412a9d14b02c7765c29190d5..f69f0e8b1e47cace8c650639034f942540e27027 100644 (file)
@@ -67,7 +67,6 @@ volume_opts = [
                help='The maximum number of times to rescan targets'
                     ' to find volume'),
     cfg.StrOpt('volume_backend_name',
-               default=None,
                help='The backend name for a given driver implementation'),
     cfg.BoolOpt('use_multipath_for_image_xfer',
                 default=False,
@@ -86,7 +85,6 @@ volume_opts = [
                default=0,
                help='Size in MiB to wipe at start of old volumes. 0 => all'),
     cfg.StrOpt('volume_clear_ionice',
-               default=None,
                help='The flag to pass to ionice to alter the i/o priority '
                     'of the process used to zero a volume after deletion, '
                     'for example "-c3" for idle only priority.'),
@@ -150,11 +148,9 @@ volume_opts = [
                     'with the value "iser". The supported iSCSI protocol '
                     'values are "iscsi" and "iser".'),
     cfg.StrOpt('driver_client_cert_key',
-               default=None,
                help='The path to the client certificate key for verification, '
                     'if the driver supports it.'),
     cfg.StrOpt('driver_client_cert',
-               default=None,
                help='The path to the client certificate for verification, '
                     'if the driver supports it.'),
     cfg.BoolOpt('driver_use_ssl',
@@ -173,7 +169,6 @@ volume_opts = [
                       'capacity. A ratio lower than 1.0 will be ignored and '
                       'the default value will be used instead.'),
     cfg.StrOpt('scst_target_iqn_name',
-               default=None,
                help='Certain ISCSI targets have predefined target names, '
                     'SCST target driver uses this name.'),
     cfg.StrOpt('scst_target_driver',
@@ -195,16 +190,13 @@ volume_opts = [
                deprecated_opts=deprecated_chap_password_opts,
                secret=True),
     cfg.StrOpt('driver_data_namespace',
-               default=None,
                help='Namespace for driver private data values to be '
                     'saved in.'),
     cfg.StrOpt('filter_function',
-               default=None,
                help='String representation for an equation that will be '
                     'used to filter hosts. Only used when the driver '
                     'filter is set to be used by the Cinder scheduler.'),
     cfg.StrOpt('goodness_function',
-               default=None,
                help='String representation for an equation that will be '
                     'used to determine the goodness of a host. Only used '
                     'when using the goodness weigher is set to be used by '
@@ -214,13 +206,11 @@ volume_opts = [
                 help='If set to True the http client will validate the SSL '
                      'certificate of the backend endpoint.'),
     cfg.ListOpt('trace_flags',
-                default=None,
                 help='List of options that control which trace info '
                      'is written to the DEBUG log level to assist '
                      'developers. Valid values are method and api.'),
     cfg.MultiOpt('replication_device',
                  item_type=types.Dict(),
-                 default=None,
                  help="Multi opt of dictionaries to represent a replication "
                       "target device.  This option may be specified multiple "
                       "times in a single config section to specify multiple "
index b0530d0bc0fbd2824410cd78091a487100a079d6..69c817259349ce01a05d23aa4e24c86d09615b56 100644 (file)
@@ -36,10 +36,8 @@ LOG = logging.getLogger(__name__)
 
 blockbridge_opts = [
     cfg.StrOpt("blockbridge_api_host",
-               default=None,
                help=_("IP address/hostname of Blockbridge API.")),
     cfg.IntOpt("blockbridge_api_port",
-               default=None,
                help=_("Override HTTPS port to connect to Blockbridge "
                       "API server.")),
     cfg.StrOpt("blockbridge_auth_scheme",
@@ -48,14 +46,11 @@ blockbridge_opts = [
                help=_("Blockbridge API authentication scheme (token "
                       "or password)")),
     cfg.StrOpt("blockbridge_auth_token",
-               default=None,
                help=_("Blockbridge API token (for auth scheme 'token')"),
                secret=True),
     cfg.StrOpt("blockbridge_auth_user",
-               default=None,
                help=_("Blockbridge API user (for auth scheme 'password')")),
     cfg.StrOpt("blockbridge_auth_password",
-               default=None,
                help=_("Blockbridge API password (for auth scheme 'password')"),
                secret=True),
     cfg.DictOpt("blockbridge_pools",
@@ -63,7 +58,6 @@ blockbridge_opts = [
                 help=_("Defines the set of exposed pools and their associated "
                        "backend query strings")),
     cfg.StrOpt("blockbridge_default_pool",
-               default=None,
                help=_("Default pool name if unspecified.")),
 ]
 
index c40cf35a50ba0cdbde63b760022f53abb5a6373f..7cf08818fbbc2b7a88140ecfa129ff28b14b640e 100644 (file)
@@ -17,15 +17,12 @@ from oslo_config import cfg
 
 cloudbyte_connection_opts = [
     cfg.StrOpt("cb_apikey",
-               default=None,
                help="Driver will use this API key to authenticate "
                     "against the CloudByte storage's management interface."),
     cfg.StrOpt("cb_account_name",
-               default=None,
                help="CloudByte storage specific account name. "
                     "This maps to a project name in OpenStack."),
     cfg.StrOpt("cb_tsm_name",
-               default=None,
                help="This corresponds to the name of "
                     "Tenant Storage Machine (TSM) in CloudByte storage. "
                     "A volume will be created in this TSM."),
@@ -50,7 +47,6 @@ cloudbyte_connection_opts = [
                     "deletion in CloudByte storage by making "
                     "this many number of attempts."),
     cfg.StrOpt("cb_auth_group",
-               default="None",
                help="This corresponds to the discovery authentication "
                     "group in CloudByte storage. "
                     "Chap users are added to this group. "
index d710a6a59bc8a7a2766cce522ff544286182dd64..7a567c4e5468786b878f0ca1c1834011a027ee46 100644 (file)
@@ -35,7 +35,6 @@ LOG = logging.getLogger(__name__)
 
 d_opts = [
     cfg.StrOpt('datera_api_token',
-               default=None,
                help='DEPRECATED: This will be removed in the Liberty release. '
                     'Use san_login and san_password instead. This directly '
                     'sets the Datera API token.'),
index 45affb0b2181e4f7d7aab901dc398a5adbccab52..cfa28019fe25b72ba67cdffcdd96a0065122b8d2 100644 (file)
@@ -46,7 +46,6 @@ common_opts = [
                 default=False,
                 help="Whether to verify DotHill array SSL certificate."),
     cfg.StrOpt('dothill_verify_certificate_path',
-               default=None,
                help="DotHill array SSL certificate path."),
 ]
 
index 26173ef8c1478fbd3fa0069eea4630cb6bbb135b..68060c7edac74f16fc8afff129b4b8064eb42c96 100644 (file)
@@ -65,18 +65,15 @@ loc_opts = [
                default='global',
                help='VNX authentication scope type.'),
     cfg.StrOpt('storage_vnx_security_file_dir',
-               default=None,
                help='Directory path that contains the VNX security file. '
                'Make sure the security file is generated first.'),
     cfg.StrOpt('naviseccli_path',
                default='',
                help='Naviseccli Path.'),
     cfg.StrOpt('storage_vnx_pool_names',
-               default=None,
                deprecated_name='storage_vnx_pool_name',
                help='Comma-separated list of storage pool names to be used.'),
     cfg.StrOpt('san_secondary_ip',
-               default=None,
                help='VNX secondary SP IP Address.'),
     cfg.IntOpt('default_timeout',
                default=60 * 24 * 365,
index b6ac505389a9a43fe2205bc5764d1ae52729f4ad..26036dc4deb0ba64407e7351579fae1ffc61e95c 100644 (file)
@@ -49,7 +49,6 @@ scaleio_opts = [
                 default=False,
                 help='Whether to verify server certificate.'),
     cfg.StrOpt('sio_server_certificate_path',
-               default=None,
                help='Server certificate path.'),
     cfg.BoolOpt('sio_round_volume_capacity',
                 default=True,
@@ -61,19 +60,14 @@ scaleio_opts = [
                 default=False,
                 help='Whether to unmap volume before deletion.'),
     cfg.StrOpt('sio_protection_domain_id',
-               default=None,
                help='Protection domain id.'),
     cfg.StrOpt('sio_protection_domain_name',
-               default=None,
                help='Protection domain name.'),
     cfg.StrOpt('sio_storage_pools',
-               default=None,
                help='Storage pools.'),
     cfg.StrOpt('sio_storage_pool_name',
-               default=None,
                help='Storage pool name.'),
     cfg.StrOpt('sio_storage_pool_id',
-               default=None,
                help='Storage pool id.')
 ]
 
index 37ec4bac73be9c8fce709313cfc6176c4edefb52..d93a50c22546afbb67ccddbb37ae79ecb9d661e7 100644 (file)
@@ -58,19 +58,14 @@ LOG = logging.getLogger(__name__)
 
 volume_opts = [
     cfg.StrOpt('hitachi_serial_number',
-               default=None,
                help='Serial number of storage system'),
     cfg.StrOpt('hitachi_unit_name',
-               default=None,
                help='Name of an array unit'),
     cfg.IntOpt('hitachi_pool_id',
-               default=None,
                help='Pool ID of storage system'),
     cfg.IntOpt('hitachi_thin_pool_id',
-               default=None,
                help='Thin pool ID of storage system'),
     cfg.StrOpt('hitachi_ldev_range',
-               default=None,
                help='Range of logical device of storage system'),
     cfg.StrOpt('hitachi_default_copy_method',
                default='FULL',
@@ -85,10 +80,8 @@ volume_opts = [
                default=10,
                help='Interval to check copy asynchronously'),
     cfg.StrOpt('hitachi_target_ports',
-               default=None,
                help='Control port names for HostGroup or iSCSI Target'),
     cfg.StrOpt('hitachi_group_range',
-               default=None,
                help='Range of group number'),
     cfg.BoolOpt('hitachi_group_request',
                 default=False,
index b15e4935cd14fa33243b66d049995dee3ce56081..658582a2c72c7045623c8547d667ea9d1645772d 100644 (file)
@@ -104,10 +104,8 @@ volume_opts = [
                default='200,201',
                help='Instance numbers for HORCM'),
     cfg.StrOpt('hitachi_horcm_user',
-               default=None,
                help='Username of storage system for HORCM'),
     cfg.StrOpt('hitachi_horcm_password',
-               default=None,
                help='Password of storage system for HORCM',
                secret=True),
     cfg.BoolOpt('hitachi_horcm_add_conf',
index efb4ed0e74fbc40c236c7541b4bbd26eaca3aa88..48925283670b01f4b8df3971a59752cd1d6d2cce 100644 (file)
@@ -39,7 +39,6 @@ volume_opts = [
                 default=False,
                 help='Add CHAP user'),
     cfg.StrOpt('hitachi_auth_method',
-               default=None,
                help='iSCSI authentication method'),
     cfg.StrOpt('hitachi_auth_user',
                default='%sCHAP-user' % basic_lib.NAME_PREFIX,
index 3e1a1a2e54dd3627c31b9d4d6772a45d42cd7f9e..0ecbbb5b37895987904846d3318a90e4c6cbf654 100644 (file)
@@ -43,7 +43,6 @@ huawei_opts = [
                default='/etc/cinder/cinder_huawei_conf.xml',
                help='The configuration file for the Cinder Huawei driver.'),
     cfg.StrOpt('hypermetro_devices',
-               default=None,
                help='The remote device hypermetro will use.'),
 ]
 
index 2c0884707f2d0eb5091454cbd0e298c22d5198d5..22cac022a38a7a910518fb427af45ee3d70add92 100644 (file)
@@ -46,15 +46,12 @@ LOG = logging.getLogger(__name__)
 
 gpfs_opts = [
     cfg.StrOpt('gpfs_mount_point_base',
-               default=None,
                help='Specifies the path of the GPFS directory where Block '
                     'Storage volume and snapshot files are stored.'),
     cfg.StrOpt('gpfs_images_dir',
-               default=None,
                help='Specifies the path of the Image service repository in '
                     'GPFS.  Leave undefined if not storing images in GPFS.'),
     cfg.StrOpt('gpfs_images_share_mode',
-               default=None,
                choices=['copy', 'copy_on_write', None],
                help='Specifies the type of image copy to be used.  Set this '
                     'when the Image service repository also uses GPFS so '
index ac03b394f85e09f2fcc41564f6479a48b92f67bd..2ed62c893a5352dff96573cf91fe06ad9622dfd8 100644 (file)
@@ -122,7 +122,6 @@ storwize_svc_opts = [
                 default=False,
                 help='Allow tenants to specify QOS on create'),
     cfg.StrOpt('storwize_svc_stretched_cluster_partner',
-               default=None,
                help='If operating in stretched cluster mode, specify the '
                     'name of the pool in which mirrored copies are stored.'
                     'Example: "pool2"'),
index 3bafc7f42ebbc268949b02c3461cdc9aa7ed8e1b..199ae9e00d1fdf89625bfc1cfade6841cb7c3a8f 100644 (file)
@@ -35,7 +35,6 @@ common_opts = [
                 default=False,
                 help="Whether to verify Lenovo array SSL certificate."),
     cfg.StrOpt('lenovo_verify_certificate_path',
-               default=None,
                help="Lenovo array SSL certificate path.")
 ]
 
index f4df96702e3fc22f1f4a49fa74c5208e30da98eb..0967b1827b514de7d7d7673e38ea1029ae6eafe4 100644 (file)
@@ -43,11 +43,9 @@ netapp_proxy_opts = [
 
 netapp_connection_opts = [
     cfg.StrOpt('netapp_server_hostname',
-               default=None,
                help='The hostname (or IP address) for the storage system or '
                     'proxy server.'),
     cfg.IntOpt('netapp_server_port',
-               default=None,
                help=('The TCP port to use for communication with the storage '
                      'system or proxy server. If not specified, Data ONTAP '
                      'drivers will use 80 for HTTP and 443 for HTTPS; '
@@ -62,11 +60,9 @@ netapp_transport_opts = [
 
 netapp_basicauth_opts = [
     cfg.StrOpt('netapp_login',
-               default=None,
                help=('Administrative user account name used to access the '
                      'storage system or proxy server.')),
     cfg.StrOpt('netapp_password',
-               default=None,
                help=('Password for the administrative user account '
                      'specified in the netapp_login option.'),
                secret=True), ]
@@ -90,14 +86,12 @@ netapp_provisioning_opts = [
 
 netapp_cluster_opts = [
     cfg.StrOpt('netapp_vserver',
-               default=None,
                help=('This option specifies the virtual storage server '
                      '(Vserver) name on the storage cluster on which '
                      'provisioning of block storage volumes should occur.')), ]
 
 netapp_7mode_opts = [
     cfg.StrOpt('netapp_vfiler',
-               default=None,
                help=('The vFiler unit on which provisioning of block storage '
                      'volumes will be done. This option is only used by the '
                      'driver when connecting to an instance with a storage '
@@ -105,7 +99,6 @@ netapp_7mode_opts = [
                      'option when utilizing the MultiStore feature on the '
                      'NetApp storage system.')),
     cfg.StrOpt('netapp_partner_backend_name',
-               default=None,
                help=('The name of the config.conf stanza for a Data ONTAP '
                      '(7-mode) HA partner.  This option is only used by the '
                      'driver when connecting to an instance with a storage '
@@ -145,7 +138,6 @@ netapp_eseries_opts = [
                      'to create the URL used by the driver to connect to the '
                      'proxy application.')),
     cfg.StrOpt('netapp_controller_ips',
-               default=None,
                help=('This option is only utilized when the storage family '
                      'is configured to eseries. This option is used to '
                      'restrict provisioning to the specified controllers. '
@@ -153,7 +145,6 @@ netapp_eseries_opts = [
                      'separated list of controller hostnames or IP addresses '
                      'to be used for provisioning.')),
     cfg.StrOpt('netapp_sa_password',
-               default=None,
                help=('Password for the NetApp E-Series storage array.'),
                secret=True),
     cfg.BoolOpt('netapp_enable_multiattach',
@@ -168,20 +159,17 @@ netapp_eseries_opts = [
 ]
 netapp_nfs_extra_opts = [
     cfg.StrOpt('netapp_copyoffload_tool_path',
-               default=None,
                help=('This option specifies the path of the NetApp copy '
                      'offload tool binary. Ensure that the binary has execute '
                      'permissions set which allow the effective user of the '
                      'cinder-volume process to execute the file.')), ]
 netapp_san_opts = [
     cfg.StrOpt('netapp_lun_ostype',
-               default=None,
                help=('This option defines the type of operating system that'
                      ' will access a LUN exported from Data ONTAP; it is'
                      ' assigned to the LUN at the time it is created.')),
     cfg.StrOpt('netapp_host_type',
                deprecated_name='netapp_eseries_host_type',
-               default=None,
                help=('This option defines the type of operating system for'
                      ' all initiators that can access a LUN. This information'
                      ' is used when mapping LUNs to individual hosts or'
index bda7f6a62deb0bbc14be069a5fa8f2ef6d2bab91..951e062deb78394c1e194d0f1c316ac7429fb9dd 100644 (file)
@@ -69,7 +69,6 @@ nfs_opts = [
                default='$state_path/mnt',
                help=('Base dir containing mount points for nfs shares.')),
     cfg.StrOpt('nfs_mount_options',
-               default=None,
                help=('Mount options passed to the nfs client. See section '
                      'of the nfs man page for details.')),
     cfg.IntOpt('nfs_mount_attempts',
index eef8d598fcd860c258699ea53960ca90509daf2b..4cb1663b120b3712baee4f1edee4dc90782a74a3 100644 (file)
@@ -46,7 +46,6 @@ LOG = logging.getLogger(__name__)
 
 PURE_OPTS = [
     cfg.StrOpt("pure_api_token",
-               default=None,
                help="REST API authorization token."),
 ]
 
index 9f6559cf97ce32a1ed86e4733eab7355ef77d63a..7f398e0bfd8b47ba9098acf48b2422a6af9e3c13 100644 (file)
@@ -35,11 +35,9 @@ LOG = logging.getLogger(__name__)
 
 volume_opts = [
     cfg.StrOpt('quobyte_volume_url',
-               default=None,
                help=('URL to the Quobyte volume e.g.,'
                      ' quobyte://<DIR host>/<volume name>')),
     cfg.StrOpt('quobyte_client_cfg',
-               default=None,
                help=('Path to a Quobyte Client configuration file.')),
     cfg.BoolOpt('quobyte_sparsed_volumes',
                 default=True,
index afbc1c07bf71fcef006a1ecdbb6db455948dc6ec..6ad26647aca55ed62779553f30307757da03263e 100644 (file)
@@ -51,7 +51,6 @@ rbd_opts = [
                default='rbd',
                help='The RADOS pool where rbd volumes are stored'),
     cfg.StrOpt('rbd_user',
-               default=None,
                help='The RADOS client name for accessing rbd volumes '
                     '- only set when using cephx authentication'),
     cfg.StrOpt('rbd_ceph_conf',
@@ -62,11 +61,9 @@ rbd_opts = [
                 help='Flatten volumes created from snapshots to remove '
                      'dependency from volume to snapshot'),
     cfg.StrOpt('rbd_secret_uuid',
-               default=None,
                help='The libvirt uuid of the secret for the rbd_user '
                     'volumes'),
     cfg.StrOpt('volume_tmp_dir',
-               default=None,
                help='Directory where temporary image files are stored '
                     'when the volume driver does not write them directly '
                     'to the volume.  Warning: this option is now deprecated, '
index 1fa8f0ccb3d5b12f5c0847dde34c3aad408e0674..4bb5d65af67a709a64c4e2ca634525ed02fdb43a 100644 (file)
@@ -81,7 +81,6 @@ nas_opts = [
                      'For example:  "/srv/export1" for an NFS server export '
                      'available at 10.0.5.10:/srv/export1 .')),
     cfg.StrOpt('nas_mount_options',
-               default=None,
                help=('Options used to mount the storage backend file system '
                      'where Cinder volumes are stored.')),
 ]
index 0418766f8c13dcd1e103169584674c8c7a53a471..9063eae37110992931f42225a157ce7c5cad6f41 100644 (file)
@@ -43,18 +43,14 @@ if hplefthandclient:
 
 hplefthand_opts = [
     cfg.StrOpt('hplefthand_api_url',
-               default=None,
                help="HP LeftHand WSAPI Server Url like "
                     "https://<LeftHand ip>:8081/lhos"),
     cfg.StrOpt('hplefthand_username',
-               default=None,
                help="HP LeftHand Super user username"),
     cfg.StrOpt('hplefthand_password',
-               default=None,
                help="HP LeftHand Super user password",
                secret=True),
     cfg.StrOpt('hplefthand_clustername',
-               default=None,
                help="HP LeftHand cluster name"),
     cfg.BoolOpt('hplefthand_iscsi_chap_enabled',
                 default=False,
index 277e878eb548e05e29a76a56dd57808d6c517554..9190effd82f154746d1c0e6878d5f3f4fd5fc799 100644 (file)
@@ -25,23 +25,18 @@ FC_VOLUME_OPTS = [
 COMMON_VOLUME_OPTS = [
     cfg.StrOpt(
         'hpxp_storage_cli',
-        default=None,
         help='Type of storage command line interface'),
     cfg.StrOpt(
         'hpxp_storage_id',
-        default=None,
         help='ID of storage system'),
     cfg.StrOpt(
         'hpxp_pool',
-        default=None,
         help='Pool of storage system'),
     cfg.StrOpt(
         'hpxp_thin_pool',
-        default=None,
         help='Thin pool of storage system'),
     cfg.StrOpt(
         'hpxp_ldev_range',
-        default=None,
         help='Logical device range of storage system'),
     cfg.StrOpt(
         'hpxp_default_copy_method',
@@ -63,11 +58,9 @@ COMMON_VOLUME_OPTS = [
         help='Interval to check copy asynchronously'),
     cfg.ListOpt(
         'hpxp_target_ports',
-        default=None,
         help='Target port names for host group or iSCSI target'),
     cfg.ListOpt(
         'hpxp_compute_target_ports',
-        default=None,
         help=(
             'Target port names of compute node '
             'for host group or iSCSI target')),
@@ -84,7 +77,6 @@ HORCM_VOLUME_OPTS = [
         help='Instance numbers for HORCM'),
     cfg.StrOpt(
         'hpxp_horcm_user',
-        default=None,
         help='Username of storage system for HORCM'),
     cfg.BoolOpt(
         'hpxp_horcm_add_conf',
index 781163092d35f0d49eb573b51c2688d9b537a5f5..b43f4b146adcc3fced0077f1e417757c508243db 100644 (file)
@@ -35,7 +35,6 @@ common_opts = [
                 default=False,
                 help="Whether to verify HPMSA array SSL certificate."),
     cfg.StrOpt('hpmsa_verify_certificate_path',
-               default=None,
                help="HPMSA array SSL certificate path."),
 
 ]
index c339a7685eb06efe7688fc40bf0855c1f9cc4fc9..0464555702827e2966866bad19b433fe66a685c2 100644 (file)
@@ -40,7 +40,6 @@ LOG = logging.getLogger(__name__)
 
 volume_opts = [
     cfg.StrOpt('scality_sofs_config',
-               default=None,
                help='Path or URL to Scality SOFS configuration file'),
     cfg.StrOpt('scality_sofs_mount_point',
                default='$state_path/scality',
index 16663267236421eb2ac8a22b2f7b97b12cd37618..d18ec5c27dc97b6786d8ea581e25d851e3ac0f40 100644 (file)
@@ -50,7 +50,6 @@ sf_opts = [
                 help='Allow tenants to specify QOS on create'),
 
     cfg.StrOpt('sf_account_prefix',
-               default=None,
                help='Create SolidFire accounts with this prefix. Any string '
                     'can be used here, but the string \"hostname\" is special '
                     'and will create a prefix using the cinder node hostname '
@@ -74,7 +73,6 @@ sf_opts = [
                      'glance and qemu-conversion on subsequent calls.'),
 
     cfg.StrOpt('sf_svip',
-               default=None,
                help='Overrides default cluster SVIP with the one specified. '
                     'This is required or deployments that have implemented '
                     'the use of VLANs for iSCSI networks in their cloud.'),
index e6a534b2779ee20967518ed7484d5dfbeca6a40e..1a22e4cb2fafa5b3e433edee17f233c4b70aa12c 100644 (file)
@@ -47,7 +47,6 @@ LOG = logging.getLogger(__name__)
 
 srb_opts = [
     cfg.StrOpt('srb_base_urls',
-               default=None,
                help='Comma-separated list of REST servers IP to connect to. '
                     '(eg http://IP1/,http://IP2:81/path'),
 ]
index 2254c2c87e507022446334131288ef1fb2eedc17..df1944b885dc25f696d1036b93f87ecce85d01a4 100644 (file)
@@ -43,13 +43,10 @@ tintri_path = '/tintri/'
 
 tintri_opts = [
     cfg.StrOpt('tintri_server_hostname',
-               default=None,
                help='The hostname (or IP address) for the storage system'),
     cfg.StrOpt('tintri_server_username',
-               default=None,
                help='User name for the storage system'),
     cfg.StrOpt('tintri_server_password',
-               default=None,
                help='Password for the storage system',
                secret=True),
     cfg.StrOpt('tintri_api_version',
index 9d1cc09c05bab96f5fb39adeae5d78184c618c66..3dfe14d652757b8161814c3d769f6f332ecc3448 100644 (file)
@@ -52,10 +52,8 @@ VMOS_SUPPORTED_VERSION_PATTERNS = ['V6.3.0.[4-9]', 'V6.3.[1-9].?[0-9]?']
 
 violin_opts = [
     cfg.StrOpt('gateway_mga',
-               default=None,
                help='IP address or hostname of mg-a'),
     cfg.StrOpt('gateway_mgb',
-               default=None,
                help='IP address or hostname of mg-b'),
     cfg.BoolOpt('use_igroups',
                 default=False,
index e331519babd8805475ed9623d86c6ded0cfd062f..c2c9c5b5ef49e736bfde5a49f28e0c76ab106f29 100644 (file)
@@ -64,19 +64,15 @@ EXTRA_CONFIG_VOLUME_ID_KEY = "cinder.volume.id"
 
 vmdk_opts = [
     cfg.StrOpt('vmware_host_ip',
-               default=None,
                help='IP address for connecting to VMware vCenter server.'),
     cfg.StrOpt('vmware_host_username',
-               default=None,
                help='Username for authenticating with VMware vCenter '
                     'server.'),
     cfg.StrOpt('vmware_host_password',
-               default=None,
                help='Password for authenticating with VMware vCenter '
                     'server.',
                secret=True),
     cfg.StrOpt('vmware_wsdl_location',
-               default=None,
                help='Optional VIM service WSDL Location '
                     'e.g http://<server>/vimService.wsdl. Optional over-ride '
                     'to default location for bug work-arounds.'),
@@ -115,7 +111,6 @@ vmdk_opts = [
                help='Directory where virtual disks are stored during volume '
                     'backup and restore.'),
     cfg.StrOpt('vmware_ca_file',
-               default=None,
                help='CA bundle file to use in verifying the vCenter server '
                     'certificate.'),
     cfg.BoolOpt('vmware_insecure',
@@ -125,7 +120,6 @@ vmdk_opts = [
                      'used for verification. This option is ignored if '
                      '"vmware_ca_file" is set.'),
     cfg.MultiStrOpt('vmware_cluster_name',
-                    default=None,
                     help='Name of a vCenter compute cluster where volumes '
                          'should be created.'),
 ]
index 8f14d8b862410c2c15ab08b5e68104b5ddb54fc5..f1659004f083346f4333dda25cb6175a4f31dc71 100644 (file)
@@ -53,7 +53,6 @@ vzstorage_opts = [
                help=('Base dir containing mount points for '
                      'vzstorage shares.')),
     cfg.ListOpt('vzstorage_mount_options',
-                default=None,
                 help=('Mount options passed to the vzstorage client. '
                       'See section of the pstorage-mount man page '
                       'for details.')),
index 783eb58ffe6ea23df772010c31eb405fa37d7747..74e3b9af29793efb79c44a6c37fa12e394206834 100644 (file)
@@ -94,7 +94,6 @@ volume_manager_opts = [
                 help='Offload pending volume delete during '
                      'volume service startup'),
     cfg.StrOpt('zoning_mode',
-               default='none',
                help='FC Zoning mode configured'),
     cfg.StrOpt('extra_capabilities',
                default='{}',
index f40595680781cf60ac7a864578460657ff07b2e7..cabe899433fb7dec1befdf1268d5e6c8b754ed3a 100644 (file)
@@ -50,15 +50,12 @@ socket_opts = [
                help="Sets the value of TCP_KEEPCNT for each "
                     "server socket. Not supported on OS X."),
     cfg.StrOpt('ssl_ca_file',
-               default=None,
                help="CA certificate file to use to verify "
                     "connecting clients"),
     cfg.StrOpt('ssl_cert_file',
-               default=None,
                help="Certificate file to use when starting "
                     "the server securely"),
     cfg.StrOpt('ssl_key_file',
-               default=None,
                help="Private key file to use when starting "
                     "the server securely"),
 ]
index 26c8fbec04d8a96f2add5beb6d0c833e0aa8de67..ec311f87b722eee7ca93763cfc42c593954087b0 100644 (file)
@@ -42,10 +42,8 @@ brcd_zone_opts = [
                 default=True,
                 help='overridden zoning activation state'),
     cfg.StrOpt('zone_name_prefix',
-               default=None,
                help='overridden zone name prefix'),
     cfg.StrOpt('principal_switch_wwn',
-               default=None,
                help='Principal switch WWN of the fabric'),
 ]
 
index 506888d85fb7f297e4fe734cd40df539c0b69228..8b8fe227b1656c50ac86aacf7b13e7f5deb125e7 100644 (file)
@@ -80,7 +80,7 @@ class BrcdFCZoneDriver(fc_zone_driver.FCZoneDriver):
             base_san_opts = []
             if not fabric_names:
                 base_san_opts.append(
-                    cfg.StrOpt('fc_fabric_names', default=None,
+                    cfg.StrOpt('fc_fabric_names',
                                help='Comma separated list of fibre channel '
                                'fabric names. This list of names is used to'
                                ' retrieve other SAN credentials for connecting'
index 0aa0ceafaaeedad5c71111633a44258068559405..b29b408d5ddd8764dcdca88c812d34f253784d54 100644 (file)
@@ -39,10 +39,8 @@ cisco_zone_opts = [
                 default=True,
                 help='overridden zoning activation state'),
     cfg.StrOpt('cisco_zone_name_prefix',
-               default=None,
                help='overridden zone name prefix'),
     cfg.StrOpt('cisco_zoning_vsan',
-               default=None,
                help='VSAN of the Fabric'),
 ]
 
index 85d69b6d9912a70f0f57f50d2a96e9d8fb14a861..b94733ede4b6dd28cdb2a730ea9707761f39a9ce 100644 (file)
@@ -79,7 +79,7 @@ class CiscoFCZoneDriver(fc_zone_driver.FCZoneDriver):
             base_san_opts = []
             if not fabric_names:
                 base_san_opts.append(
-                    cfg.StrOpt('fc_fabric_names', default=None,
+                    cfg.StrOpt('fc_fabric_names',
                                help='Comma separated list of fibre channel '
                                'fabric names. This list of names is used to'
                                ' retrieve other SAN credentials for connecting'
index 011890be43eec9e76333a836ca30ff6e5bb54153..c19565a853858e17820b321ed7dc187f7295f89d 100644 (file)
@@ -53,7 +53,6 @@ zone_manager_opts = [
                help='Zoning policy configured by user; valid values include '
                '"initiator-target" or "initiator"'),
     cfg.StrOpt('fc_fabric_names',
-               default=None,
                help='Comma separated list of Fibre Channel fabric names.'
                ' This list of names is used to retrieve other SAN credentials'
                ' for connecting to each SAN fabric'),