Changeset 59402 for trunk/.github/workflows/reusable-upgrade-testing.yml

Timestamp:

11/13/2024 07:06:57 PM (7 months ago)

Author:

desrosj

Message:

Build/Test Tools: Avoid starting the database twice.

The database container is started when the services are initially set up. Having a separate step for this sometimes introduces unexpected failures for an unknown reason.

Props johnbillion.
See #62221.

File:

• trunk/.github/workflows/reusable-upgrade-testing.yml (modified) (3 diffs)

trunk/.github/workflows/reusable-upgrade-testing.yml

@@ -43 +43 @@
   # Performs the following steps:
   # - Sets up PHP.
-  # - Starts the database server.
   # - Downloads the specified version of WordPress.
   # - Creates a `wp-config.php` file.
@@ -69 +68 @@
           -c "exec docker-entrypoint.sh mysqld${{ inputs.db-type == 'mysql' && contains( fromJSON('["7.2", "7.3"]'), inputs.php ) && ' --default-authentication-plugin=mysql_native_password' || '' }}"
 
-
     steps:
       - name: Set up PHP ${{ inputs.php }}
@@ -77 +75 @@
           coverage: none
           tools: wp-cli
-
-      - name: Start the database server
-        run: |
-          sudo systemctl start ${{ inputs.db-type }}
 
       - name: Download WordPress ${{ inputs.wp }}