Merge lp:~james-page/charms/precise/cinder/hook-fixup into lp:charms/raring/cinder

Proposed by James Page
Status: Merged
Merge reported by: Adam Gandelman
Merged at revision: not available
Proposed branch: lp:~james-page/charms/precise/cinder/hook-fixup
Merge into: lp:charms/raring/cinder
Diff against target: 55 lines (+11/-4)
3 files modified
hooks/cinder-common (+5/-1)
hooks/cinder-hooks (+5/-2)
revision (+1/-1)
To merge this branch: bzr merge lp:~james-page/charms/precise/cinder/hook-fixup
Reviewer Review Type Date Requested Status
Adam Gandelman (community) Approve
Review via email: mp+138184@code.launchpad.net

Description of the change

Charm currently does not deploy due to syntax error in hooks case statement.

To post a comment you must log in.
19. By James Page

Add future fix for move to start-stop-daemon in grizzly

Revision history for this message
Adam Gandelman (gandelman-a) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hooks/cinder-common'
--- hooks/cinder-common 2012-11-28 17:45:31 +0000
+++ hooks/cinder-common 2012-12-05 17:45:26 +0000
@@ -2,7 +2,7 @@
22
3CHARM="cinder"3CHARM="cinder"
44
5COMMON_PACKAGES="cinder-common python-mysqldb"5COMMON_PACKAGES="cinder-common python-mysqldb gdisk"
6API_PACKAGES="cinder-api"6API_PACKAGES="cinder-api"
7VOL_PACKAGES="cinder-volume"7VOL_PACKAGES="cinder-volume"
8SCHED_PACKAGES="cinder-scheduler"8SCHED_PACKAGES="cinder-scheduler"
@@ -124,6 +124,10 @@
124 fi124 fi
125 echo "yes" | pvremove -ff "$block_dev" ||125 echo "yes" | pvremove -ff "$block_dev" ||
126 error_out "Could not pvremove $block_dev"126 error_out "Could not pvremove $block_dev"
127 else
128 juju-log "Zapping disk of all GPT and MBR structures"
129 sgdisk --zap-all $block_dev ||
130 error_out "Unable to zap $block_dev"
127 fi131 fi
128}132}
129133
130134
=== modified file 'hooks/cinder-hooks'
--- hooks/cinder-hooks 2012-11-29 20:00:26 +0000
+++ hooks/cinder-hooks 2012-12-05 17:45:26 +0000
@@ -153,6 +153,9 @@
153 # XXX: Horrid kludge to make cinder-volume use153 # XXX: Horrid kludge to make cinder-volume use
154 # a different ceph username than admin154 # a different ceph username than admin
155 echo "CEPH_ARGS=--id $SERVICE_NAME" >> /etc/environment155 echo "CEPH_ARGS=--id $SERVICE_NAME" >> /etc/environment
156 # Also add it to the overrides for cinder volume
157 # in preparation for move to start-stop-daemon.
158 echo "env CEPH_ARGS=\"--id $SERVICE_NAME\"" > /etc/init/cinder-volume.override
156159
157 # Create the cinder pool if it does not already exist160 # Create the cinder pool if it does not already exist
158 if ! rados --id $SERVICE_NAME lspools | grep -q cinder; then161 if ! rados --id $SERVICE_NAME lspools | grep -q cinder; then
@@ -169,8 +172,8 @@
169juju-log "cinder: Attempting to fire hook for: $arg0"172juju-log "cinder: Attempting to fire hook for: $arg0"
170case $arg0 in173case $arg0 in
171 "install") install_hook ;;174 "install") install_hook ;;
172 "start") cinder_ctl all start175 "start") cinder_ctl all start;;
173 "stop") cinder_ctl all stop176 "stop") cinder_ctl all stop;;
174 "shared-db-relation-joined") db_joined ;;177 "shared-db-relation-joined") db_joined ;;
175 "shared-db-relation-changed") db_changed ;;178 "shared-db-relation-changed") db_changed ;;
176 "amqp-relation-joined") amqp_joined ;;179 "amqp-relation-joined") amqp_joined ;;
177180
=== modified file 'revision'
--- revision 2012-11-29 00:04:15 +0000
+++ revision 2012-12-05 17:45:26 +0000
@@ -1,1 +1,1 @@
1112

Subscribers

People subscribed via source and target branches

to all changes: