Browse code

Merge "Fix install_ceilometer and install_ceilometerclient"

Jenkins authored on 2014/07/23 22:03:19
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
... ...
@@ -802,7 +802,6 @@ if is_service_enabled ceilometer; then
802 802
     install_ceilometer
803 803
     echo_summary "Configuring Ceilometer"
804 804
     configure_ceilometer
805
-    configure_ceilometerclient
806 805
 fi
807 806
 
808 807
 if is_service_enabled heat; then