[openstack-dev] [glance] Deprecating osprofiler option 'enabled' in favour of 'profiler_enabled'

Flavio Percoco flavio at redhat.com
Tue Dec 2 09:44:50 UTC 2014


On 02/12/14 12:16 +0800, Zhi Yan Liu wrote:
>Why not change other services instead of glance? I see one reason is
>"glance is the only one service use this option name", but to me one
>reason to keep it as-it in glance is that original name makes more
>sense due to the option already under "profiler" group, adding
>"profiler" prefix to it is really redundant, imo, and in other
>existing config group there's no one go this naming way. Then in the
>code we can just use a clear way:
>
>    CONF.profiler.enabled
>
>instead of:
>
>    CONF.profiler.profiler_enabled

I'm with Zhi Yan on this one. Adding profiler sounds redundant.

Cheers,
Flavio

>
>thanks,
>zhiyan
>
>On Mon, Dec 1, 2014 at 11:43 PM, Ian Cordasco
><ian.cordasco at rackspace.com> wrote:
>> On 12/1/14, 08:37, "Louis Taylor" <kragniz at gmail.com> wrote:
>>
>>>Hi all,
>>>
>>>In order to enable or disable osprofiler in Glance, we currently have an
>>>option:
>>>
>>>    [profiler]
>>>    # If False fully disable profiling feature.
>>>    enabled = False
>>>
>>>However, all other services with osprofiler integration use a similar
>>>option
>>>named profiler_enabled.
>>>
>>>For consistency, I'm proposing we deprecate this option's name in favour
>>>of
>>>profiler_enabled. This should make it easier for someone to configure
>>>osprofiler across projects with less confusion. Does anyone have any
>>>thoughts
>>>or concerns about this?
>>>
>>>Thanks,
>>>Louis
>>
>> We *just* introduced this if I remember the IRC discussion from last
>> month. I’m not sure how many people will be immediately making use of it.
>> I’m in favor of consistency where possible and while this would require a
>> deprecation, I think it’s a worthwhile change.
>>
>> +1 from me
>>
>>>>
>> Ian
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>_______________________________________________
>OpenStack-dev mailing list
>OpenStack-dev at lists.openstack.org
>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

-- 
@flaper87
Flavio Percoco
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20141202/b416cbe7/attachment.pgp>


More information about the OpenStack-dev mailing list