bash format 01/25601/1
authorJulien <zhang.jun3g@zte.com.cn>
Wed, 7 Dec 2016 07:48:22 +0000 (15:48 +0800)
committerJulien <zhang.jun3g@zte.com.cn>
Wed, 7 Dec 2016 07:55:33 +0000 (15:55 +0800)
1. use 4 header spaces instead of 8
2. keep max line width less than 80
3. don't mix use header tab and header space according pep8
4. for we use -x debug switch, no necessary to echo debug info

Change-Id: I1cb4adf0fcfdad3bc015213ae4ad9b822748dd82
Signed-off-by: Julien <zhang.jun3g@zte.com.cn>
ci/build_rpm/build_rpms.sh

index 2d5d23e..279340c 100755 (executable)
@@ -13,54 +13,51 @@ DAISYDIR=$1
 OPNFV_ARTIFACT_VERSION=$2
 
 function build_rpm_pkg {
-        # Cleanup prev build resutls
-        rm -rf $DAISYDIR/build_output
-        mkdir -p $DAISYDIR/build_output
-
-        sudo docker build -t daisy4nfv_rpm .
-        sudo docker run --rm -v $DAISYDIR:/opt/daisy4nfv -v $CACHE_DIRECTORY:/home/cache -t  daisy4nfv_rpm \
-                      /opt/daisy4nfv/ci/build_rpm/build_rpms_docker.sh $OPNFV_ARTIFACT_VERSION
-
-       # Here to collect build result from $DAISYDIR/build_output
+    # Cleanup prev build resutls
+    rm -rf $DAISYDIR/build_output
+    mkdir -p $DAISYDIR/build_output
+
+    sudo docker build -t daisy4nfv_rpm .
+    sudo docker run --rm -v $DAISYDIR:/opt/daisy4nfv \
+                    -v $CACHE_DIRECTORY:/home/cache \
+                    -t daisy4nfv_rpm \
+                    /opt/daisy4nfv/ci/build_rpm/build_rpms_docker.sh \
+                    $OPNFV_ARTIFACT_VERSION
 }
 
 function cleanup_container {
-        containers_to_kill=$(sudo docker ps --filter "label=daisy_image_version" \
-                --format "{{.Names}}" -a)
-
-        if [[ -z "$containers_to_kill" ]]; then
-                echo "No containers to cleanup."
-        else
-                volumes_to_remove=$(sudo docker inspect -f \
-                        '{{range .Mounts}} {{printf "%s\n" .Name }}{{end}}' \
-                        ${containers_to_kill} | egrep -v '(^\s*$)' | sort | uniq)
-                echo "Stopping containers... $containers_to_kill"
-                sudo docker stop -t 2 ${containers_to_kill} >/dev/null 2>&1
-
-                echo "Removing containers... $containers_to_kill"
-                sudo docker rm -v -f ${containers_to_kill} >/dev/null 2>&1
-
-                if [[ -z "$volumes_to_remove" ]]; then
-                        echo "No volumes to cleanup."
-                else
-                        echo "Removing volumes... $volumes_to_remove"
-                        sudo docker volume rm ${volumes_to_remove} >/dev/null 2>&1
-                fi
+    containers_to_kill=$(sudo docker ps -a \
+                                        --filter "label=daisy_image_version" \
+                                        --format "{{.Names}}")
+
+    if [[ ! -z "$containers_to_kill" ]]; then
+        ops='{{range .Mounts}} {{printf "%s\n" .Name }}{{end}}'
+        volumes_to_remove=$(sudo docker inspect -f \
+                                                $ops ${containers_to_kill} \
+                                                | egrep -v '(^\s*$)' \
+                                                | sort | uniq)
+
+        sudo docker stop -t 2 ${containers_to_kill} >/dev/null 2>&1
+        sudo docker rm -v -f ${containers_to_kill} >/dev/null 2>&1
+
+        if [[ ! -z "$volumes_to_remove" ]]; then
+            sudo docker volume rm ${volumes_to_remove} >/dev/null 2>&1
         fi
+    fi
 }
 
 function cleanup_docker_image {
-        images_to_delete=$(sudo docker images -a --filter "label=daisy_image_version" \
-                --format "{{.ID}}")
-
-        echo "Removing images... $images_to_delete"
-        if [[ -z "$images_to_delete" ]]; then
-                echo "No images to cleanup"
-        else
-                sudo docker rmi -f ${images_to_delete} >/dev/null 2>&1
-        fi
+    images_to_delete=$(sudo docker images -a \
+                                          --format "{{.ID}}" \
+                                          --filter "label=daisy_image_version")
+
+    echo "Removing images... $images_to_delete"
+    if [[ ! -z "$images_to_delete" ]]; then
+        sudo docker rmi -f ${images_to_delete} >/dev/null 2>&1
+    fi
 }
 
 cleanup_container
 build_rpm_pkg
 cleanup_container
+