Add rpc tunables to cinder.conf

Bug #1617040 reported by Bjoern
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Wishlist
Bjoern

Bug Description

Unlike other services we missed to set the RPC settings by default inside the cinder.conf
I would welcome to set this as default setting and honoring overrides like nova so no one has override it just for cinder.

Bjoern (bjoern-t)
Changed in openstack-ansible:
assignee: nobody → Bjoern Teipel (bjoern-teipel)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-os_cinder (master)

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

Changed in openstack-ansible:
status: New → In Progress
Changed in openstack-ansible:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on openstack-ansible-os_cinder (master)

Change abandoned by Jesse Pretorius (odyssey4me) (<email address hidden>) on branch: master
Review: https://review.openstack.org/360749
Reason: This patch did not pass gate tests, nor has it been revised. I also agree with git-harry's feedback in PS6. If you still wish for this patch to be included then please restore it and re-propose.

Revision history for this message
Bjoern (bjoern-t) wrote :

Patch updated.
Personally, I'm not happy that you can override the RPC settings everywhere else with a override but not cinder. If you set RPC overrides you would except that the will be applied with cinder too and you don't have to override cinder in a different way than nova for example.

This patch addresses this issue, since you still want to make it easy for openstack ops folks to configure the services. There is no benefit of forcing the operator to override system wide settings like rpc_response_timeout which exist in every since openstack service

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

Reviewed: https://review.openstack.org/360749
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_cinder/commit/?id=5fab05666e81411661c3b6f1f82524c2a3e0d1dd
Submitter: Jenkins
Branch: master

commit 5fab05666e81411661c3b6f1f82524c2a3e0d1dd
Author: Bjoern Teipel <email address hidden>
Date: Wed Oct 12 18:05:16 2016 -0500

    Add remote procedure call tunables to cinder

    This allows us to override the default settings, which is useful for
    large deployments or deploying a large number of cinder volumes at
    once where backend response times are slowed down.

    Change-Id: Ibd69473c8d66decea17daad5ce9a5f9e499d7779
    Closes-Bug: #1617040

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_cinder (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/391182

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

Reviewed: https://review.openstack.org/391182
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_cinder/commit/?id=07cd802be70105e4160aa1dac4cb8c33b31c3116
Submitter: Jenkins
Branch: stable/newton

commit 07cd802be70105e4160aa1dac4cb8c33b31c3116
Author: Bjoern Teipel <email address hidden>
Date: Wed Oct 12 18:05:16 2016 -0500

    Add remote procedure call tunables to cinder

    This allows us to override the default settings, which is useful for
    large deployments or deploying a large number of cinder volumes at
    once where backend response times are slowed down.

    Change-Id: Ibd69473c8d66decea17daad5ce9a5f9e499d7779
    Closes-Bug: #1617040
    (cherry picked from commit 5fab05666e81411661c3b6f1f82524c2a3e0d1dd)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-os_cinder 15.0.0.0b1

This issue was fixed in the openstack/openstack-ansible-os_cinder 15.0.0.0b1 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-os_cinder 14.0.2

This issue was fixed in the openstack/openstack-ansible-os_cinder 14.0.2 release.

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.