[GitHub] marcoabreu closed pull request #10581: [MXNET-327] Fix docker disk-space leak by rearranging ubuntu_build_cuda

[GitBox <gi...@apache.org>]

marcoabreu closed pull request #10581: [MXNET-327] Fix docker disk-space leak by rearranging ubuntu_build_cuda
URL: https://github.com/apache/incubator-mxnet/pull/10581
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/ci/docker/Dockerfile.build.ubuntu_build_cuda b/ci/docker/Dockerfile.build.ubuntu_build_cuda
index 8bafed43b83..9156d6f7b69 100755
--- a/ci/docker/Dockerfile.build.ubuntu_build_cuda
+++ b/ci/docker/Dockerfile.build.ubuntu_build_cuda
@@ -41,14 +41,16 @@ COPY install/ubuntu_clang.sh /work/
 RUN /work/ubuntu_clang.sh
 COPY install/ubuntu_mklml.sh /work/
 RUN /work/ubuntu_mklml.sh
-COPY install/ubuntu_adduser.sh /work/
-RUN /work/ubuntu_adduser.sh
 
 # Special case because the CPP-Package requires the CUDA runtime libs
-# and not only stubs..
+# and not only stubs (which are provided by the base image)
 COPY install/ubuntu_nvidia.sh /work/
 RUN /work/ubuntu_nvidia.sh
 
+# Keep this at the end since this command is not cachable
+COPY install/ubuntu_adduser.sh /work/
+RUN /work/ubuntu_adduser.sh
+
 COPY runtime_functions.sh /work/
 
 WORKDIR /work/mxnet


 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services