Skip to content

chore: pin docker workflows to ASF allowlist versions#862

Merged
paleolimbot merged 4 commits intoapache:mainfrom
amoeba:chore/workflows-use-asf-allowed
Apr 6, 2026
Merged

chore: pin docker workflows to ASF allowlist versions#862
paleolimbot merged 4 commits intoapache:mainfrom
amoeba:chore/workflows-use-asf-allowed

Conversation

@amoeba
Copy link
Copy Markdown
Member

@amoeba amoeba commented Apr 5, 2026

Fixes CI jobs that are currently failing due to errors like this one,

The actions docker/setup-buildx-action@v3, docker/login-action@v3, and docker/setup-qemu-action@v3 are not allowed in apache/arrow-nanoarrow because all actions must be from a repository owned by your enterprise, created by GitHub, or match one of the patterns

Updates references to workflows that need to be pinned to ASF allowlisted refs from https://github.com/apache/infrastructure-actions/blob/main/actions.yml. All other workflows are from actions/* so those should be fine (though outdated).

Closes #860

@amoeba amoeba requested a review from paleolimbot April 5, 2026 17:25
@amoeba
Copy link
Copy Markdown
Member Author

amoeba commented Apr 5, 2026

The alpine-s390x verify job is failing on trying to install fs so I'm trying out 516421f.

@amoeba amoeba marked this pull request as draft April 5, 2026 22:11
@amoeba
Copy link
Copy Markdown
Member Author

amoeba commented Apr 6, 2026

Ah, I see the image needs to get rebuilt, I'll make another PR to add libuv-dev to the dockerfile. This is ready for review.

@amoeba amoeba marked this pull request as ready for review April 6, 2026 01:31
Copy link
Copy Markdown
Member

@paleolimbot paleolimbot left a comment

Choose a reason for hiding this comment

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

Thank you!

I forgot we have to merge this before we can rebuild the image on main (I tried: https://github.com/apache/arrow-nanoarrow/actions/runs/24015244616 ). I'll merge + manually trigger the build and then verify actions to check.

@paleolimbot paleolimbot merged commit 23fc93c into apache:main Apr 6, 2026
27 of 28 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.

Docker build and run CI that requires QEMU is failing

2 participants