Skip to content

Commit

Permalink
ci: switch qemu-user-static setup to fix setuid
Browse files Browse the repository at this point in the history
Problem: flux-sched side ran into issues with setuid binaries in some
cases under aarch64 emulation building containers.  It seems to depend
on which workers pick up the job, so seems best to bring the solution
over.

Solution: Run the docker container with the setup script directly with
the appropriate option to handle setuid credentials.
  • Loading branch information
trws committed Sep 23, 2023
1 parent c349c94 commit 5b0862c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -154,9 +154,9 @@ jobs:
uses: docker/setup-buildx-action@v2
if: matrix.needs_buildx

- uses: dbhi/qus/action@main
with:
targets: aarch64
- name: setup qemu-user-static
run: |
docker run --rm --privileged aptman/qus -s -- -p --credential aarch64
- name: docker-run-checks
env: ${{matrix.env}}
Expand Down

0 comments on commit 5b0862c

Please sign in to comment.