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

Bug #1584558 reported by jingtao liang
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Low
jingtao liang

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

Tags: config options
Changed in glance:
assignee: nobody → jingtao liang (liang-jingtao)
status: New → In Progress
Yafei Yu (yu-yafei)
summary: - some config options in codes use "" ,but most of the options use ''.I
- think it's need to normalize
+ Some config options in codes use double quotes but most of the options
+ use single quotes and that need to be normalizedsome config options in
+ codes use "" ,but most of the options use ''.I think it's need to
+ normalize
summary: Some config options in codes use double quotes but most of the options
- use single quotes and that need to be normalizedsome config options in
- codes use "" ,but most of the options use ''.I think it's need to
- normalize
+ use single quotes and that need to be normalized
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

Fix proposed to branch: master
Review: https://review.openstack.org/319720

Changed in glance:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

Reviewed: https://review.openstack.org/319720
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=208c96ab1438e643eb274aaf0c07883c556cd635
Submitter: Jenkins
Branch: master

commit 208c96ab1438e643eb274aaf0c07883c556cd635
Author: liangjingtao <email address hidden>
Date: Mon May 23 11:29:41 2016 +0800

    Normalize the options use singele quotes

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

    DocImpact
    Closes-Bug: #1584558

    Change-Id: I828326d9bb22b6cb98885255ebc0dfcc1efc0497

Changed in glance:
status: In Progress → Fix Released
Yafei Yu (yu-yafei)
Changed in cinder:
assignee: nobody → Ji.Wei (jiwei)
importance: Undecided → Low
Changed in nova:
importance: Undecided → Low
assignee: nobody → jingtao liang (liang-jingtao)
Ji.Wei (jiwei)
Changed in heat:
assignee: nobody → Ji.Wei (jiwei)
Changed in heat:
assignee: Ji.Wei (jiwei) → jingtao liang (liang-jingtao)
Changed in ceilometer:
assignee: nobody → jingtao liang (liang-jingtao)
Yafei Yu (yu-yafei)
Changed in nova:
status: New → In Progress
Changed in heat:
status: New → In Progress
Changed in ceilometer:
status: New → In Progress
Changed in cinder:
status: New → In Progress
tags: added: config options
Revision history for this message
Zhenyu Zheng (zhengzhenyu) wrote :

You shuld not mark them to In Progress until you have a patch up, if a patch is related to this, it will automatically updated, and also, you should not mark it to low by yourself, this will not considered as a bug in most of the projects.

Ji.Wei (jiwei)
Changed in cinder:
importance: Low → Undecided
ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
status: In Progress → Won't Fix
Yafei Yu (yu-yafei)
Changed in nova:
importance: Low → Undecided
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/320215

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

Fix proposed to branch: master
Review: https://review.openstack.org/320219

Thomas Herve (therve)
no longer affects: heat
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/320292

Revision history for this message
Liusheng (liusheng) wrote :

I cannot understand what's the worth of this change...

Revision history for this message
Ji.Wei (jiwei) wrote :

Just in order to standardize the code

Jay Bryant (jsbryant)
Changed in cinder:
status: In Progress → Won't Fix
Revision history for this message
Markus Zoeller (markus_z) (mzoeller) wrote :

There's no real benefit in doing this.

Changed in nova:
status: In Progress → Won't Fix
Yafei Yu (yu-yafei)
no longer affects: ceilometer
no longer affects: cinder
no longer affects: nova
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/glance 13.0.0.0b1

This issue was fixed in the openstack/glance 13.0.0.0b1 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.