123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161 |
- # TODO(billy): this workflow has not been re-named from `acceptance` because
- # Visual Snapshots compares against artifacts from the same workflow name (on main branch)
- # We should rename this when we have a more finalized naming scheme.
- #
- # Also note that this name *MUST* match the filename because GHA
- # only provides the workflow name (https://docs.github.com/en/free-pro-team@latest/actions/reference/environment-variables#default-environment-variables)
- # and GH APIs only support querying by workflow *FILENAME* (https://developer.github.com/v3/actions/workflows/#get-a-workflow)
- name: acceptance
- on:
- push:
- branches:
- - master
- - releases/**
- pull_request:
- # Cancel in progress workflows on pull_requests.
- # https://docs.github.com/en/actions/using-jobs/using-concurrency#example-using-a-fallback-value
- concurrency:
- group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
- cancel-in-progress: true
- # hack for https://github.com/actions/cache/issues/810#issuecomment-1222550359
- env:
- SEGMENT_DOWNLOAD_TIMEOUT_MINS: 3
- jobs:
- files-changed:
- name: detect what files changed
- runs-on: ubuntu-20.04
- timeout-minutes: 3
- # Map a step output to a job output
- outputs:
- acceptance: ${{ steps.changes.outputs.acceptance }}
- backend_all: ${{ steps.changes.outputs.backend_all }}
- steps:
- - uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # v3.1.0
- - name: Check for backend file changes
- uses: getsentry/paths-filter@4512585405083f25c027a35db413c2b3b9006d50 # v2.11.1
- id: changes
- with:
- token: ${{ github.token }}
- filters: .github/file-filters.yml
- acceptance:
- if: needs.files-changed.outputs.acceptance == 'true'
- needs: files-changed
- name: acceptance
- runs-on: ubuntu-20.04
- timeout-minutes: 25
- strategy:
- # This helps not having to run multiple jobs because one fails, thus, reducing resource usage
- # and reducing the risk that one of many runs would turn red again (read: intermittent tests)
- fail-fast: false
- matrix:
- # XXX: When updating this, make sure you also update MATRIX_INSTANCE_TOTAL.
- instance: [0, 1, 2, 3, 4]
- pg-version: ['9.6']
- env:
- # XXX: MATRIX_INSTANCE_TOTAL must be hardcoded to the length of strategy.matrix.instance.
- MATRIX_INSTANCE_TOTAL: 5
- VISUAL_SNAPSHOT_ENABLE: 1
- TEST_GROUP_STRATEGY: roundrobin
- steps:
- - uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # v3.1.0
- name: Checkout sentry
- - uses: getsentry/action-setup-volta@54775a59c41065f54ecc76d1dd5f2cdc7a1550cb # v1.1.0
- - name: Step configurations
- id: config
- run: echo "webpack-path=.webpack_cache" >> "$GITHUB_OUTPUT"
- - name: webpack cache
- uses: actions/cache@9b0c1fce7a93df8e3bb8926b0d6e9d89e92f20a7 # v3.0.11
- with:
- path: ${{ steps.config.outputs.webpack-path }}
- key: ${{ runner.os }}-v2-webpack-cache-${{ hashFiles('webpack.config.ts') }}
- - name: node_modules cache
- uses: actions/cache@9b0c1fce7a93df8e3bb8926b0d6e9d89e92f20a7 # v3.0.11
- id: nodemodulescache
- with:
- path: node_modules
- key: ${{ runner.os }}-node-modules-${{ hashFiles('yarn.lock', 'api-docs/yarn.lock') }}
- - name: Install Javascript Dependencies
- if: steps.nodemodulescache.outputs.cache-hit != 'true'
- run: yarn install --frozen-lockfile
- - name: webpack
- env:
- WEBPACK_CACHE_PATH: ${{ steps.config.outputs.webpack-path }}
- SENTRY_INSTRUMENTATION: 1
- # this is fine to not have for forks, it shouldn't fail
- SENTRY_WEBPACK_WEBHOOK_SECRET: ${{ secrets.SENTRY_WEBPACK_WEBHOOK_SECRET }}
- run: |
- yarn build-acceptance
- - name: Build chartcuterie configuration module
- run: |
- make build-chartcuterie-config
- - name: Setup sentry env
- uses: ./.github/actions/setup-sentry
- id: setup
- with:
- snuba: true
- chartcuterie: true
- pg-version: ${{ matrix.pg-version }}
- - name: Run acceptance tests (#${{ steps.setup.outputs.matrix-instance-number }} of ${{ steps.setup.outputs.matrix-instance-total }})
- run: |
- mkdir -p ${{ steps.setup.outputs.acceptance-dir }}
- mkdir -p ${{ steps.setup.outputs.acceptance-dir }}-mobile
- mkdir -p ${{ steps.setup.outputs.acceptance-dir }}-tooltips
- make run-acceptance
- env:
- PYTEST_SNAPSHOTS_DIR: ${{ steps.setup.outputs.acceptance-dir }}
- - name: Save snapshots
- uses: getsentry/action-visual-snapshot@ea663f1be8b73c86b18081d714423ce52352e934
- with:
- save-only: true
- snapshot-path: .artifacts/visual-snapshots
- # This job runs when FE or BE changes happen, however, we only upload coverage data for
- # BE changes since it conflicts with codecov's carry forward functionality
- # Upload coverage data even if running the tests step fails since
- # it reduces large coverage fluctuations
- - name: Handle artifacts
- uses: ./.github/actions/artifacts
- if: ${{ always() && needs.files-changed.outputs.backend_all == 'true' }}
- acceptance-required-checks:
- # this is a required check so we need this job to always run and report a status.
- if: always()
- name: Acceptance
- needs: [acceptance, files-changed]
- runs-on: ubuntu-20.04
- timeout-minutes: 20
- steps:
- # If any jobs we depend on fail, we will fail since this check calls the
- # visual snapshot service which creates the "Visual Snapshot" status,
- # which is a required check.
- # NOTE: A timeout is considered a failure
- - name: Check for failures
- if: contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled')
- run: |
- echo "One of the dependent jobs have failed. You may need to re-run it." && exit 1
- - name: Diff snapshots
- uses: getsentry/action-visual-snapshot@ea663f1be8b73c86b18081d714423ce52352e934
- # Run this step only if there are acceptance related code changes
- # Forks are handled in visual-diff.yml
- if: needs.files-changed.outputs.acceptance == 'true' && github.event.pull_request.head.repo.full_name == 'getsentry/sentry'
- with:
- api-token: ${{ secrets.VISUAL_SNAPSHOT_SECRET }}
- gcs-bucket: 'sentry-visual-snapshots'
- gcp-service-account-key: ${{ secrets.SNAPSHOT_GOOGLE_SERVICE_ACCOUNT_KEY }}
|