Skip to content

ci: replace disallowed third-party actions in build workflow#16

Merged
andygrove merged 1 commit into
apache:mainfrom
andygrove:ci-fix-disallowed-actions
May 13, 2026
Merged

ci: replace disallowed third-party actions in build workflow#16
andygrove merged 1 commit into
apache:mainfrom
andygrove:ci-fix-disallowed-actions

Conversation

@andygrove
Copy link
Copy Markdown
Member

Which issue does this PR close?

Follow-up to #15.

Rationale for this change

The Build workflow added in #15 fails immediately on every run with startup_failure:

dtolnay/rust-toolchain@stable is not allowed in apache/datafusion-java because all actions must be from a repository owned by […]

ASF infra restricts GitHub Actions to a curated allowlist (mostly actions/* plus a small set of approved third parties). Both dtolnay/rust-toolchain and Swatinem/rust-cache fall outside that list, so the workflow aborts before any step runs and main currently has no working CI.

What changes are included in this PR?

  • Drop dtolnay/rust-toolchain@stable. ubuntu-latest runners already ship with rustup and a stable toolchain, so a plain run step (rustup update stable && rustup default stable) is sufficient.
  • Replace Swatinem/rust-cache@v2 with actions/cache@v4, keyed on native/Cargo.lock and caching ~/.cargo/registry, ~/.cargo/git, and native/target.

How are these changes tested?

Verified that the previous run on main (commit 058840a) failed with startup_failure due to the disallowed action. CI on this PR will exercise the replacement steps end-to-end before merge.

ASF infra restricts GitHub Actions to a curated allowlist. The Build
workflow added in apache#15 used dtolnay/rust-toolchain and Swatinem/rust-cache,
both of which fail the policy check and cause the workflow to abort with
a startup_failure before any step runs.

Drop the rust-toolchain action — ubuntu-latest runners already ship with
rustup, so a plain run step that refreshes and defaults to stable
covers the same ground. Swap the cargo cache to actions/cache@v4 keyed
on native/Cargo.lock and pointing at the cargo registry, git, and the
native target directory.
@andygrove andygrove merged commit 1da3233 into apache:main May 13, 2026
1 check passed
@andygrove andygrove deleted the ci-fix-disallowed-actions branch May 13, 2026 03:43
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.

1 participant