Introduce DaCapo cache job dependency (#208)

This commit also allows the old CI runs to be canceled if the PR is
updated.

Closes #197.

Author: k-sareen

.github/workflows/ci-assert.yml

@@ -1,10 +1,14 @@
-name: Dacapo Tests
+name: DaCapo Tests
 
 on:
   pull_request:
     branches:
       - master
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+
 jobs:
   test:
     runs-on: ubuntu-18.04
@@ -16,13 +20,13 @@ jobs:
           ./.github/scripts/ci-setup.sh
 
       # Run the tests
-      - name: Dacapo Tests
+      - name: DaCapo Tests
         run: |
           ./.github/scripts/ci-build.sh
           ./.github/scripts/ci-test-only-normal.sh
-      - name: DaCapo Tests for Global Allocation Bit
+      - name: DaCapo Tests with global allocation bit
         run: ./.github/scripts/ci-test-global-alloc-bit.sh
-      - name: DaCapo Tests for malloc mark sweep
+      - name: DaCapo Tests with malloc Mark-Sweep
         run: ./.github/scripts/ci-test-malloc-mark-sweep.sh
 
       # Style check
@@ -39,11 +43,24 @@ jobs:
           ./.github/scripts/ci-setup.sh
 
       # Run the tests
-      - name: Dacapo Tests with Weak ref
+      - name: DaCapo Tests with weak reference processing
         run: |
           ./.github/scripts/ci-build.sh
           ./.github/scripts/ci-test-only-weak-ref.sh
 
+  test-assertions:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup Environments
+        run: |
+            ./.github/scripts/ci-checkout.sh
+            ./.github/scripts/ci-setup.sh
+
+      # Run the tests with assertions
+      - name: DaCapo Test (with extreme_assertions)
+        run: ./.github/scripts/ci-test-assertions.sh
+
   msrv:
     runs-on: ubuntu-18.04
     steps:

.github/workflows/ci.yml

@@ -6,7 +6,7 @@ on:
       - master
 
 concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
+  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
   cancel-in-progress: true
 
 jobs:
@@ -49,10 +49,29 @@ jobs:
           path: ./git/openjdk/build/linux-x86_64-normal-server-${{ matrix.debug-level }}/bundles/*_bin.tar.gz
           retention-days: 2
 
+  cache-dacapo:
+    name: Cache DaCapo Chopin git-04132797
+    runs-on: ubuntu-22.04
+    steps:
+      - name: Check DaCapo Chopin git-04132797 cache
+        id: dacapo-04132797
+        uses: actions/cache@v3
+        with:
+          path: dacapo/dacapo-evaluation-git-04132797.zip
+          key: dacapo-chopin-git-04132797
+      - name: Install DaCapo Chopin git-04132797
+        if: steps.dacapo-04132797.outputs.cache-hit != 'true'
+        run: |
+          mkdir -p dacapo
+          pushd dacapo
+          wget -q "https://download.dacapobench.org/dacapo-evaluation-git-04132797.zip" -O dacapo-evaluation-git-04132797.zip
+          popd
+
   test-linux-x64:
     name: linux-x64
     needs:
       - build-linux-x64
+      - cache-dacapo
     runs-on: ubuntu-22.04
     strategy:
       fail-fast: false
@@ -95,13 +114,7 @@ jobs:
         with:
           path: dacapo/dacapo-evaluation-git-04132797.zip
           key: dacapo-chopin-git-04132797
-      - name: Install DaCapo Chopin git-04132797
-        if: steps.dacapo-04132797.outputs.cache-hit != 'true'
-        run: |
-          mkdir -p dacapo
-          pushd dacapo
-          wget -q "https://cloudstor.aarnet.edu.au/plus/s/9cMreAgANcMUOzq/download" -O dacapo-evaluation-git-04132797.zip
-          popd
+          fail-on-cache-miss: true    # We should never have a cache miss here as we cache DaCapo in an earlier job
       - name: Unzip DaCapo Chopin git-04132797
         run: |
           pushd dacapo