Fix: bashate E010 warnings

E010 rule ensures consistency of "do" directive being on the same line
as it's command.

Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: Id7717658728968da017107c4b496cdfbe0c5b6b0

diff --git a/shell/deploy-maven-file.sh b/shell/deploy-maven-file.sh
index e5bf050..9721ee2 100644 (file)
--- a/shell/deploy-maven-file.sh
+++ b/shell/deploy-maven-file.sh
@@ -37,8 +37,7 @@ export MAVEN_PARAMS
 DEPLOY_LOG="$WORKSPACE/archives/deploy-maven-file.log"
 mkdir -p "$WORKSPACE/archives"
 
-while IFS="" read -r file
-do
+while IFS="" read -r file; do
     file_size=$(stat --printf="%s" "${file}")
     echo "Deploy ${file##*/} with ${file_size} bytes."
     lftools deploy maven-file "$MAVEN_REPO_URL" \

diff --git a/shell/docker-login.sh b/shell/docker-login.sh
index 72d420f..751ac20 100644 (file)
--- a/shell/docker-login.sh
+++ b/shell/docker-login.sh
@@ -94,8 +94,7 @@ do_login() {
 # Loop through Registry and Ports to concatentate and login to nexus
 if [ "${DOCKER_REGISTRY:-none}" != 'none' ]
 then
-    for PORT in $REGISTRY_PORTS
-    do
+    for PORT in $REGISTRY_PORTS; do
         REGISTRY="${DOCKER_REGISTRY}:${PORT}"
 
         # docker login requests an email address if nothing is passed to it

diff --git a/shell/jjb-verify-build-nodes.sh b/shell/jjb-verify-build-nodes.sh
index b4457e4..93d85be 100644 (file)
--- a/shell/jjb-verify-build-nodes.sh
+++ b/shell/jjb-verify-build-nodes.sh
@@ -44,7 +44,9 @@ isBadLabel () {
 isValueInArray () {
   local e match="$1"
   shift
-  for e; do [[ "$e" == "$match" ]] && return 0; done
+  for e; do
+      [[ "$e" == "$match" ]] && return 0
+  done
   return 1
 }
 

diff --git a/shell/openstack-kubernetes-create.sh b/shell/openstack-kubernetes-create.sh
index 5ee05b0..e94dc94 100755 (executable)
--- a/shell/openstack-kubernetes-create.sh
+++ b/shell/openstack-kubernetes-create.sh
@@ -59,8 +59,7 @@ cluster_uuid=$(openstack coe cluster create "$cluster_name" \
 # Sleep for a little, because sometimes OpenStack has to catch up with itself
 sleep 15
 
-while [ "$(openstack --os-cloud "$os_cloud" coe cluster show "$cluster_uuid" -c status -f value)" == "CREATE_IN_PROGRESS" ]
-do
+while [ "$(openstack --os-cloud "$os_cloud" coe cluster show "$cluster_uuid" -c status -f value)" == "CREATE_IN_PROGRESS" ]; do
   # echo "sleeping $(date)"
   sleep 2m
 done

diff --git a/shell/pipeline-linter.sh b/shell/pipeline-linter.sh
index 8ed681d..fad1f72 100644 (file)
--- a/shell/pipeline-linter.sh
+++ b/shell/pipeline-linter.sh
@@ -18,8 +18,7 @@ JENKINS_VAL="$JENKINS_URL/pipeline-model-converter/validate"
 mapfile -t JENKINS_FILE_LIST < <(grep -lr "^pipeline\s*{" vars src Jenkinsfile*)
 exit_code=0
 
-for JENKINS_FILE in "${JENKINS_FILE_LIST[@]}"
-do
+for JENKINS_FILE in "${JENKINS_FILE_LIST[@]}"; do
     ret=$(curl --silent -X POST -H "$JENKINS_CRUMB" -F "jenkinsfile=<$JENKINS_FILE" "$JENKINS_VAL")
     if [[ $ret == *"Errors"* ]];then
         echo "ERROR: Linting error for $JENKINS_FILE"

diff --git a/shell/sysstat.sh b/shell/sysstat.sh
index 5b49625..e071ad1 100644 (file)
--- a/shell/sysstat.sh
+++ b/shell/sysstat.sh
@@ -47,8 +47,7 @@ SAR_DIR="$WORKSPACE/archives/sar-reports"
 mkdir -p "$SAR_DIR"
 cp "$SYSSTAT_PATH/"* "$_"
 # convert sar data to ascii format
-while IFS="" read -r sarfilenum
-do
+while IFS="" read -r sarfilenum; do
     [ -f "$sarfilenum" ] && LC_TIME=POSIX sar -A -f "$sarfilenum" > "$SAR_DIR/sar${sarfilenum//[!0-9]/}"
 done < <(find "$SYSSTAT_PATH" -name "sa[0-9]*" || true)