kszucs commented on a change in pull request #9891:
URL: https://github.com/apache/arrow/pull/9891#discussion_r607820035
##########
File path: ci/docker/conda-integration.dockerfile
##########
@@ -25,32 +25,28 @@ ARG node=14
ARG jdk=8
ARG go=1.15
+# Uninstall unused space-consuming packages
+# (XXX: it would be better not to install them, but they are used by other
+# builds which are also based on conda-cpp)
+RUN conda uninstall -q clang llvmdev valgrind
+
+# Install Archery and integration dependencies
COPY ci/conda_env_archery.yml /arrow/ci/
RUN conda install -q \
--file arrow/ci/conda_env_archery.yml \
numpy \
maven=${maven} \
nodejs=${node} \
openjdk=${jdk} && \
- conda clean --all
+ conda clean --all --force-pkgs-dirs
-RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
+# Install Rust with only the needed components
+# (rustfmt is needed for tonic-build to compile the protobuf definitions)
+RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s --
--profile=minimal -y && \
+ $HOME/.cargo/bin/rustup component add rustfmt
ENV GOROOT=/opt/go \
GOBIN=/opt/go/bin \
GOPATH=/go \
PATH=/opt/go/bin:$PATH
RUN wget -nv -O - https://dl.google.com/go/go${go}.linux-${arch}.tar.gz | tar
-xzf - -C /opt
-
-ENV ARROW_BUILD_INTEGRATION=ON \
Review comment:
The primary purpose of defining the environment variables in the
dockerfiles to let the child images/services inherit them. We can override
these from docker-compose so please keep them here as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
