[cicd[ Make coverage comparison optional (llm-d#762)

* make coverage comparison optional

Signed-off-by: Etai Lev Ran <elevran@gmail.com>
Co-authored-by: Alexey Roytman <roytman@il.ibm.com>

* Handle failure cases in extract_total: missing, empty and corrupt files.

In all these cases the function returns "", and the caller handles that as missing data and not a script abort.

Signed-off-by: Etai Lev Ran <elevran@gmail.com>

---------

Signed-off-by: Etai Lev Ran <elevran@gmail.com>
Co-authored-by: Alexey Roytman <roytman@il.ibm.com>

Author: elevran

.github/workflows/ci-pr-checks.yaml

@@ -22,11 +22,11 @@ jobs:
           filters: |
             src:
               - '**/*.go'
-              - '**/*.py'
               - Dockerfile.epp
               - Dockerfile.sidecar
               - Makefile*
               - go.mod
+              - scripts/**
   lint-and-test:
     needs: check-changes
     if: ${{ needs.check-changes.outputs.src == 'true' }}
@@ -77,6 +77,7 @@ jobs:
 
       - name: Compare coverage against main baseline
         if: github.event_name == 'pull_request'
+        continue-on-error: true
         shell: bash
         run: make coverage-compare BASELINE_DIR=coverage/baseline
 

scripts/compare-coverage.sh

@@ -22,12 +22,12 @@ THRESHOLD="${3:-0}"
 # extract_total <profile.out> → percentage as a bare number, e.g. "72.4"
 extract_total() {
     local profile="$1"
-    if [[ ! -f "$profile" ]]; then
+    if [[ ! -s "$profile" ]]; then
         echo ""
         return
     fi
     go tool cover -func="$profile" 2>/dev/null \
-        | awk '/^total:/{gsub(/%/,"",$NF); print $NF}'
+        | awk '/^total:/{gsub(/%/,"",$NF); print $NF}' || true
 }
 
 # delta_str <base> <cur> → e.g. "+1.2" or "-0.5" or "0.0"
@@ -57,7 +57,7 @@ for f in "$BASELINE_DIR"/*.out "$CURRENT_DIR"/*.out; do
     [[ -f "$f" ]] || continue
     name=$(basename "$f" .out)
     # deduplicate
-    if [[ ! " ${all_names[*]} " =~ " ${name} " ]]; then
+    if [[ ! " ${all_names[*]-} " =~ " ${name} " ]]; then
         all_names+=("$name")
     fi
 done