From d6d4784c379b8067de54b2a09d4d9e64a0499dc9 Mon Sep 17 00:00:00 2001
From: Kefu Chai <kefu.chai@scylladb.com>
Date: Wed, 8 May 2024 09:55:05 +0800
Subject: [PATCH] github/workflow: do not override <clang++,23,release>

before this change, we

* override the combination of <clang++,23,release>
  by adding settings to build with dpdk enabled,
* override the combination of <clang++,23,debug>
  by adding settings to build with C++20 moduels
  enabled.

but what we intended was to add two additional
builds.

so, in this change, we preserve the existing
combinations of <clang++,23,release> and
<clang++,23,debug> by includeing all the
combinations explicitly, and add the builds
with dpdk and C++20 modules. also the matrix
does not allows one to add extra settings without
overriding an existing combination.
see also https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs

also remove the confusing comment

based on the feedback of reviewer,

> the phrase "only build this combination when dpdk is enable" confusing
> - the phrase "in addition to the above" says the opposite, that we did
> build this combination without dpdk but now we'll build it again with
> dpdk.

Signed-off-by: Kefu Chai <kefu.chai@scylladb.com>

Closes #2227
---
 .github/workflows/tests.yaml | 44 +++++++++++++++++++++++++++++-------
 1 file changed, 36 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
index c9d58940aeb..d5c8a0e11d5 100644
--- a/.github/workflows/tests.yaml
+++ b/.github/workflows/tests.yaml
@@ -21,19 +21,47 @@ jobs:
         standard: [20, 23]
         mode: [dev, debug, release]
         include:
-          # in addition to the above combinations, add a combination to build with
-          # dpdk. but only build this combination when dpdk is enabled, so we don't
-          # double the size of the test matrix, and can at least test the build
-          # with dpdk enabled.
+          - compiler: clang++-18
+            standard: 20
+            mode: dev
+          - compiler: clang++-18
+            standard: 20
+            mode: debug
+          - compiler: clang++-18
+            standard: 20
+            mode: release
+          - compiler: clang++-18
+            standard: 23
+            mode: dev
+          - compiler: clang++-18
+            standard: 23
+            mode: debug
+          - compiler: clang++-18
+            standard: 23
+            mode: release
+          - compiler: gcc-13
+            standard: 20
+            mode: dev
+          - compiler: gcc-13
+            standard: 20
+            mode: debug
+          - compiler: gcc-13
+            standard: 20
+            mode: release
+          - compiler: gcc-13
+            standard: 23
+            mode: dev
+          - compiler: gcc-13
+            standard: 23
+            mode: debug
+          - compiler: gcc-13
+            standard: 23
+            mode: release
           - compiler: clang++-18
             standard: 23
             mode: release
             cooks: --cook dpdk
             enables: --enable-dpdk
-          # in addition to the above combinations, add a combination to build with
-          # C++20 modules. but only build this combination with C++20 modules enabled,
-          # so we don't double the size of the test matrix, and can at least test the
-          # build with C++20 modules enabled.
           - compiler: clang++-18
             standard: 23
             mode: debug