reduce indent to improve readability

Author: shizunge

shepherd

@@ -87,48 +87,49 @@ update_services() {
 
       if ! DOCKER_CLI_EXPERIMENTAL=enabled docker "${config_flag[@]}" manifest inspect $insecure_registry_flag "$image" > /dev/null; then
         logger "Error updating service $name! Image $image does not exist or it is not available"
-      else
-        logger "Trying to update service $name with image $image" "true"
-
-	# shellcheck disable=SC2086
-        if ! docker "${config_flag[@]}" service update "$name" $detach_option $registry_auth $no_resolve_image_flag ${UPDATE_OPTIONS} --image="$image" > /dev/null; then
-          logger "Service $name update failed on $hostname!"
-          if [[ "${ROLLBACK_ON_FAILURE+x}" ]]; then
-            logger "Rolling $name back"
-	    # shellcheck disable=SC2086
-            docker "${config_flag[@]}" service update "$name" $detach_option $registry_auth $no_resolve_image_flag ${ROLLBACK_OPTIONS} --rollback > /dev/null
-          fi
-          if [[ "$apprise_sidecar_url" != "" ]]; then
-            title="[Shepherd] Service $name update failed on $hostname"
-            body="$(date) Service $name failed to update to $(docker service inspect "$name" -f '{{.Spec.TaskTemplate.ContainerSpec.Image}}')"
-            curl -X POST -H "Content-Type: application/json" --data "{\"title\": \"$title\", \"body\": \"$body\"}" "$apprise_sidecar_url"
-          fi
-          continue    # continue with next service
+        continue    # continue with next service
+      fi
+
+      logger "Trying to update service $name with image $image" "true"
+      # shellcheck disable=SC2086
+      if ! docker "${config_flag[@]}" service update "$name" $detach_option $registry_auth $no_resolve_image_flag ${UPDATE_OPTIONS} --image="$image" > /dev/null; then
+        logger "Service $name update failed on $hostname!"
+        if [[ "${ROLLBACK_ON_FAILURE+x}" ]]; then
+          logger "Rolling $name back"
+          # shellcheck disable=SC2086
+          docker "${config_flag[@]}" service update "$name" $detach_option $registry_auth $no_resolve_image_flag ${ROLLBACK_OPTIONS} --rollback > /dev/null
         fi
+        if [[ "$apprise_sidecar_url" != "" ]]; then
+          title="[Shepherd] Service $name update failed on $hostname"
+          body="$(date) Service $name failed to update to $(docker service inspect "$name" -f '{{.Spec.TaskTemplate.ContainerSpec.Image}}')"
+          curl -X POST -H "Content-Type: application/json" --data "{\"title\": \"$title\", \"body\": \"$body\"}" "$apprise_sidecar_url"
+        fi
+        continue    # continue with next service
+      fi
+
+      previousImage=$(docker service inspect "$name" -f '{{.PreviousSpec.TaskTemplate.ContainerSpec.Image}}')
+      currentImage=$(docker service inspect "$name" -f '{{.Spec.TaskTemplate.ContainerSpec.Image}}')
+      if [ "$previousImage" == "$currentImage" ]; then
+        logger "No updates to service $name!" "true"
+        continue    # continue with next service
+      fi
+
+      logger "Service $name was updated!"
+      if [[ "$apprise_sidecar_url" != "" ]]; then
+        title="[Shepherd] Service $name updated on $hostname"
+        body="$(date) Service $name was updated from $previousImage to $currentImage"
+        curl -X POST -H "Content-Type: application/json" --data "{\"title\": \"$title\", \"body\": \"$body\"}" "$apprise_sidecar_url"
+      fi
 
-        previousImage=$(docker service inspect "$name" -f '{{.PreviousSpec.TaskTemplate.ContainerSpec.Image}}')
-        currentImage=$(docker service inspect "$name" -f '{{.Spec.TaskTemplate.ContainerSpec.Image}}')
-        if [ "$previousImage" == "$currentImage" ]; then
-          logger "No updates to service $name!" "true"
-        else
-          logger "Service $name was updated!"
-          if [[ "$apprise_sidecar_url" != "" ]]; then
-            title="[Shepherd] Service $name updated on $hostname"
-            body="$(date) Service $name was updated from $previousImage to $currentImage"
-            curl -X POST -H "Content-Type: application/json" --data "{\"title\": \"$title\", \"body\": \"$body\"}" "$apprise_sidecar_url"
-          fi
-
-          if [[ "$image_autoclean_limit" != "" ]]; then
-            logger "Cleaning up old docker images, leaving last $image_autoclean_limit"
-            img_name=$(docker service inspect "$name" -f '{{.Spec.TaskTemplate.ContainerSpec.Image}}' | awk -F':' '{print $1}')
-            image_ids=$(docker images -aq --filter=reference="$img_name")
-            image_ids_count=$(echo "$image_ids" | wc -w)
-            if [[ $image_ids_count > $image_autoclean_limit ]]; then
-              docker container prune -f
-              read -ra images_to_remove < <(echo "$image_ids" | xargs -n 1 | tail -n $(("$image_ids_count" - "$image_autoclean_limit")))
-              docker rmi "${images_to_remove[@]}"
-            fi
-          fi
+      if [[ "$image_autoclean_limit" != "" ]]; then
+        logger "Cleaning up old docker images, leaving last $image_autoclean_limit"
+        img_name=$(docker service inspect "$name" -f '{{.Spec.TaskTemplate.ContainerSpec.Image}}' | awk -F':' '{print $1}')
+        image_ids=$(docker images -aq --filter=reference="$img_name")
+        image_ids_count=$(echo "$image_ids" | wc -w)
+        if [[ $image_ids_count > $image_autoclean_limit ]]; then
+          docker container prune -f
+          read -ra images_to_remove < <(echo "$image_ids" | xargs -n 1 | tail -n $(("$image_ids_count" - "$image_autoclean_limit")))
+          docker rmi "${images_to_remove[@]}"
         fi
       fi
     fi