Skip to content

Commit

Permalink
Revert "perf.: run integration tests only on approved PRs (#2609)" (#…
Browse files Browse the repository at this point in the history
…2612)

This reverts commit a29182d.

Signed-off-by: Spencer Schrock <sschrock@google.com>

Signed-off-by: Spencer Schrock <sschrock@google.com>
  • Loading branch information
spencerschrock authored Jan 23, 2023
1 parent f25d010 commit 9385905
Showing 1 changed file with 8 additions and 11 deletions.
19 changes: 8 additions & 11 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,17 +14,14 @@

# Run secret-dependent integration tests only after approval
name: Integration tests
on:
pull_request_review:
types: [submitted]
on: pull_request_target

permissions:
contents: read
pull-requests: write

jobs:
approve:
if: github.event.review.state == 'approved'
runs-on: ubuntu-latest
steps:
- name: Harden Runner
Expand Down Expand Up @@ -53,14 +50,14 @@ jobs:
- name: setup-go
uses: actions/setup-go@6edd4406fa81c3da01a34fa6f6343087c207a568 # v2.2.0
with:
go-version: "1.19"
go-version: '1.19'
check-latest: true

- name: Prepare test env
run: |
go mod download
go mod download
- name: Run GITHUB_TOKEN E2E #using retry because the GitHub token is being throttled.
- name: Run GITHUB_TOKEN E2E #using retry because the GitHub token is being throttled.
uses: nick-invision/retry@943e742917ac94714d2f408a0e8320f2d1fcafcd
env:
GITHUB_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Expand All @@ -71,7 +68,7 @@ jobs:
timeout_minutes: 30
command: make e2e-gh-token

- name: Run PAT E2E #using retry because the GitHub token is being throttled.
- name: Run PAT E2E #using retry because the GitHub token is being throttled.
uses: nick-invision/retry@943e742917ac94714d2f408a0e8320f2d1fcafcd
env:
GITHUB_AUTH_TOKEN: ${{ secrets.GH_AUTH_TOKEN }}
Expand All @@ -84,15 +81,15 @@ jobs:
- name: codecov
uses: codecov/codecov-action@81cd2dc8148241f03f5839d295e000b8f761e378 # 2.1.0
with:
files: ./e2e-coverage.out
verbose: true
files: ./e2e-coverage.out
verbose: true

- name: find comment
uses: peter-evans/find-comment@f4499a714d59013c74a08789b48abe4b704364a0 # v2.1.0
id: fc
with:
issue-number: ${{ github.event.pull_request.number || github.event.client_payload.pull_request.number }}
comment-author: "github-actions[bot]"
comment-author: 'github-actions[bot]'
body-includes: Integration tests ran for

- name: create or update comment
Expand Down

0 comments on commit 9385905

Please sign in to comment.