Refactor install_python function 19/6419/2
authorVictor Morales <victor.morales@intel.com>
Mon, 31 Jul 2017 14:10:11 +0000 (09:10 -0500)
committerVictor Morales <victor.morales@intel.com>
Mon, 31 Jul 2017 14:21:20 +0000 (09:21 -0500)
It was extract the installation of pip packages from the
install_python function, this change allows to install python
packages isolated. This also includes the manual configuration
of java certificates required for maven execution.

Change-Id: Iac60d7521101d5c4997f95c16f4decd857cc5c3c
Signed-off-by: Victor Morales <victor.morales@intel.com>
bootstrap/vagrant-onap/lib/functions
bootstrap/vagrant-onap/lib/sdnc
bootstrap/vagrant-onap/lib/vfc

index b0c6e97..a2bb98e 100755 (executable)
@@ -62,6 +62,8 @@ function clone_repo {
 function install_dev_tools {
     install_package apt-transport-https
     install_package ca-certificates
+    # ca-certificates-java is not a dependency in the Oracle JDK/JRE so this must be explicitly installed.
+    /var/lib/dpkg/info/ca-certificates-java.postinst configure
     install_package curl
 }
 
@@ -122,11 +124,23 @@ function install_nodejs {
 function install_python {
     install_package python2.7
     install_package python-dev
+}
+
+# _install_pip() - Install Python Package Manager
+function _install_pip {
+    install_python
     curl -sL https://bootstrap.pypa.io/get-pip.py | python
-    pip install tox
 }
 
-# install_docker() - Download and install docker-engine 
+# install_python_package() - Install a python module
+function install_python_package {
+    local python_package=$1
+
+    _install_pip
+    pip install $python_package
+}
+
+# install_docker() - Download and install docker-engine
 function install_docker {
     if is_package_installed docker-ce; then
         return
@@ -192,7 +206,7 @@ function compile_src {
     pushd $src_folder
     if [ -f pom.xml ]; then
         install_maven
-        mvn clean install -U -DskipTests=true -Dmaven.test.skip=true -Dmaven.javadoc.skip=true -Dadditionalparam=-Xdoclint:none
+        mvn clean install -DskipTests=true -Dmaven.test.skip=true -Dmaven.javadoc.skip=true -Dadditionalparam=-Xdoclint:none
     fi
     popd
 }
index 6765f14..8bf6ae6 100755 (executable)
@@ -32,7 +32,6 @@ function _build_sdnc_images {
     build_docker_image $folder/installation/ubuntu
     asserts_image openecomp/ubuntu-sdnc-image
 
-    /var/lib/dpkg/info/ca-certificates-java.postinst configure
     build_docker_image $folder/installation/sdnc
     asserts_image openecomp/sdnc-image
 
index 44e433d..5aba76f 100755 (executable)
@@ -17,6 +17,8 @@ function clone_all_vfc_repos {
 
 # compile_all_vfc_repos() - Function that compiles VF-C source repo.
 function compile_all_vfc_repos {
+    install_python_package tox
+
     pushd $src_folder/gvnfm/vnflcm/lcm
     tox -e py27
     popd