Skip to content

Use ASF-allowlisted dtolnay/rust-toolchain revision in setup-native-build - #4851

Merged
StevenLuMT merged 1 commit into
apache:masterfrom
StevenLuMT:use-approved-rust-toolchain-action20260729
Jul 29, 2026
Merged

Use ASF-allowlisted dtolnay/rust-toolchain revision in setup-native-build#4851
StevenLuMT merged 1 commit into
apache:masterfrom
StevenLuMT:use-approved-rust-toolchain-action20260729

Conversation

@StevenLuMT

@StevenLuMT StevenLuMT commented Jul 29, 2026

Copy link
Copy Markdown
Member

Motivation

Last update PR #4828

CI fails on all workflows that use the setup-native-build composite action (bk-ci.yml, codeql.yml, java21-daily-build.yml, windows-daily-build.yml, bk-streamstorage-python.yml, website-pr-validation.yml) with:

https://github.com/apache/bookkeeper/actions/runs/30416112341/job/90462887407?pr=4850

Error: The action dtolnay/rust-toolchain@4be7066ada62dd38de10e7b70166bc74ed198c30 is not allowed in apache/bookkeeper because all actions must be from a repository owned by your enterprise, created by GitHub, or match one of the patterns: 1Password/load-secrets-action/configure@3a12b0ab99d9cd590a3e9b5a90ea017210ed9556, 1Password/load-secrets-action/configure@92467eb28f72e8255933372f1e0707c567ce2259, 1Password/load-secrets-action/configure@eb2efd0703da22a93c467f2d1ffbb6826c11e19c, 1Password/load-secrets-action@3a12b0ab99d9cd590a3e9b5a90ea017210ed9556, 1Password/load-secrets-action@92467eb28f72e8255933372f1e0707c567ce2259, 1Password/load-secrets-action@eb2efd0703da22a93c467f2d1ffbb6826c11e19c, AdoptOpenJDK/install-jdk@*, BobAnkh/auto-generate-changelog@*, DavidAnson/markdownlint-cli2-action@6bf21b07787794f89a243495939cd651942aeabe, DavidAnson/markdownlint-cli2-action@8de2aa07cae85fd17c0b35642db70cf5495f1d25, DavidAnson/markdownlint-cli2-action@ce4853d43830c74c1753b39f3cf40f71c2031eb9, DavidA...

The ASF approved actions allowlist (apache/infrastructure-actions) only permits dtolnay/rust-toolchain@4cda84d5c5c54efe2404f9d843567869ab1699d4 (tag: stable).

The allowlisted revision is also newer (2026-07-19) than the currently pinned one (2026-06-30); both point at the stable toolchain, so this is a plain upgrade.

Changes

  • Pin dtolnay/rust-toolchain in .github/actions/setup-native-build/action.yml to the ASF-allowlisted revision 4cda84d5c5c54efe2404f9d843567869ab1699d4.

@StevenLuMT
StevenLuMT requested review from Copilot, eolivelli, hezhangjian, lhotari and zymap and removed request for lhotari July 29, 2026 06:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Updates the setup-native-build composite action to use the ASF-allowlisted dtolnay/rust-toolchain revision so CI workflows stop failing due to GitHub Actions allowlist enforcement.

Changes:

  • Bumps dtolnay/rust-toolchain pinned SHA in the setup-native-build composite action to the ASF-allowlisted revision.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@StevenLuMT
StevenLuMT merged commit 5b0cf0d into apache:master Jul 29, 2026
20 checks passed
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.

4 participants