From 577ed8d45d389d1a887980cea9323e9bb1a54ec1 Mon Sep 17 00:00:00 2001
From: Andrey Sidorov <andrey.sidorov@gmail.com>
Date: Wed, 1 Feb 2023 18:17:32 +1100
Subject: [PATCH] ci: only start mysql docker if matrix.mysql-version is
 defined

---
 .github/workflows/ci-linux.yml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/ci-linux.yml b/.github/workflows/ci-linux.yml
index d6dc17c917..f1760fda90 100644
--- a/.github/workflows/ci-linux.yml
+++ b/.github/workflows/ci-linux.yml
@@ -70,7 +70,7 @@ jobs:
       - uses: actions/checkout@v3
 
       - name: Set up MySQL
-        if: ${{ !startsWith(env.MYSQL_CONNECTION_URL, 'mysql://') }}
+        if: ${{ matrix.mysql-version }}
         run: docker run -d -e MYSQL_ALLOW_EMPTY_PASSWORD=1 -e MYSQL_DATABASE=${{ env.MYSQL_DATABASE }} -v $PWD/mysqldata:/var/lib/mysql/ -v $PWD/examples/custom-conf:/etc/mysql/conf.d -v $PWD/examples/ssl/certs:/certs -p ${{ env.MYSQL_PORT }}:3306 ${{ matrix.mysql-version }}
 
       - name: Set up Node.js ${{ matrix.node-version }}
@@ -89,13 +89,13 @@ jobs:
         run: npm ci
 
       - name: Wait mysql server is ready
-        if: ${{ !startsWith(env.MYSQL_CONNECTION_URL, 'mysql://') }}
+        if: ${{ matrix.mysql-version }}
         run: node tools/wait-up.js
 
       - name: Run tests
         run: FILTER=${{matrix.filter}} MYSQL_USE_TLS=${{ matrix.use-tls }} MYSQL_USE_COMPRESSION=${{ matrix.use-compression }} npm run coverage-test
 
-      - run: echo "coverage-artifact-name=`echo -n "${{github.run_id}}-${{ matrix.node-version }}-${{ matrix.mysql-version }}-${{matrix.use-tls}}-${{matrix.use-compression}}" | shasum | cut -d " " -f 1`"  >> $GITHUB_ENV   
+      - run: echo "coverage-artifact-name=`echo -n "${{github.run_id}}-${{ matrix.node-version }}-${{ matrix.mysql-version }}-{${{ matrix.mysql_connection_url_key }}-${{matrix.use-tls}}-${{matrix.use-compression}}" | shasum | cut -d " " -f 1`"  >> $GITHUB_ENV   
       - uses: actions/upload-artifact@v3
         with:
           name: coverage-${{env.coverage-artifact-name}}