Skip to content

GH-50293: [CI] Run check-labels for all triggers to avoid cancelling further steps and add tag to set_enabled#50340

Merged
kou merged 2 commits into
apache:mainfrom
raulcd:GH-50293
Jul 2, 2026
Merged

GH-50293: [CI] Run check-labels for all triggers to avoid cancelling further steps and add tag to set_enabled#50340
kou merged 2 commits into
apache:mainfrom
raulcd:GH-50293

Conversation

@raulcd

@raulcd raulcd commented Jul 2, 2026

Copy link
Copy Markdown
Member

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

Copilot AI review requested due to automatic review settings July 2, 2026 15:51
@raulcd raulcd marked this pull request as draft July 2, 2026 15:51
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

⚠️ GitHub issue #50293 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Jul 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates several “extra” CI workflows to enable their runs for tag events and to adjust gating around the check-labels dependency.

Changes:

  • Add github.ref_type == 'tag' as an enabling condition for the “extra” workflows.
  • Add a job-level if: guard on each workflow’s check-enabled job.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/r_extra.yml Extends enablement logic to tags; adds job-level gating for check-enabled.
.github/workflows/package_linux.yml Extends enablement logic to tags; adds job-level gating for check-enabled.
.github/workflows/cuda_extra.yml Extends enablement logic to tags; adds job-level gating for check-enabled.
.github/workflows/cpp_extra.yml Extends enablement logic to tags; adds job-level gating for check-enabled.

Comment thread .github/workflows/r_extra.yml Outdated
Comment thread .github/workflows/package_linux.yml Outdated
Comment thread .github/workflows/cuda_extra.yml Outdated
Comment thread .github/workflows/cpp_extra.yml Outdated
Comment thread .github/workflows/r_extra.yml
@raulcd

raulcd commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

I've tested a couple of things on my fork. I've merged the commits on this branches individually and have created a tag on my fork to trigger the jobs.
With the first commit the job was still skipped due to the cancellation propagation on check-labels.
https://github.com/raulcd/arrow/actions/runs/28603697459
With the second commit the jobs were run:
https://github.com/raulcd/arrow/actions/runs/28604924082

@raulcd raulcd changed the title GH-50293: [CI] Run extra workflows in the case of tags even if check-labels is cancelled GH-50293: [CI] Run check-labels for all triggers to avoid cancelling further steps and add tag to set_enabled Jul 2, 2026
@raulcd raulcd marked this pull request as ready for review July 2, 2026 16:28
Copilot AI review requested due to automatic review settings July 2, 2026 16:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@kou kou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Jul 2, 2026
@kou kou merged commit ebde321 into apache:main Jul 2, 2026
33 checks passed
@kou kou removed the awaiting merge Awaiting merge label Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants