Browse code

Fix install_ceilometer and install_ceilometerclient

Other projects do a setup_develop in install_*. This standard is used by
grenade to do installations. Currently grenade uses install ceilometer
as part of the upgrade script, but this doesn't actually install the new
ceilometer.

Change-Id: I4e44bec6574e6a4fcfc252418bc267191aef28fb
Closes-Bug: #1347178

Joe Gordon authored on 2014/07/23 06:26:05
Showing 2 changed files
... ...
@@ -113,16 +113,8 @@ function cleanup_ceilometer {
113 113
     fi
114 114
 }
115 115
 
116
-# configure_ceilometerclient() - Set config files, create data dirs, etc
117
-function configure_ceilometerclient {
118
-    setup_develop $CEILOMETERCLIENT_DIR
119
-    sudo install -D -m 0644 -o $STACK_USER {$CEILOMETERCLIENT_DIR/tools/,/etc/bash_completion.d/}ceilometer.bash_completion
120
-}
121
-
122 116
 # configure_ceilometer() - Set config files, create data dirs, etc
123 117
 function configure_ceilometer {
124
-    setup_develop $CEILOMETER_DIR
125
-
126 118
     [ ! -d $CEILOMETER_CONF_DIR ] && sudo mkdir -m 755 -p $CEILOMETER_CONF_DIR
127 119
     sudo chown $STACK_USER $CEILOMETER_CONF_DIR
128 120
 
... ...
@@ -218,11 +210,15 @@ function init_ceilometer {
218 218
 # install_ceilometer() - Collect source and prepare
219 219
 function install_ceilometer {
220 220
     git_clone $CEILOMETER_REPO $CEILOMETER_DIR $CEILOMETER_BRANCH
221
+    setup_develop $CEILOMETER_DIR
222
+
221 223
 }
222 224
 
223 225
 # install_ceilometerclient() - Collect source and prepare
224 226
 function install_ceilometerclient {
225 227
     git_clone $CEILOMETERCLIENT_REPO $CEILOMETERCLIENT_DIR $CEILOMETERCLIENT_BRANCH
228
+    setup_develop $CEILOMETERCLIENT_DIR
229
+    sudo install -D -m 0644 -o $STACK_USER {$CEILOMETERCLIENT_DIR/tools/,/etc/bash_completion.d/}ceilometer.bash_completion
226 230
 }
227 231
 
228 232
 # start_ceilometer() - Start running processes, including screen
... ...
@@ -796,7 +796,6 @@ if is_service_enabled ceilometer; then
796 796
     install_ceilometer
797 797
     echo_summary "Configuring Ceilometer"
798 798
     configure_ceilometer
799
-    configure_ceilometerclient
800 799
 fi
801 800
 
802 801
 if is_service_enabled heat; then