Skip to content

Commit

Permalink
chore(ci): Pass verification arguments from manual dispatch to docker…
Browse files Browse the repository at this point in the history
… compose verify (#239)
  • Loading branch information
paleolimbot authored Jun 16, 2023
1 parent ab24d42 commit a682731
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .env
Original file line number Diff line number Diff line change
Expand Up @@ -27,3 +27,6 @@ NANOARROW_ARCH=amd64

# Source directory (so that the docker compose app can be used on another checkout)
NANOARROW_SOURCE_DIR="."

# By default, verify HEAD
NANOARROW_VERIFY_ARGS=""
1 change: 1 addition & 0 deletions .github/workflows/verify.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,7 @@ jobs:
NANOARROW_ARCH: ${{ matrix.config.arch }}
NANOARROW_PLATFORM: ${{ matrix.config.platform }}
DOCKER_DEFAULT_PLATFORM: linux/${{ matrix.config.arch }}
NANOARROW_VERIFY_ARGS: ${{ inputs.version }} ${{ inputs.rc }}

run: |
cd src
Expand Down
2 changes: 1 addition & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ services:
NANOARROW_ARCH: ${NANOARROW_ARCH}
volumes:
- ${NANOARROW_SOURCE_DIR}:/nanoarrow
command: "/bin/bash /nanoarrow/dev/release/verify-release-candidate.sh"
command: "/bin/bash /nanoarrow/dev/release/verify-release-candidate.sh $NANOARROW_VERIFY_ARGS"

coverage:
image: ${REPO}:ubuntu-${NANOARROW_ARCH}
Expand Down

0 comments on commit a682731

Please sign in to comment.