Skip to content

Commit 2590393

Browse files
committed
Revert "CLOUDP-304211: Only accept pull_request_target with safe-to-test label (#2211)"
This reverts commit 194bc76.
1 parent 194bc76 commit 2590393

File tree

2 files changed

+29
-25
lines changed

2 files changed

+29
-25
lines changed

.github/workflows/test.yml

Lines changed: 29 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,28 +8,56 @@ on:
88
- 'main'
99
paths-ignore:
1010
- 'docs/**'
11+
pull_request:
12+
types: [opened, synchronize, reopened, ready_for_review, converted_to_draft, labeled, unlabeled]
13+
branches:
14+
- '**'
15+
paths-ignore:
16+
- 'docs/**'
1117
merge_group:
1218
workflow_dispatch:
13-
workflow_call:
1419

1520
concurrency:
1621
group: test-${{ github.head_ref || github.ref_name }}
1722
cancel-in-progress: true
1823

1924
jobs:
25+
run-tests:
26+
name: Run Tests
27+
runs-on: ubuntu-latest
28+
# Contributions do NOT run any testing by default, a label is needed to allow testing
29+
if: |
30+
github.event.pull_request.base.repo.full_name == github.event.pull_request.head.repo.full_name ||
31+
contains(github.event.pull_request.labels.*.name, 'safe-to-test') ||
32+
contains(github.event.pull_request.labels.*.name, 'cloud-tests') ||
33+
contains(github.event.pull_request.labels.*.name, 'retest')
34+
steps:
35+
- name: allowed message
36+
run: echo "Allowed to run tests"
37+
2038
lint:
39+
needs:
40+
- run-tests
2141
uses: ./.github/workflows/lint.yaml
2242

2343
validate-manifests:
44+
needs:
45+
- run-tests
2446
uses: ./.github/workflows/validate-manifests.yml
2547

2648
unit-tests:
49+
needs:
50+
- run-tests
2751
uses: ./.github/workflows/test-unit.yml
2852

2953
check-licenses:
54+
needs:
55+
- run-tests
3056
uses: ./.github/workflows/check-licenses.yml
3157

3258
cloud-tests-filter:
59+
needs:
60+
- run-tests
3361
uses: ./.github/workflows/cloud-tests-filter.yml
3462

3563
cloud-tests:

.github/workflows/trigger.yml

Lines changed: 0 additions & 24 deletions
This file was deleted.

0 commit comments

Comments
 (0)