[GitHub] marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo
marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380617930 Alrighty, fine with me :) Could you please add a jira ticket before we merge? 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
[GitHub] marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo
marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380612813 I see, thanks for elaborating. So just to clarify: This means if a new slave for the internal CI gets deployed or if the docker cache gets cleaned, this task is going to fail, right? Fine with me as a temporary solution, we just have to be aware of that risk. 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
[GitHub] marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo
marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380609669 I think pushing to dockerhub could cause confusion since this is basically an empty cuda9 container and is not really related to mxnet itself. Why can't we just create a simple Dockerfile (exactly what you did in order to create that image locally), put it here and re-use it instead of relying on the local cache or the dockerhub? 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
[GitHub] marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo
marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380609669 I think pushing to dockerhub could cause confusion since this is basically an empty cuda9 container and is not really related to mxnet itself. 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
[GitHub] marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo
marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380608929 Ah, is it because you're trying to replicate what we're basically doing in our containers, just from a users point of view and that's why a pretty much empty docker container is required? 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