Merge lp:~james-page/charms/precise/cinder/fix-ceph-host-stickyness into lp:~openstack-charmers/charms/precise/cinder/ha-support

Proposed by James Page
Status: Merged
Merged at revision: 40
Proposed branch: lp:~james-page/charms/precise/cinder/fix-ceph-host-stickyness
Merge into: lp:~openstack-charmers/charms/precise/cinder/ha-support
Diff against target: 23 lines (+5/-1)
1 file modified
hooks/cinder-hooks (+5/-1)
To merge this branch: bzr merge lp:~james-page/charms/precise/cinder/fix-ceph-host-stickyness
Reviewer Review Type Date Requested Status
OpenStack Charmers Pending
Review via email: mp+153245@code.launchpad.net

Description of the change

Couple of minor fixes

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/cinder-hooks'
2--- hooks/cinder-hooks 2013-03-08 21:29:37 +0000
3+++ hooks/cinder-hooks 2013-03-13 21:23:18 +0000
4@@ -189,6 +189,10 @@
5 # Reconfigure cinder-volume
6 set_or_update volume_driver cinder.volume.driver.RBDDriver
7 set_or_update rbd_pool cinder
8+ # Set host to service name to ensure that requests get
9+ # distributed across all cinder servers in a cluster
10+ # as they can all service ceph requests.
11+ set_or_update host "$SERVICE_NAME"
12 cinder_ctl "cinder-volume" restart
13 }
14
15@@ -199,7 +203,7 @@
16 local cfg_api_port="$(config-get api-listening-port)"
17 local haproxy_port="$(determine_haproxy_port $cfg_api_port)"
18 local backend_port="$(determine_api_port $cfg_api_port)"
19- service cinder-api stop
20+ service cinder-api stop || :
21 configure_haproxy "cinder_api:$haproxy_port:$backend_port"
22 set_or_update osapi_volume_listen_port "$backend_port"
23 service cinder-api start

Subscribers

People subscribed via source and target branches