Use OSprofiler options consolidated in lib itself

Bug #1552511 reported by OpenStack Infra
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-manuals
Fix Released
Medium
KATO Tomoyuki

Bug Description

https://review.openstack.org/278155
Dear bug triager. This bug was created since a commit was marked with DOCIMPACT.
Your project "openstack/cinder" is set up so that we directly report the documentation bugs against it. If this needs changing, the docimpact-group option needs to be added for the project. You can ask the OpenStack infra team (#openstack-infra on freenode) for help if you need to.

commit 16660696f91236369ffa7a943f5ff7279a445199
Author: Dina Belova <email address hidden>
Date: Tue Feb 9 15:58:14 2016 -0800

    Use OSprofiler options consolidated in lib itself

    Starting with 1.0.0 osprofiler release options needed for
    its workability are consolidated inside osprofiler itself.
    Let's use them.

    UpgradeImpact
    DocImpact
    Change-Id: Ie20e827f26c9be6867c233afdea9e05e2869d6e1

Revision history for this message
Sean McGinnis (sean-mcginnis) wrote :

Dina, please add notes as to the documentation impact for this. It can then be assigned to openstack-manuals.

Changed in cinder:
assignee: nobody → Dina Belova (dbelova)
Revision history for this message
venkatamahesh (venkatamaheshkotha) wrote :

This changes will affects the cinder tables in configuration reference.
This affects the table https://github.com/openstack/openstack-manuals/blob/master/doc/config-reference/source/tables/cinder-profiler.rst and is autogenerated one

Thanks
mahesh

Revision history for this message
venkatamahesh (venkatamaheshkotha) wrote :

I think now you can move to openstack-manuals

Changed in cinder:
status: New → Confirmed
importance: Undecided → Medium
importance: Medium → Undecided
status: Confirmed → New
Revision history for this message
venkatamahesh (venkatamaheshkotha) wrote :

I will change the status and importance when this is moved to openstack-manuals

Thanks
mahesh

Revision history for this message
Dina Belova (dbelova) wrote :

Mahesh,

yes, you're right. The only difference is that options are right now got from the library itself, and one of them has changed its name:

Previously it was like this:

[profiler]
profiler_enabled = True

Right now it'll be:

[profiler]
enabled = True

The profiler_enabled is set as a deprecated name is osprofiler for this option.

Cheers,
Dina

Revision history for this message
Dina Belova (dbelova) wrote :

Sadly I do not see openstack-manuals in the assignees search field. Mahesh, can you reassign the bug?

affects: cinder → openstack-manuals
Changed in openstack-manuals:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → mitaka
tags: added: autogenerate-config-docs
removed: doc
Changed in openstack-manuals:
milestone: mitaka → newton
Revision history for this message
KATO Tomoyuki (kato-tomoyuki) wrote :

This has already been updated with bulk update for Newton release.

Changed in openstack-manuals:
assignee: Dina Belova (dbelova) → KATO Tomoyuki (kato-tomoyuki)
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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