From: Tim Johnson <tijohnson@linuxfoundation.org>
Date: Wed, 13 Nov 2019 16:24:24 +0000 (-0800)
Subject: Cleanup lf-activate-venv() function
X-Git-Tag: v0.48.1~2
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=c7429332cabb34d18b8343b98f7d7dde54663b48;p=releng%2Fglobal-jjb.git

Cleanup lf-activate-venv() function

Minor changes to logging.

Change-Id: If635d9dcdf024bd7bd1170463ff4484ac81cca25
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>
---

diff --git a/jenkins-init-scripts/lf-env.sh b/jenkins-init-scripts/lf-env.sh
index ae28a19d..37aaf7e1 100644
--- a/jenkins-init-scripts/lf-env.sh
+++ b/jenkins-init-scripts/lf-env.sh
@@ -145,7 +145,7 @@ function lf-activate-venv()
         return 1
     fi
 
-    echo "${FUNCNAME[0]}(): Creating '$python' venv ($lf_tmp_venv)"
+    echo "${FUNCNAME[0]}(): INFO: Creating '$python' venv ($lf_tmp_venv)"
 
     case $python in
         python2*)
@@ -176,7 +176,6 @@ function lf-activate-venv()
             return 0
         fi
         echo "${FUNCNAME[0]}(): INFO: Installing: $pkg_list"
-        echo "Installing: $pkg_list"
         $lf_tmp_venv/bin/pip install --upgrade --quiet --upgrade-strategy eager \
                              $pkg_list || return 1
         ;;
@@ -185,7 +184,7 @@ function lf-activate-venv()
         return 1
         ;;
     esac
-    echo "${FUNCNAME[0]}(): Adding $lf_tmp_venv/bin to PATH"
+    echo "${FUNCNAME[0]}(): INFO: Adding $lf_tmp_venv/bin to PATH"
     PATH=$lf_tmp_venv/bin:$PATH
     return 0