Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
apacheGH-41481: [CI] Update how extra environment variables are speci…
…fied for the integration test docker job (apache#42009) ### Rationale for this change Currently, nanoarrow and Rust are not being included in the integration tests. The command issued by archery includes multiple environment variable definitions and the rightmost ones disable the extra environment variables. https://github.com/apache/arrow/actions/runs/9397807525/job/25881776553#step:9:353 ``` DEBUG:archery:Executing `['docker', 'run', '--rm', '-e', 'ARCHERY_DEFAULT_BRANCH=main', '-e', 'ARCHERY_INTEGRATION_WITH_NANOARROW=1', '-e', 'ARCHERY_INTEGRATION_WITH_RUST=1', '-e', 'ARCHERY_INTEGRATION_WITH_NANOARROW=0', '-e', 'ARCHERY_INTEGRATION_WITH_RUST=0', '-e', 'ARROW_CPP_EXE_PATH=/build/cpp/debug', '-e', 'ARROW_NANOARROW_PATH=/build/nanoarrow', '-e', 'ARROW_RUST_EXE_PATH=/build/rust/debug', '-e', 'CCACHE_COMPILERCHECK=content', '-e', 'CCACHE_COMPRESS=1', '-e', 'CCACHE_COMPRESSLEVEL=6', '-e', 'CCACHE_DIR=/ccache', '-e', 'CCACHE_MAXSIZE=1G', '-e', 'GITHUB_ACTIONS=true', '-v', '/home/runner/work/arrow/arrow:/arrow', '-v', '/home/runner/work/arrow/arrow/.docker/conda-ccache:/ccache', 'apache/arrow-dev:amd64-conda-integration', '/arrow/ci/scripts/integration_arrow_build.sh /arrow /build && /arrow/ci/scripts/integration_arrow.sh /arrow /build']` # ... + /arrow/ci/scripts/rust_build.sh /arrow /build ===================================================================== Not building the Rust implementation. ===================================================================== + /arrow/ci/scripts/nanoarrow_build.sh /arrow /build ===================================================================== Not building nanoarrow ===================================================================== ``` ### What changes are included in this PR? This PR updates how environment variables are specified such that the intended value is passed to the docker build. ### Are these changes tested? Yes ### Are there any user-facing changes? No * GitHub Issue: apache#41481 Lead-authored-by: Dewey Dunnington <dewey@fishandwhistle.net> Co-authored-by: Dewey Dunnington <dewey@voltrondata.com> Co-authored-by: Sutou Kouhei <kou@cozmixng.org> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
- Loading branch information