Skip to content

Commit ebde321

Browse files
authored
GH-50293: [CI] Run check-labels for all triggers to avoid cancelling further steps and add tag to set_enabled (#50340)
### Rationale for this change Currently when check-labels is not a pull_request even the following steps are cancelled. Also we don't enable tags execution which is required for releases. ### What changes are included in this PR? Run check-labels for the specified events and enable jobs for tags. ### Are these changes tested? I've tested on my fork by pushing to main and creating tags, more details on the comment on the PR. ### Are there any user-facing changes? No * GitHub Issue: #50293 Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
1 parent bffe3e4 commit ebde321

4 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/cpp_extra.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ permissions:
6666

6767
jobs:
6868
check-labels:
69-
if: github.event_name == 'pull_request'
7069
uses: ./.github/workflows/check_labels.yml
7170
with:
7271
parent-workflow: cpp_extra
@@ -80,6 +79,7 @@ jobs:
8079
steps:
8180
- id: set_enabled
8281
if: >-
82+
github.ref_type == 'tag' ||
8383
(github.event_name == 'schedule' && github.repository == 'apache/arrow') ||
8484
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra') ||
8585
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra: C++')

.github/workflows/cuda_extra.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ permissions:
4949

5050
jobs:
5151
check-labels:
52-
if: github.event_name == 'pull_request'
5352
uses: ./.github/workflows/check_labels.yml
5453
with:
5554
parent-workflow: cuda_extra
@@ -63,6 +62,7 @@ jobs:
6362
steps:
6463
- id: set_enabled
6564
if: >-
65+
github.ref_type == 'tag' ||
6666
(github.event_name == 'schedule' && github.repository == 'apache/arrow') ||
6767
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra') ||
6868
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra: CUDA')

.github/workflows/package_linux.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ permissions:
5858

5959
jobs:
6060
check-labels:
61-
if: github.event_name == 'pull_request'
6261
uses: ./.github/workflows/check_labels.yml
6362
with:
6463
parent-workflow: package_linux
@@ -72,6 +71,7 @@ jobs:
7271
steps:
7372
- id: set_enabled
7473
if: >-
74+
github.ref_type == 'tag' ||
7575
(github.event_name == 'schedule' && github.repository == 'apache/arrow') ||
7676
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra') ||
7777
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra: Package: Linux')

.github/workflows/r_extra.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@ permissions:
6161

6262
jobs:
6363
check-labels:
64-
if: github.event_name == 'pull_request'
6564
uses: ./.github/workflows/check_labels.yml
6665
with:
6766
parent-workflow: r_extra
@@ -75,6 +74,7 @@ jobs:
7574
steps:
7675
- id: set_enabled
7776
if: >-
77+
github.ref_type == 'tag' ||
7878
(github.event_name == 'schedule' && github.repository == 'apache/arrow') ||
7979
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra') ||
8080
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra: R')

0 commit comments

Comments
 (0)