Use (# of CPUs) osapi_volume_workers by default

[OpenStack Infra]

https://review.openstack.org/103206
commit 6f24ca2496e37f3f75496c0e4191e71e55fd7b5d
Author: Matt Riedemann <email address hidden>
Date: Fri Jun 27 10:57:33 2014 -0700

    Use (# of CPUs) osapi_volume_workers by default

    This changes the default number of cinder API workers to be equal to the
    number of CPUs available on the host, rather than defaulting to 1 as it
    did before.

    Commit 75c96a48fc7e5dfb59d8258142b01422f81b0253 did the same thing in
    Nova in Icehouse. Similar changes are being made to Glance and Trove as
    well.

    DocImpact: osapi_volume_workers will now be equal to the number of
               CPUs available by default if not explicitly specified
               in cinder.conf.

    UpgradeImpact: Anyone upgrading to this change that does not have
               osapi_volume_workers specified in cinder.conf will now be
               running multiple API workers by default when they restart
               the cinder-api service.

    Closes-Bug: #1333370

    Change-Id: I8dec104800c18618e0c8422bbb93d5dc19c12876

[Gauvain Pocentek (gpocentek)]

Hey Oslo devs,

Do you think that this setting can be easily sanitized in the configuration sample generator? Is it even worth it?

We'll have to deal with this in the autogenerated config-reference tables too.

[Doug Hellmann (doug-hellmann)]

@Gauvain: The new sample generator in oslo.config uses a static sample instead of the sanitizer callbacks (see http://git.openstack.org/cgit/openstack/oslo.config/tree/oslo/config/cfg.py#n597).

[Doug Hellmann (doug-hellmann)]

I'm marking this invalid not because I don't think it's a real problem, but because it's solved another way now and "won't fix" didn't feel quite right.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/115256
Committed: https://git.openstack.org/cgit/openstack/openstack-manuals/commit/?id=8b2016b5d035b3475fe197b2132a8e1e8abff350
Submitter: Jenkins
Branch: master

commit 8b2016b5d035b3475fe197b2132a8e1e8abff350
Author: Gauvain Pocentek <email address hidden>
Date: Tue Aug 19 11:22:22 2014 +0200

    Update the config reference tables

    Include the new tables in the documentation.

    Fix the cinder-prophetstor_dpl.xml filename and the xiv category.

    Manually fix those files for niceness test:
    - glance-logging.xml (lines 34, 38), fixed in oslo-incubator
    - trove-logging.xml (lines 26, 30), fixed in oslo-incubator
    - neutron-openvswitch_agent.xml (line 45), to be fixed in neutron

    Closes-Bug: #1340858
    Closes-Bug: #1344231
    Closes-Bug: #1345956
    Closes-Bug: #1346711
    Closes-Bug: #1347978
    Partial-Bug: #1348329
    Closes-Bug: #1352074
    Partial-Bug: #1353417
    Closes-Bug: #1354622
    Closes-Bug: #1339754
    Closes-Bug: #1358598
    Closes-Bug: #1358259
    Closes-Bug: #1357865
    Partial-Bug: #1357457
    Closes-Bug: #1357421
    Change-Id: Id2da7d7762ca954bd552dbf89a9ff28b144efb68

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/openstack-manuals 15.0.0 release.