← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1584558] Re: Some config options in codes use double quotes but most of the options use single quotes and that need to be normalized

 

** Changed in: cinder
       Status: In Progress => Won't Fix

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1584558

Title:
  Some config options in codes use double quotes but most of the options
  use single quotes and that need to be normalized

Status in Ceilometer:
  Won't Fix
Status in Cinder:
  Won't Fix
Status in Glance:
  Fix Released
Status in OpenStack Compute (nova):
  In Progress

Bug description:
  description:

  most options like this use '':
  volume_manager_opts = [
      cfg.StrOpt('volume_driver',
                 default='cinder.volume.drivers.lvm.LVMISCSIDriver',
                 help='Driver to use for volume creation'),
      cfg.IntOpt('migration_create_volume_timeout_secs',
                 default=300,
                 help='Timeout for creating the volume to migrate to '
                      'when performing volume migration (seconds)'),
      cfg.BoolOpt('volume_service_inithost_offload',
                  default=False,
                  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='{}',
                 help='User defined capabilities, a JSON formatted string '
                      'specifying key/value pairs. The key/value pairs can '
                      'be used by the CapabilitiesFilter to select between '
                      'backends when requests specify volume types. For '
                      'example, specifying a service level or the geographical '
                      'location of a backend, then creating a volume type to '
                      'allow the user to select by these different '
                      'properties.'),
  ]

  
  but some option like this use ""
  store_type_opts = [
      cfg.ListOpt("store_type_preference",
                  default=[],
                  help=_("The store names to use to get store preference order. "
                         "The name must be registered by one of the stores "
                         "defined by the 'stores' config option. "
                         "This option will be applied when you using "
                         "'store_type' option as image location strategy "
                         "defined by the 'location_strategy' config option."))
  ]
  profiler_opts = [
      cfg.BoolOpt("enabled", default=False,
                  help=_('If False fully disable profiling feature.')),
      cfg.BoolOpt("trace_sqlalchemy", default=False,
                  help=_("If False doesn't trace SQL requests."))
  ]

  I think it's need to normalize

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


References