From bc369b3f55349a7f8cbde8fe8321b230c2af1c14 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Svensson?= <bjorn.a.svensson@est.tech>
Date: Fri, 11 Oct 2024 08:52:11 +0200
Subject: [PATCH] Install the Valkey package when available in CI
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Björn Svensson <bjorn.a.svensson@est.tech>
---
 .github/workflows/build.yml |  7 +------
 .github/workflows/ci.yml    | 19 ++++++++++---------
 2 files changed, 11 insertions(+), 15 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 2e34783..c32d002 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -43,12 +43,7 @@ jobs:
           dnf -y install https://rpms.remirepo.net/enterprise/remi-release-8.rpm
           dnf -y module install redis:remi-6.0
           dnf -y group install "Development Tools"
-          dnf -y install openssl-devel cmake libevent-devel
-
-      - name: Install Valkey
-        run: |
-          git clone --depth 1 --branch 7.2.5 https://github.com/valkey-io/valkey.git
-          cd valkey && BUILD_TLS=yes make install
+          dnf -y install openssl-devel cmake libevent-devel valkey
 
       - name: Build using cmake
         env:
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 71a00c0..b2ec034 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -58,10 +58,15 @@ jobs:
       uses: jwlawson/actions-setup-cmake@802fa1a2c4e212495c05bf94dba2704a92a472be # v2.0.2
       with:
         cmake-version: ${{ matrix.cmake-version }}
-    - name: Install Valkey for non-cluster tests
+    - name: Build and install Valkey for non-cluster tests
+      if: ${{ matrix.runner == 'ubuntu-20.04' }}
       run: |
         git clone --depth 1 --branch 7.2.5 https://github.com/valkey-io/valkey.git
         cd valkey && BUILD_TLS=yes make install
+    - name: Install Valkey for non-cluster tests
+      if: ${{ matrix.runner != 'ubuntu-20.04' }}
+      run: |
+        sudo apt install valkey
     - name: Generate makefiles
       env:
         CC: ${{ matrix.compiler }}
@@ -89,7 +94,7 @@ jobs:
 
   ubuntu-make:
     name: Build with make
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     steps:
       - uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
       - name: Prepare
@@ -98,9 +103,7 @@ jobs:
           packages: libevent-dev valgrind
           version: 1.0
       - name: Install Valkey
-        run: |
-          git clone --depth 1 --branch 7.2.5 https://github.com/valkey-io/valkey.git
-          cd valkey && sudo BUILD_TLS=yes make install
+        run: sudo apt install valkey
       - name: Build
         run: USE_TLS=1 TEST_ASYNC=1 make
       - name: Run tests
@@ -119,7 +122,7 @@ jobs:
 
   ubuntu-32bit:
     name: Build for 32-bit
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     steps:
       - uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
       - name: Prepare
@@ -128,9 +131,7 @@ jobs:
           packages: gcc-multilib
           version: 1.0
       - name: Install Valkey
-        run: |
-          git clone --depth 1 --branch 7.2.5 https://github.com/valkey-io/valkey.git
-          cd valkey && sudo BUILD_TLS=yes make install
+        run: sudo apt install valkey
       - name: Build
         run: |
           make 32bit