Fix GitHub Actions workflow issues

- Add fail-fast: false to prevent cancellations - Fix typos in artifact names and make them unique - Add proper error handling and verbosity to Python tests - Fix gather-digests job name and make it more resilient - Add proper ARM64 skip configuration
google · kasinadhsarma · Oct 24, 2024 · Oct 24, 2024 · Oct 24, 2024 · Oct 24, 2024
commit 70afaf3cda3cc66e68cac1424394489cc9ffea85
diff --git a/.github/workflows/cmake.yml b/.github/workflows/cmake.yml
@@ -14,6 +14,7 @@ permissions:
 jobs:
   build:
     strategy:
+      fail-fast: false
       matrix:
         os: [ ubuntu-latest, ubuntu-20.04, windows-latest, macOS-11 ]
         arch: [ x64 ]
@@ -23,7 +24,7 @@ jobs:
     runs-on: ${{ matrix.os }}
 
     permissions:
-      contents: write # svenstaro/upload-release-action 
+      contents: write # svenstaro/upload-release-action
 
     steps:
     - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
@@ -63,14 +64,14 @@ jobs:
       working-directory: ${{github.workspace}}/python
       run: |
         python -m pip install --require-hashes --no-dependencies -r ../.github/workflows/requirements/base.txt
-        python setup.py build
-        python setup.py bdist_wheel
-        python -m pytest
+        python setup.py build -v
+        python setup.py bdist_wheel -v
+        python -m pytest -v --log-cli-level=INFO
 
-    - name: Upload artifcacts
+    - name: Upload artifacts
       uses: actions/upload-artifact@v3
       with:
-        name: artifcacts
+        name: artifacts-${{ matrix.os }}-${{ matrix.arch }}
         path: ./build/*.7z
 
     - name: Upload Release Assets

diff --git a/.github/workflows/wheel.yml b/.github/workflows/wheel.yml
@@ -18,6 +18,7 @@ jobs:
       digests-macos: ${{ steps.hash-macos.outputs.digests }}
       digests-windows: ${{ steps.hash-windows.outputs.digests }}
     strategy:
+      fail-fast: false
       matrix:
         os: [ubuntu-latest, windows-latest, macOS-11]
     runs-on: ${{ matrix.os }}
@@ -69,8 +70,8 @@ jobs:
           CIBW_ARCHS_LINUX: auto aarch64
           CIBW_ARCHS_MACOS: x86_64 universal2 arm64
           CIBW_ARCHS_WINDOWS: auto ARM64
-          CIBW_SKIP: "pp* *-musllinux_*"
-          CIBW_BUILD_VERBOSITY: 1
+          CIBW_SKIP: "pp* *-musllinux_* *-win_arm64"
+          CIBW_BUILD_VERBOSITY: 2
 
       - name: Build sdist archive
         working-directory: ${{github.workspace}}/python
@@ -93,7 +94,7 @@ jobs:
       - name: Upload artifact
         uses: actions/upload-artifact@v3
         with:
-          name: artifacts
+          name: artifacts-${{ matrix.os }}
           path: |
             ./python/wheelhouse/*.whl
             ./python/wheelhouse/*.tar.gz
@@ -124,7 +125,7 @@ jobs:
         if: runner.os == 'Windows'
         run: echo "digests=$(sha256sum ./python/wheelhouse/* | base64 -w0)" >> $GITHUB_OUTPUT
 
-  gather-disgests:
+  gather-digests:
     needs: [build_wheels]
     outputs:
       digests: ${{ steps.hash.outputs.digests }}
@@ -138,9 +139,16 @@ jobs:
           WINDOWS_DIGESTS: "${{ needs.build_wheels.outputs.digests-windows }}"
         run: |
           set -euo pipefail
-          echo "$LINUX_DIGESTS" | base64 -d > checksums.txt
-          echo "$MACOS_DIGESTS" | base64 -d >> checksums.txt
-          echo "$WINDOWS_DIGESTS" | base64 -d >> checksums.txt
+          touch checksums.txt
+          if [ ! -z "${LINUX_DIGESTS:-}" ]; then
+            echo "$LINUX_DIGESTS" | base64 -d >> checksums.txt
+          fi
+          if [ ! -z "${MACOS_DIGESTS:-}" ]; then
+            echo "$MACOS_DIGESTS" | base64 -d >> checksums.txt
+          fi
+          if [ ! -z "${WINDOWS_DIGESTS:-}" ]; then
+            echo "$WINDOWS_DIGESTS" | base64 -d >> checksums.txt
+          fi
           echo "digests=$(cat checksums.txt | base64 -w0)" >> $GITHUB_OUTPUT
 
   provenance: