Refactor CI workflows to reorder checkout step and update Alpine inst…

…allation to include Redis Signed-off-by: avifenesh <[email protected]>
valkey-io · Dec 26, 2024 · 32f06cb · 32f06cb
1 parent 9f034f1
commit 32f06cb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/.github/workflows/java-cd.yml b/.github/workflows/java-cd.yml
@@ -225,13 +225,13 @@ jobs:
                 host: ${{ fromJson(needs.load-platform-matrix.outputs.PLATFORM_MATRIX) }}
         runs-on: ${{ matrix.host.RUNNER }}
         steps:
-            - name: Checkout
-              uses: actions/checkout@v4
-
             - name: Setup self-hosted runner access
               if: ${{matrix.host.TARGET == 'aarch64-unknown-linux-gnu' }}
               run: sudo chown -R $USER:$USER /home/ubuntu/action-runner-ilia/_work/valkey-glide
 
+            - name: Checkout
+              uses: actions/checkout@v4
+
             - name: Set up JDK
               uses: actions/setup-java@v4
               with:

diff --git a/.github/workflows/npm-cd.yml b/.github/workflows/npm-cd.yml
@@ -309,11 +309,11 @@ jobs:
               run: sudo chown -R $USER:$USER /home/ubuntu/action-runner-ilia/_work/valkey-glide
 
 
-            - name: install and git for alpine
+            - name: install redis and git for alpine
               if: ${{ contains(matrix.build.TARGET, 'musl') }}
               run: |
                   apk update
-                  apk add git
+                  apk add git redis
                   node -v
 
             - name: install Python for ubuntu
@@ -336,6 +336,7 @@ jobs:
                   arch: ${{ matrix.build.ARCH }}
 
             - name: Install engine
+              if: ${{ !contains(matrix.build.TARGET, 'musl') }}
               uses: ./.github/workflows/install-engine
               with:
                   engine-version: "8.0"