From 4e630a8e8f3a95c2af2fade553aafcb86771071b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Meusel?= <rene.meusel@rohde-schwarz.com>
Date: Fri, 17 Jan 2025 15:44:44 +0100
Subject: [PATCH] Integrate valgrind build matrix into one job

---
 .github/workflows/nightly.yml | 38 +++++++++--------------------------
 1 file changed, 10 insertions(+), 28 deletions(-)

diff --git a/.github/workflows/nightly.yml b/.github/workflows/nightly.yml
index 6c31759634..e121be89fa 100644
--- a/.github/workflows/nightly.yml
+++ b/.github/workflows/nightly.yml
@@ -170,22 +170,30 @@ jobs:
         compiler: ["clang", "gcc"]
         cxxflags: ["-O1", "-O2", "-O3"]
         target:   ["valgrind-ct-full"]
+        platform: ["ubuntu-24.04", "ubuntu-24.04-arm"]
+
+        exclude:
+          - platform: "ubuntu-24.04-arm"
+            cxxflags: "-O1"
 
         include:
           - compiler: clang
             cxxflags: ""              # default compilation flags
             target:   "valgrind-full" # memory bug detection
+            platform: "ubuntu-24.04"
           - compiler: clang
             cxxflags: "-Os"
             # Clang's -Os generated binary is fast enough to run the full test suite.
             target:   "valgrind-ct-full"
+            platform: "ubuntu-24.04"
           - compiler: gcc
             cxxflags: "-Os"
             # GCC with -Os generates a much slower binary, that won't finish
             # before timing out on GH Actions, so we run a reduced set of tests.
             target:   "valgrind-ct"
+            platform: "ubuntu-24.04"
 
-    runs-on: ubuntu-24.04
+    runs-on: ${{ matrix.platform }}
 
     steps:
       - uses: actions/checkout@v4
@@ -197,37 +205,11 @@ jobs:
         uses: ./.github/actions/setup-build-agent
         with:
           target: ${{ matrix.target }}
-          cache-key: linux-x86_64-${{ matrix.compiler }}-${{ matrix.target }}-${{ matrix.cxxflags }}
+          cache-key: ${{ matrix.platform }}-${{ matrix.compiler }}-${{ matrix.target }}-${{ matrix.cxxflags }}
 
       - name: Valgrind Checks
         run: python3 ./src/scripts/ci_build.py --make-tool=make --cc=${{ matrix.compiler }} --custom-optimization-flags="${{ matrix.cxxflags }}" ${{ matrix.target }}
 
-  valgrind_aarch64:
-    name: "valgrind"
-    strategy:
-      fail-fast: false
-
-      matrix:
-        compiler: ["clang", "gcc"]
-        cxxflags: ["-O2", "-O3"]
-
-    runs-on: ubuntu-24.04-arm
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: Read Repository Configuration
-        uses: ./.github/actions/read-repo-config
-
-      - name: Setup Build Agent
-        uses: ./.github/actions/setup-build-agent
-        with:
-          target: valgrind-ct-full
-          cache-key: linux-aarch64-${{ matrix.compiler }}-valgrind-${{ matrix.cxxflags }}
-
-      - name: Valgrind Checks
-        run: python3 ./src/scripts/ci_build.py --make-tool=make --cc=${{ matrix.compiler }} --custom-optimization-flags="${{ matrix.cxxflags }}" valgrind-ct-full
-
   hybrid_tls_interop:
     name: "PQ/T TLS 1.3"