Opt build steps (#1238)

* Add null value judgment

Signed-off-by: ZePan110 <[email protected]>

.github/workflows/pr-helm-test.yaml

@@ -63,7 +63,7 @@ jobs:
 
   Chart-test:
     needs: [job1]
-    if: always() && ${{ needs.job1.outputs.run_matrix.service.length }} > 0
+    if: always() && ${{ fromJSON(needs.job1.outputs.run_matrix).length != 0 }}
     uses: ./.github/workflows/_run-helm-chart.yml
     strategy:
       matrix: ${{ fromJSON(needs.job1.outputs.run_matrix) }}

.github/workflows/push-image-build.yml

@@ -32,22 +32,28 @@ jobs:
       - name: Get Test Services
         id: get-services
         run: |
+          set -x
           base_commit=$(git rev-parse HEAD~1)
           merged_commit=$(git log -1 --format='%H')
           # git diff --name-only ${base_commit} ${merged_commit} | grep -E "cores|comps/__init__.py" | grep -Ev ".md"
-          # if [ $? -eq 0 ]; then
+
           if git diff --name-only ${base_commit} ${merged_commit} | grep -E "cores|comps/__init__.py" | grep -Ev ".md"; then
             echo "ALL image build!!!"
             services=$(basename -a .github/workflows/docker/compose/*-compose.yaml | sed 's/-compose.yaml//' | jq -R '.' )
           else
             changed_src="$(git diff --name-only ${base_commit} ${merged_commit} | grep 'src/' | grep -vE '\.md')" || true
             changed_yamls="$(git diff --name-only ${base_commit} ${merged_commit} | grep '.github/workflows/docker/compose/')" || true
-            services=$(printf '%s\n' "${changed_src[@]}" | cut -d'/' -f2 | grep -vE '\.py' | sort -u | jq -R '.' ) || true
-            while IFS= read -r line; do
-              filename=$(basename "$line" -compose.yaml)
-              echo "$line $(printf '%s\n' "$filename" | jq -R '.' )"
-              services+=" $(printf '%s\n' "$filename" | jq -R '.' )" || true
-            done <<< "$changed_yamls"
+            [[ -n "$changed_src" ]] && services=$(printf '%s\n' "${changed_src[@]}" | cut -d'/' -f2 | grep -vE '\.py' | sort -u | jq -R '.' ) || true
+
+            if [[ -n "$changed_yamls" ]]; then
+              while IFS= read -r line; do
+                filename=$(basename "$line" -compose.yaml)
+                echo "$line $(printf '%s\n' "$filename" | jq -R '.' )"
+                services+=" $(printf '%s\n' "$filename" | jq -R '.' )" || true
+              done <<< "$changed_yamls"
+            else
+              echo "No changes in YAML files."
+            fi
           fi
 
           echo "services=$(echo "$services" | jq -sc 'unique | sort')"
@@ -56,6 +62,7 @@ jobs:
 
   image-build:
     needs: get-build-matrix
+    if: ${{ fromJSON(needs.get-build-matrix.outputs.services).length != 0 }}
     strategy:
       matrix:
         service: ${{ fromJSON(needs.get-build-matrix.outputs.services) }}
@@ -65,6 +72,7 @@ jobs:
     steps:
       - name: Clean up Working Directory
         run: |
+          echo "matrix.service=${{ matrix.service }}"
           sudo rm -rf ${{github.workspace}}/*
 
       - name: Checkout out Repo