From: Bartek Grzybowski <b.grzybowski@partner.samsung.com>
Date: Fri, 17 Jan 2020 13:54:11 +0000 (+0100)
Subject: Run env setup only once
X-Git-Tag: 6.0.0-ONAP~50^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F11%2F100411%2F1;p=oom%2Foffline-installer.git

Run env setup only once

prep_ubuntu_16_04_for_molecule routine was taken out of
run_molecule function so that it's not superfluously run
on each molecule invocation

Change-Id: I1e4d3f32fbec4fa1e96389d9d65740747bffcef6
Signed-off-by: Bartek Grzybowski <b.grzybowski@partner.samsung.com>
Issue-ID: OOM-1853
---

diff --git a/offline-installer-review.sh b/offline-installer-review.sh
index 69f438dd..f839308a 100755
--- a/offline-installer-review.sh
+++ b/offline-installer-review.sh
@@ -31,7 +31,6 @@ function prep_ubuntu_16_04_for_molecule() {
 }
 
 function run_molecule() {
-  prep_ubuntu_16_04_for_molecule
   local roles=("$@")
   local MOLECULE_RC
   for role in ${roles[@]}
@@ -51,6 +50,8 @@ function run_molecule() {
 #######################################################################$
 FAILED_ROLES=()
 ALL_PLAYBOOKS=(`ls -d ansible/test/play-*`) # enumerate all playbook tests for later usage
+# Setup environment
+prep_ubuntu_16_04_for_molecule
 
 # Check for changes in Ansible roles
 ROLE_CHANGES=(`git diff HEAD^ HEAD --name-only ansible/roles | cut -f 1-3 -d "/" | sort -u`)