os_trove doesn't set conductor workers

Bug #1664639 reported by Kyle L. Henderson
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Undecided
Ravi Gummadi

Bug Description

In the existing service roles supported by OSA, the number of worker threads can be controlled by the user or by a default calculation in the role itself. It looks like controlling the number of trove_conductor workers was missed.

The value for trove_conductor_workers should be set in the trove-conductor.conf.j2 file similar to how it's done for trove_api_workers in trove.conf.j2

Ravi Gummadi (ravigumm)
Changed in openstack-ansible:
assignee: nobody → Ravi Gummadi (ravigumm)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-os_trove (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_trove (master)

Reviewed: https://review.openstack.org/435278
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_trove/commit/?id=fdd59ffe60672e7f4401f2b179d5fb152e0d33a3
Submitter: Jenkins
Branch: master

commit fdd59ffe60672e7f4401f2b179d5fb152e0d33a3
Author: Ravi Gummadi <email address hidden>
Date: Fri Feb 17 02:17:13 2017 -0500

    Configure trove_conductor_workers in os_trove

    Configure trove_conductor_workers by calculating the default value.
    Similar to other services worker threads, the default value for
    trove_conductor_workers is half the number of vcpus on the machine.

    Also added capping of worker threads for trove.

    Change-Id: If853b0e1e45b7137572df8a1123273539343c83e
    Closes-Bug: #1664639

Changed in openstack-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-os_trove (stable/ocata)

Fix proposed to branch: stable/ocata
Review: https://review.openstack.org/435685

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_trove (stable/ocata)

Reviewed: https://review.openstack.org/435685
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_trove/commit/?id=26d2d17472b65f66e36f0c48af303764b35f400b
Submitter: Jenkins
Branch: stable/ocata

commit 26d2d17472b65f66e36f0c48af303764b35f400b
Author: Ravi Gummadi <email address hidden>
Date: Fri Feb 17 02:17:13 2017 -0500

    Configure trove_conductor_workers in os_trove

    Configure trove_conductor_workers by calculating the default value.
    Similar to other services worker threads, the default value for
    trove_conductor_workers is half the number of vcpus on the machine.

    Also added capping of worker threads for trove.

    Change-Id: If853b0e1e45b7137572df8a1123273539343c83e
    Closes-Bug: #1664639
    (cherry picked from commit fdd59ffe60672e7f4401f2b179d5fb152e0d33a3)

tags: added: in-stable-ocata
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-os_trove 15.0.0.0rc2

This issue was fixed in the openstack/openstack-ansible-os_trove 15.0.0.0rc2 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-os_trove 16.0.0.0b1

This issue was fixed in the openstack/openstack-ansible-os_trove 16.0.0.0b1 development milestone.

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.