Make WordPress Core


Ignore:
Timestamp:
06/30/2022 04:34:14 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.

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

Location:
branches/5.7
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • branches/5.7

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

    r51935 r53598  
    44  push:
    55    branches:
    6       - master
     6      - trunk
    77      - '3.[7-9]'
    88      - '[4-9].[0-9]'
     
    1212  pull_request:
    1313    branches:
    14       - master
     14      - trunk
    1515      - '3.[7-9]'
    1616      - '[4-9].[0-9]'
     
    2020    - cron: '0 0 * * 0'
    2121
     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
    2229env:
    2330  PUPPETEER_SKIP_CHROMIUM_DOWNLOAD: ${{ true }}
     
    2835
    2936jobs:
    30   # Sets up the workflow for testing.
    31   #
    32   # Performs the following steps:
    33   # - Cancels all previous workflow runs for pull requests that have not completed.
    34   setup-workflow:
    35     name: Setup Workflow
    36     runs-on: ubuntu-latest
    37     if: ${{ github.repository == 'WordPress/wordpress-develop' || github.event_name == 'pull_request' }}
    38 
    39     steps:
    40       - name: Cancel previous runs of this workflow (pull requests only)
    41         if: ${{ github.event_name == 'pull_request' }}
    42         uses: styfle/cancel-workflow-action@3d86a7cc43670094ac248017207be0295edbc31d # v0.8.0
    43 
    4437  # Runs the PHPUnit tests for WordPress.
    4538  #
    4639  # Performs the following steps:
    47   # - Set environment variables.
     40  # - Sets environment variables.
    4841  # - Sets up the environment variables needed for testing with memcached (if desired).
    49   # - Installs NodeJS 14.
    50   # - Sets up caching for NPM.
     42  # - Installs NodeJS.
    5143  # - Installs NPM dependencies
    5244  # - Configures caching for Composer.
    53   # - Installs Composer dependencies (if desired).
    54   # - Logs Docker debug information (about both the Docker installation within the runner).
     45  # - Installs Composer dependencies.
     46  # - Logs Docker debug information (about the Docker installation within the runner).
    5547  # - Starts the WordPress Docker container.
    56   # - Starts the memcached server after the Docker network has been created (if desired).
    57   # - Logs WordPress Docker container debug information.
    58   # - Logs debug general information.
     48  # - Starts the Memcached server after the Docker network has been created (if desired).
     49  # - Logs general debug information about the runner.
    5950  # - Logs the running Docker containers.
     51  # - Logs debug information from inside the WordPress Docker container.
    6052  # - Logs debug information about what's installed within the WordPress Docker containers.
    6153  # - Install WordPress within the Docker container.
     
    6456  # - Reconnect the directory to the Git repository.
    6557  # - Submit the test results to the WordPress.org host test results.
    66   # - todo: Configure Slack notifications for failing tests.
    6758  test-php:
    68     name: ${{ matrix.php }}${{ matrix.multisite && ' multisite' || '' }}${{ matrix.split_slow && ' slow tests' || '' }} ${{ matrix.memcached && ' with memcached' || '' }} on ${{ matrix.os }}
     59    name: ${{ matrix.php }}${{ matrix.multisite && ' multisite' || '' }}${{ matrix.split_slow && ' slow tests' || '' }}${{ matrix.memcached && ' with memcached' || '' }} on ${{ matrix.os }}
    6960    runs-on: ${{ matrix.os }}
     61    timeout-minutes: 20
    7062    if: ${{ github.repository == 'WordPress/wordpress-develop' || github.event_name == 'pull_request' }}
    7163    strategy:
     
    116108
    117109      - name: Checkout repository
    118         uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # v2.3.4
     110        uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # v3.0.2
    119111
    120112      - name: Install NodeJS
    121         uses: actions/setup-node@46071b5c7a2e0c34e49c3cb8a0e792e86e18d5ea # v2.1.5
     113        uses: actions/setup-node@eeb10cff27034e7acf239c5d29f62154018672fd # v3.3.0
    122114        with:
    123           node-version: 14
    124 
    125       - name: Use cached Node modules
    126         uses: actions/cache@26968a09c0ea4f3e233fdddbafd1166051a095f6 # v2.1.4
    127         env:
    128           cache-name: cache-node-modules
    129         with:
    130           # npm cache files are stored in `~/.npm` on Linux/macOS
    131           path: ~/.npm
    132           key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }}
     115          node-version-file: '.nvmrc'
     116          cache: npm
    133117
    134118      - name: Install Dependencies
    135         run: npx install-changed --install-command="npm ci"
     119        run: npm ci
    136120
    137121      - name: Get composer cache directory
     
    140124
    141125      - name: Cache Composer dependencies
    142         uses: actions/cache@26968a09c0ea4f3e233fdddbafd1166051a095f6 # v2.1.4
     126        uses: actions/cache@c3f1317a9e7b1ef106c153ac8c0f00fed3ddbc0d # v3.0.4
    143127        env:
    144128          cache-name: cache-composer-dependencies
     
    239223
    240224      - name: Checkout the WordPress Test Reporter
    241         if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
    242         uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # v2.3.4
     225        if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/trunk' && matrix.report }}
     226        uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # v3.0.2
    243227        with:
    244228          repository: 'WordPress/phpunit-test-runner'
     
    246230
    247231      - name: Submit test results to the WordPress.org host test results
    248         if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
     232        if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/trunk' && matrix.report }}
    249233        env:
    250234          WPT_REPORT_API_KEY: "${{ secrets.WPT_REPORT_API_KEY }}"
    251235        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
     236
     237  slack-notifications:
     238    name: Slack Notifications
     239    uses: WordPress/wordpress-develop/.github/workflows/slack-notifications.yml@trunk
     240    needs: [ test-php ]
     241    if: ${{ github.repository == 'WordPress/wordpress-develop' && github.event_name != 'pull_request' && always() }}
     242    with:
     243      calling_status: ${{ needs.test-php.result == 'success' && 'success' || needs.test-php.result == 'cancelled' && 'cancelled' || 'failure' }}
     244    secrets:
     245      SLACK_GHA_SUCCESS_WEBHOOK: ${{ secrets.SLACK_GHA_SUCCESS_WEBHOOK }}
     246      SLACK_GHA_CANCELLED_WEBHOOK: ${{ secrets.SLACK_GHA_CANCELLED_WEBHOOK }}
     247      SLACK_GHA_FIXED_WEBHOOK: ${{ secrets.SLACK_GHA_FIXED_WEBHOOK }}
     248      SLACK_GHA_FAILURE_WEBHOOK: ${{ secrets.SLACK_GHA_FAILURE_WEBHOOK }}
Note: See TracChangeset for help on using the changeset viewer.