Bugfix: Fix lab reconfig issue by moving fuel dependency installation to master node 75/1775/3
authorFatih Degirmenci <fatih.degirmenci@ericsson.com>
Fri, 18 Sep 2015 23:18:20 +0000 (01:18 +0200)
committerFatih Degirmenci <fatih.degirmenci@ericsson.com>
Fri, 18 Sep 2015 23:22:32 +0000 (23:22 +0000)
ODL plugin build dependency installation was incorrectly put into lab-reconfig
which runs on master node. This change moves the dependency installation into
fuel-odl-plugin-build builder which runs on jumphost.

Change-Id: Ic353cabeabb0a98fcdb8b2f94ababbbc79d3e963
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
jjb/genesis/genesis-fuel.yml
utils/lab-reconfiguration/create_venv.sh

index f99fa93..f13dc07 100644 (file)
             set -o nounset
             set -o pipefail
 
+            if [[ ! $(rpm -qa | grep rubygems) ]]; then
+                echo "Rubygems not found, but required for Fuel Plugin build ... attempting to install"
+                sudo yum install -y rubygems ruby-devel
+            fi
+
+            if [[ ! $(gem list | grep fpm) ]]; then
+                echo "fpm gem not found, but required for Fuel Plugin build ... attempting to install"
+                sudo gem install fpm
+            fi
+
             # log info to console
             echo "Starting the build of Fuel Opendaylight Plugin. This could take some time..."
             echo "--------------------------------------------------------"
index e94b22b..4e45f84 100755 (executable)
@@ -30,16 +30,6 @@ if [[ ! $(rpm -qa | grep python-2.7) ]]; then
     sudo yum install -y python-2.7*
 fi
 
-if [[ ! $(rpm -qa | grep rubygems) ]]; then
-    echo "Rubygems not found, but required for Fuel Plugin build ... attempting to install"
-    sudo yum install -y rubygems ruby-devel
-fi
-
-if [[ ! $(gem list | grep fpm) ]]; then
-    echo "fpm gem not found, but required for Fuel Plugin build ... attempting to install"
-    sudo gem install fpm
-fi
-
 if [ ! -f "$BASEDIR/venv/updated" -o $BASEDIR/requirements.pip -nt $BASEDIR/venv/updated ]; then
     source $BASEDIR/venv/bin/activate
     pip install -r $BASEDIR/requirements.pip