Comment usage of parallel-vars.sh

+ Add comments to scripts that need the values.
+ Add comments to shell stanzas to not split inclusions.
+ Remove parallel-vars.sh from a job that does not use the values.
  - It seems compilation time is not affected by the values.

Change-Id: If844486411753fcac7bece1db8416f2d23ea3749
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

diff --git a/jjb/scripts/vpp/arm-build-no-test.sh b/jjb/scripts/vpp/arm-build-no-test.sh
index 94cc824..ae5aa34 100644 (file)
--- a/jjb/scripts/vpp/arm-build-no-test.sh
+++ b/jjb/scripts/vpp/arm-build-no-test.sh
@@ -1,4 +1,10 @@
 #!/bin/bash
+
+# This script wants environment variables
+# set by parallel-vars.sh script.
+# Otherwise "make test" is not parallelized,
+# possibly leading to jobs overstepping their time limit.
+
 # basic build script example
 set -xe -o pipefail
 

diff --git a/jjb/scripts/vpp/arm-build.sh b/jjb/scripts/vpp/arm-build.sh
index 9de736a..2abe610 100644 (file)
--- a/jjb/scripts/vpp/arm-build.sh
+++ b/jjb/scripts/vpp/arm-build.sh
@@ -1,4 +1,10 @@
 #!/bin/bash
+
+# This script wants environment variables
+# set by parallel-vars.sh script.
+# Otherwise "make test" is not parallelized,
+# possibly leading to jobs overstepping their time limit.
+
 # basic build script example
 set -xe -o pipefail
 

diff --git a/jjb/vpp/vpp.yaml b/jjb/vpp/vpp.yaml
index 08fcf3c..1faaf52 100644 (file)
--- a/jjb/vpp/vpp.yaml
+++ b/jjb/vpp/vpp.yaml
@@ -386,6 +386,7 @@
       - shell:
           !include-raw-escape:
           - ../scripts/vpp/parallel-vars.sh
+          # Do not split, the line below wants env vars from the line above.
           - ../scripts/vpp/arm-build.sh
 
     publishers:
@@ -461,6 +462,7 @@
       - shell:
           !include-raw-escape:
           - ../scripts/vpp/parallel-vars.sh
+          # Do not split, the line below wants env vars from the line above.
           - ../scripts/vpp/arm-build-no-test.sh
 
       - provide-maven-settings:
@@ -949,9 +951,6 @@
       - shell:
           !include-raw-escape:
           - ../scripts/setup_vpp_dpdk_dev_env.sh
-      - shell:
-          !include-raw-escape:
-          - ../scripts/vpp/parallel-vars.sh
       - shell:
           !include-raw-escape:
           - ../scripts/vpp/csit-device.sh