From a0e49a7022e4d2addd302c00e7a668391a5b2615 Mon Sep 17 00:00:00 2001
From: Carlos <carlos@carlossegarra.com>
Date: Thu, 14 Mar 2024 11:18:26 +0000
Subject: [PATCH] gha: fix wasm cache key (#37)

* gha: fix wasm cache key

* gha: still update python/cpp

* gha: checkout with submodules for right cache key
---
 .github/workflows/tests.yml | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 9c39daf..fbc3088 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -142,6 +142,8 @@ jobs:
 
       - name: "Checkout code"
         uses: actions/checkout@v4
+        with:
+          submodules: true
       - name: "Install faasmctl"
         run: pip3 install $(cat ./requirements.txt | grep faasmctl | tr -d '\n')
       - name: "Start a Faasm cluster to run the examples"
@@ -158,16 +160,13 @@ jobs:
         run: |
           sudo apt install -y zstd
           git config --global --add safe.directory "$GITHUB_WORKSPACE"
-          git submodule update --init ./cpp
-          git submodule update --init ./python
-          echo "cpp-commit=$(git submodule status ./cpp | cut '-d ' -f 2)" >> $GITHUB_OUTPUT
-          echo "py-commit=$(git submodule status ./python | cut '-d ' -f 2)" >> $GITHUB_OUTPUT
+          echo "digest=$(git submodule status | awk '{ print $1; }' | md5sum | cut -d' ' -f1)" >> $GITHUB_OUTPUT
       - name: "Get WASM cache"
         uses: actions/cache/restore@v4
         id: cpp-wasm-cache
         with:
           path: ./wasm
-          key: wasm-${{ steps.submodule-commit.outputs.cpp-commit }}-${{ steps.submodule-commit.outputs.py-commit }}
+          key: wasm-${{ steps.submodule-commit.outputs.digest }}
           fail-on-cache-miss: true
       - name: "Upload all WASM files to the cluster"
         id: wasm-upload
@@ -182,11 +181,9 @@ jobs:
           # in tasks/kernels.py
           # amr: does not work because we are missing MPI_Comm_split
           # faasmctl invoke kernels-mpi amr --cmdline '10 1024 16 16 16 8 4 HIGH_WATER' --mpi-world-size 4
-          echo "branch"
           # faasmctl invoke kernels-mpi branch --cmdline '10 10 vector_go' --mpi-world-size 4
           # dgemm: does not work because we are missing MPI_Comm_group, MPI_Group_incl, and MPI_Comm_create
           # faasmctl invoke kernels-mpi dgemm --cmdline '10 1024 32 1' --mpi-world-size 4
-          echo "nstream"
           faasmctl invoke kernels-mpi nstream --cmdline '10 1024 32' --mpi-world-size 4
           # pic: does not work due to what looks like an integer overflow
           # faasmctl invoke kernels-mpi pic --cmdline "10 10 10 2 5 SINUSOIDAL" --mpi-world-size 4