lebeg commented on a change in pull request #10510: [MXNET-311] Change the
docker image for Installation Guide Test - needs sudo
URL: https://github.com/apache/incubator-mxnet/pull/10510#discussion_r180998909
##########
File path: tests/jenkins/run_test_installation_docs.sh
##########
@@ -298,17 +298,20 @@ LINUX_PYTHON_GPU_END_LINENO=$(grep -n "END - Linux
Python GPU Installation Instr
set_instruction_set ${LINUX_PYTHON_GPU_START_LINENO}
${LINUX_PYTHON_GPU_END_LINENO}
+
+# mxnet/base-cuda9 is a simple Docker Image with
'nvidia/cuda:9.0-cudnn7-devel' and 'apt-get install sudo'.
+
echo
echo "### Testing Virtualenv ###"
echo "${virtualenv_commands}"
echo
-nvidia-docker run --rm nvidia/cuda:9.0-cudnn7-devel bash -c
"${virtualenv_commands}"
+nvidia-docker run --rm mxnet/base-cuda9 bash -c "${virtualenv_commands}"
Review comment:
Where can I see the contents of this image? Also it would be good to keep
consistency with nvidia images (I assume they are taken as base anyway) and
name and branch them with minor versions of cuda: ```mxnet/base:cuda9.0```
```mxnet/base:cuda9.1```
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
