WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
06/30/2022 05:28:11 PM (2 years ago)
Author:
desrosj
Message:

Build/Test Tools: Use the Slack notifications workflow as a reusable one.

This backports several changesets that are required to remove the reliance on the workflow_run event for posting Slack notifications.

The Slack notification workflow will now be called as a reusable one, which has several benefits (see [53591]).

Several other minor GitHub Actions related updates are also being backported in this commit to maintain tooling consistency across branches that still receive security updates as a courtesy when necessary.

Workflows that are not relevant to this branch that were mistakenly backported are also being deleted.

Merges [50473], [50704], [50930], [51341], [51355], [51498], [51511], [51535], [51924], [51925], [51937], [52002], [52130], [52183], [52233], [53112], [53581], [53582], [53592] to the 4.1 branch.
See #56095.

Location:
branches/4.1
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • branches/4.1

  • branches/4.1/.github/workflows/phpunit-tests.yml

    r50643 r53620  
    44  push:
    55    branches:
    6       - master
    76      - trunk
    87      - '3.[7-9]'
    … …  
    1312  pull_request:
    1413    branches:
    15       - master
    1614      - trunk
    1715      - '3.[7-9]'
    … …  
    2220    - cron: '0 0 * * 0'
    2321
     22# Cancels all previous workflow runs for pull requests that have not completed.
     23concurrency:
     24    # The concurrency group contains the workflow name and the branch name for pull requests
     25    # or the commit hash for any other events.
     26  group: ${{ github.workflow }}-${{ github.event_name == 'pull_request' && github.head_ref || github.sha }}
     27  cancel-in-progress: true
     28
    2429env:
    2530  LOCAL_DIR: build
    … …  
    3641  # - Cancels all previous workflow runs for pull requests that have not completed.
    3742  # - Checks out the repository.
    38   # - Logs debug information about the runner container.
    39   # - Installs NodeJS 14.
    40   # - Sets up caching for NPM.
    41   # _ Installs NPM dependencies using install-changed to hash the `package.json` file.
     43  # - Logs debug information about the GitHub Action runner.
     44  # - Installs NodeJS.
     45  # _ Installs NPM dependencies.
    4246  # - Builds WordPress to run from the `build` directory.
    4347  # - Creates a ZIP file of compiled WordPress.
    … …  
    4953
    5054    steps:
    51       - name: Cancel previous runs of this workflow (pull requests only)
    52         if: ${{ github.event_name == 'pull_request' }}
    53         uses: styfle/cancel-workflow-action@3d86a7cc43670094ac248017207be0295edbc31d # v0.8.0
    54 
    5555      - name: Checkout repository
    56         uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # v2.3.4
     56        uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # v3.0.2
    5757
    5858      - name: Log debug information
    … …  
    7070
    7171      - name: Install NodeJS
    72         uses: actions/setup-node@46071b5c7a2e0c34e49c3cb8a0e792e86e18d5ea # v2.1.5
    73         with:
    74           node-version: 14
    75 
    76       - name: Cache NodeJS modules
    77         uses: actions/cache@26968a09c0ea4f3e233fdddbafd1166051a095f6 # v2.1.4
    78         env:
    79           cache-name: cache-node-modules
    80         with:
    81           # npm cache files are stored in `~/.npm` on Linux/macOS
    82           path: ~/.npm
    83           key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }}
     72        uses: actions/setup-node@eeb10cff27034e7acf239c5d29f62154018672fd # v3.3.0
     73        with:
     74          node-version-file: '.nvmrc'
     75          cache: npm
    8476
    8577      - name: Install Dependencies
    … …  
    9082
    9183      - name: Create ZIP artifact
    92         uses: thedoctor0/zip-release@a1afcab9c664c9976ac398fa831eac67bed1eb0e # v0.4.1
     84        uses: thedoctor0/zip-release@09336613be18a8208dfa66bd57efafd9e2685657 # v0.6.2
    9385        with:
    9486          filename: built-wp-${{ github.sha }}.zip
    … …  
    9688
    9789      - name: Upload build artifact
    98         uses: actions/upload-artifact@e448a9b857ee2131e752b06002bf0e093c65e571 # v2.2.2
     90        uses: actions/upload-artifact@3cea5372237819ed00197afe530f5a7ea3e805c8 # v3.1.0
    9991        with:
    10092          name: built-wp-${{ github.sha }}
    … …  
    10597  #
    10698  # Performs the following steps:
    107   # - Set environment variables.
     99  # - Sets environment variables.
    108100  # - Sets up the environment variables needed for testing with memcached (if desired).
    109101  # - Downloads the built WordPress artifact from the previous job.
    110102  # - Unzips the artifact.
    111   # - Installs NodeJS 14.
    112   # - Sets up caching for NPM.
    113   # _ Installs NPM dependencies using install-changed to hash the `package.json` file.
     103  # - Installs NodeJS.
     104  # _ Installs NPM dependencies.
    114105  # - Configures caching for Composer.
    115106  # _ Installs Composer dependencies (if desired).
    116   # - Logs Docker debug information (about both the Docker installation within the runner).
     107  # - Logs Docker debug information (about the Docker installation within the runner).
    117108  # - Starts the WordPress Docker container.
    118   # - Starts the memcached server after the Docker network has been created (if desired).
    119   # - Logs WordPress Docker container debug information.
    120   # - Logs debug general information.
     109  # - Starts the Memcached server after the Docker network has been created (if desired).
     110  # - Logs general debug information about the runner.
    121111  # - Logs the running Docker containers.
     112  # - Logs debug information from inside the WordPress Docker container.
    122113  # - Logs debug information about what's installed within the WordPress Docker containers.
    123114  # - Install WordPress within the Docker container.
    … …  
    126117  # - Reconnect the directory to the Git repository.
    127118  # - Submit the test results to the WordPress.org host test results.
    128   # - todo: Configure Slack notifications for failing tests.
    129119  test-php:
    130     name: ${{ matrix.php }}${{ matrix.multisite && ' multisite' || '' }}${{ matrix.split_slow && ' slow tests' || '' }} ${{ matrix.memcached && ' with memcached' || '' }} on ${{ matrix.os }}
     120    name: ${{ matrix.php }}${{ matrix.multisite && ' multisite' || '' }}${{ matrix.split_slow && ' slow tests' || '' }}${{ matrix.memcached && ' with memcached' || '' }} on ${{ matrix.os }}
    131121    needs: setup-wordpress
    132122    runs-on: ${{ matrix.os }}
     123    timeout-minutes: 20
    133124    strategy:
    134125      fail-fast: false
    … …  
    178169
    179170      - name: Download the built WordPress artifact
    180         uses: actions/download-artifact@4a7a711286f30c025902c28b541c10e147a9b843 # v2.0.8
     171        uses: actions/download-artifact@fb598a63ae348fa914e94cd0ff38f362e927b741 # v3.0.0
    181172        with:
    182173          name: built-wp-${{ github.sha }}
    … …  
    186177
    187178      - name: Install NodeJS
    188         uses: actions/setup-node@46071b5c7a2e0c34e49c3cb8a0e792e86e18d5ea # v2.1.5
    189         with:
    190           node-version: 14
    191 
    192       - name: Use cached Node modules
    193         uses: actions/cache@26968a09c0ea4f3e233fdddbafd1166051a095f6 # v2.1.4
    194         env:
    195           cache-name: cache-node-modules
    196         with:
    197           # npm cache files are stored in `~/.npm` on Linux/macOS
    198           path: ~/.npm
    199           key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }}
     179        uses: actions/setup-node@eeb10cff27034e7acf239c5d29f62154018672fd # v3.3.0
     180        with:
     181          node-version-file: '.nvmrc'
     182          cache: npm
    200183
    201184      - name: Install Dependencies
    … …  
    204187      - name: Cache Composer dependencies
    205188        if: ${{ env.COMPOSER_INSTALL == true }}
    206         uses: actions/cache@26968a09c0ea4f3e233fdddbafd1166051a095f6 # v2.1.4
     189        uses: actions/cache@c3f1317a9e7b1ef106c153ac8c0f00fed3ddbc0d # v3.0.4
    207190        env:
    208191          cache-name: cache-composer-dependencies
    … …  
    280263
    281264      - name: Checkout the WordPress Test Reporter
    282         if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
    283         uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # v2.3.4
     265        if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/trunk' && matrix.report }}
     266        uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # v3.0.2
    284267        with:
    285268          repository: 'WordPress/phpunit-test-runner'
    … …  
    287270
    288271      - name: Set up the Git repository
    289         if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
     272        if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/trunk' && matrix.report }}
    290273        run: |
    291274          git init
    292275          git remote add origin https://github.com/WordPress/wordpress-develop.git
    293276          git fetch
    294           git reset origin/master
     277          git reset origin/trunk
    295278
    296279      - name: Submit test results to the WordPress.org host test results
    297         if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
     280        if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/trunk' && matrix.report }}
    298281        env:
    299282          WPT_REPORT_API_KEY: "${{ secrets.WPT_REPORT_API_KEY }}"
    300283        run: docker-compose run --rm -e WPT_REPORT_API_KEY -e WPT_PREPARE_DIR=/var/www -e WPT_TEST_DIR=/var/www php php test-runner/report.php
     284
     285  slack-notifications:
     286    name: Slack Notifications
     287    uses: WordPress/wordpress-develop/.github/workflows/slack-notifications.yml@trunk
     288    needs: [ setup-wordpress, test-php ]
     289    if: ${{ github.repository == 'WordPress/wordpress-develop' && github.event_name != 'pull_request' && always() }}
     290    with:
     291      calling_status: ${{ needs.test-php.result == 'success' && 'success' || needs.test-php.result == 'cancelled' && 'cancelled' || 'failure' }}
     292    secrets:
     293      SLACK_GHA_SUCCESS_WEBHOOK: ${{ secrets.SLACK_GHA_SUCCESS_WEBHOOK }}
     294      SLACK_GHA_CANCELLED_WEBHOOK: ${{ secrets.SLACK_GHA_CANCELLED_WEBHOOK }}
     295      SLACK_GHA_FIXED_WEBHOOK: ${{ secrets.SLACK_GHA_FIXED_WEBHOOK }}
     296      SLACK_GHA_FAILURE_WEBHOOK: ${{ secrets.SLACK_GHA_FAILURE_WEBHOOK }}
Note: See TracChangeset for help on using the changeset viewer.