GHA: switch from set-output to env file.

The set-output mechanism is deprecated for security reasons so set
environment variable outputs in GITHUB_OUTPUT instead. Change
get-ci-matrix.py to output the env file settings to stdout and move all
debug output to stderr.

Also, add build-push summary statements for the workflow if the
GITHUB_STEP_SUMMARY variable is set.

Author: kanaka

.github/workflows/main.yml

@@ -18,9 +18,9 @@ jobs:
   get-matrix:
     runs-on: ubuntu-24.04
     outputs:
-      do-linux: ${{ steps.get-matrix-step.outputs.do-linux }}
+      do-linux: ${{ steps.get-matrix-step.outputs.do_linux }}
       matrix-linux: ${{ steps.get-matrix-step.outputs.linux }}
-      do-macos: ${{ steps.get-matrix-step.outputs.do-macos }}
+      do-macos: ${{ steps.get-matrix-step.outputs.do_macos }}
       matrix-macos: ${{ steps.get-matrix-step.outputs.macos }}
     steps:
       - uses: actions/checkout@v4
@@ -31,7 +31,7 @@ jobs:
         run: |
           export OVERRIDE_IMPLS="${{ github.event.inputs.impls }}" # "
           echo "OVERRIDE_IMPLS: ${OVERRIDE_IMPLS}"
-          ./get-ci-matrix.py ${{ steps.files.outputs.all }}
+          ./get-ci-matrix.py ${{ steps.files.outputs.all }} > "${GITHUB_OUTPUT}"
 
   linux:
     needs: get-matrix

ci.sh

@@ -63,10 +63,17 @@ echo "MAKE: ${MAKE}"
 
 case "${ACTION}" in
 docker-build-push)
-    if ! docker pull ${IMAGE}; then
+    if docker pull ${IMAGE}; then
+        [ "${GITHUB_STEP_SUMMARY}" ] && \
+            echo "${MAL_IMPL:-${IMPL}} - pulled ${IMAGE}"
+    else
         make "docker-build^${MAL_IMPL:-${IMPL}}"
+        [ "${GITHUB_STEP_SUMMARY}" ] && \
+            echo "${MAL_IMPL:-${IMPL}} - built ${IMAGE}"
         if [ "${GITHUB_REF}" = "refs/heads/main" ]; then
             docker push ${IMAGE}
+            [ "${GITHUB_STEP_SUMMARY}" ] && \
+                echo "${MAL_IMPL:-${IMPL}} - pushed ${IMAGE}"
         fi
     fi
     ;;

get-ci-matrix.py

@@ -12,6 +12,9 @@
 
 OVERRIDE_IMPLS = os.environ.get('OVERRIDE_IMPLS', '').split()
 
+def eprint(*args, **kwargs):
+    print(*args, file=sys.stderr, **kwargs)
+
 def impl_text(impl):
     s = "IMPL=%s" % impl['IMPL']
     for k, v in impl.items():
@@ -37,11 +40,11 @@ def impl_text(impl):
         do_full = True
 
 
-print("OVERRIDE_IMPLS: %s" % OVERRIDE_IMPLS)
-print("code_changes: %s (%d)" % (code_changes, len(code_changes)))
-print("impl_changes: %s (%d)" % (impl_changes, len(impl_changes)))
-print("run_impls: %s (%d)" % (run_impls, len(run_impls)))
-print("do_full: %s" % do_full)
+eprint("OVERRIDE_IMPLS: %s" % OVERRIDE_IMPLS)
+eprint("code_changes: %s (%d)" % (code_changes, len(code_changes)))
+eprint("impl_changes: %s (%d)" % (impl_changes, len(impl_changes)))
+eprint("run_impls: %s (%d)" % (run_impls, len(run_impls)))
+eprint("do_full: %s" % do_full)
 
 # Load the full implementation description file
 all_impls = yaml.safe_load(open(IMPLS_FILE))
@@ -60,7 +63,7 @@ def impl_text(impl):
     elif do_full:
         targ.append(impl_text(impl))
 
-print("::set-output name=do-linux::%s" % json.dumps(len(linux_impls)>0))
-print("::set-output name=do-macos::%s" % json.dumps(len(macos_impls)>0))
-print("::set-output name=linux::{\"IMPL\":%s}" % json.dumps(linux_impls))
-print("::set-output name=macos::{\"IMPL\":%s}" % json.dumps(macos_impls))
+print("do_linux=%s" % json.dumps(len(linux_impls)>0))
+print("do_macos=%s" % json.dumps(len(macos_impls)>0))
+print("linux='{\"IMPL\":%s}'" % json.dumps(linux_impls))
+print("macos='{\"IMPL\":%s}'" % json.dumps(macos_impls))