Skip to content

GH-49725: [CI] Use environment variables instead of template expressions in workflow run blocks#49733

Merged
kou merged 1 commit into
apache:mainfrom
thisisnic:GH-49725-env_vars
Apr 15, 2026
Merged

GH-49725: [CI] Use environment variables instead of template expressions in workflow run blocks#49733
kou merged 1 commit into
apache:mainfrom
thisisnic:GH-49725-env_vars

Conversation

@thisisnic

@thisisnic thisisnic commented Apr 14, 2026

Copy link
Copy Markdown
Member

Rationale for this change

Update CI jobs to be more robust

What changes are included in this PR?

Update CI jobs to use env vars instead of template expressions

Are these changes tested?

CI should run

Are there any user-facing changes?

No

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Apr 14, 2026
@thisisnic

Copy link
Copy Markdown
Member Author

I believe that these failures are expected because there's no active RC staged right now

@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

@kou
kou merged commit d2c2f00 into apache:main Apr 15, 2026
37 of 48 checks passed
@kou kou removed the awaiting committer review Awaiting committer review label Apr 15, 2026
@github-actions github-actions Bot added the awaiting merge Awaiting merge label Apr 15, 2026
@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit d2c2f00.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Mottl pushed a commit to Mottl/arrow that referenced this pull request May 26, 2026
…pressions in workflow run blocks (apache#49733)

### Rationale for this change

Update CI jobs to be more robust

### What changes are included in this PR?

Update CI jobs to use env vars instead of template expressions

### Are these changes tested?

CI should run

### Are there any user-facing changes?

No
* GitHub Issue: apache#49725

Authored-by: Nic Crane <thisisnic@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants