hs-test: output docker build command for ci console log

- clean up shellcheck warnings
- rename extras/hs-test/test script
- add -x attribute to compress script for consistancy

Type: test

Change-Id: I5d1a9d16eeaff18562461b1e445e32ac696266d3
Signed-off-by: Dave Wallace <dwallacelf@gmail.com>

5 files changed, 16 insertions, 14 deletions

diff --git a/extras/hs-test/script/build_boringssl.sh b/extras/hs-test/script/build_boringssl.sh
index 441878a77ca..cca4e4a31d1 100755
--- a/extras/hs-test/script/build_boringssl.sh
+++ b/extras/hs-test/script/build_boringssl.sh
@@ -1,4 +1,4 @@
 #!/bin/bash
-cd boringssl
+cd boringssl || exit 1
 cmake -GNinja -B build
 ninja -C build
diff --git a/extras/hs-test/script/build_hst.sh b/extras/hs-test/script/build_hst.sh
index 3b4bc28e275..f3a8e272207 100755
--- a/extras/hs-test/script/build_hst.sh
+++ b/extras/hs-test/script/build_hst.sh
@@ -1,17 +1,17 @@
 #!/usr/bin/env bash
 
-if [ $(lsb_release -is) != Ubuntu ]; then
+if [ "$(lsb_release -is)" != Ubuntu ]; then
 	echo "Host stack test framework is supported only on Ubuntu"
 	exit 1
 fi
 
-if [ -z $(which ab) ]; then
+if [ -z "$(which ab)" ]; then
 	echo "Host stack test framework requires apache2-utils to be installed"
 	echo "It is recommended to run 'sudo make install-dep'"
 	exit 1
 fi
 
-if [ -z $(which wrk) ]; then
+if [ -z "$(which wrk)" ]; then
 	echo "Host stack test framework requires wrk to be installed"
 	echo "It is recommended to run 'sudo make install-dep'"
 	exit 1
@@ -28,9 +28,7 @@ else
 fi
 echo "Taking build objects from ${VPP_BUILD_ROOT}"
 
-if [ -z "$UBUNTU_VERSION" ] ; then
-	export UBUNTU_VERSION=$(lsb_release -rs)
-fi
+export UBUNTU_VERSION=${UBUNTU_VERSION:-"$(lsb_release -rs)"}
 echo "Ubuntu version is set to ${UBUNTU_VERSION}"
 
 export HST_LDPRELOAD=${VPP_BUILD_ROOT}/lib/x86_64-linux-gnu/libvcl_ldpreload.so
@@ -57,12 +55,14 @@ fi
 docker_build () {
     tag=$1
     dockername=$2
+    set -x
     docker build --build-arg UBUNTU_VERSION             \
-                 --build-arg http_proxy=$HTTP_PROXY     \
-                 --build-arg https_proxy=$HTTP_PROXY    \
-                 --build-arg HTTP_PROXY=$HTTP_PROXY     \
-                 --build-arg HTTPS_PROXY=$HTTP_PROXY    \
-                 -t $tag -f docker/Dockerfile.$dockername .
+                 --build-arg http_proxy="$HTTP_PROXY"     \
+                 --build-arg https_proxy="$HTTP_PROXY"    \
+                 --build-arg HTTP_PROXY="$HTTP_PROXY"     \
+                 --build-arg HTTPS_PROXY="$HTTP_PROXY"    \
+                 -t "$tag" -f docker/Dockerfile."$dockername" .
+    set +x
 }
 
 docker_build hs-test/vpp vpp
@@ -77,5 +77,6 @@ fi
 # cleanup detached images
 images=$(docker images --filter "dangling=true" -q --no-trunc)
 if [ "$images" != "" ]; then
+		# shellcheck disable=SC2086
     docker rmi $images
 fi
diff --git a/extras/hs-test/script/build_nginx.sh b/extras/hs-test/script/build_nginx.sh
index 69d366aab0e..f21201c4297 100755
--- a/extras/hs-test/script/build_nginx.sh
+++ b/extras/hs-test/script/build_nginx.sh
@@ -1,5 +1,5 @@
 #!/bin/bash
-cd nginx
+cd nginx || exit 1
 ./auto/configure --with-debug --with-http_v3_module --with-cc-opt="-I../boringssl/include" --with-ld-opt="-L../boringssl/build/ssl -L../boringssl/build/crypto" --without-http_rewrite_module --without-http_gzip_module
 make
 make install
diff --git a/extras/hs-test/script/compress.sh b/extras/hs-test/script/compress.sh
index 1f0205c1efb..1f0205c1efb 100644..100755
--- a/extras/hs-test/script/compress.sh
+++ b/extras/hs-test/script/compress.sh
diff --git a/extras/hs-test/script/nginx_ldp.sh b/extras/hs-test/script/nginx_ldp.sh
index a6b0a20e47b..416aa5499af 100755
--- a/extras/hs-test/script/nginx_ldp.sh
+++ b/extras/hs-test/script/nginx_ldp.sh
@@ -1,3 +1,4 @@
 #!/usr/bin/env bash
 
-$1 -v && LD_PRELOAD=$LDP $@ 2>&1 > /proc/1/fd/1
+# shellcheck disable=SC2068
+$1 -v && LD_PRELOAD=$LDP $@ > /proc/1/fd/1 2>&1