You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2021/02/16 17:47:57 UTC

[GitHub] [incubator-mxnet] josephevans opened a new pull request #19903: Forward port #19895

josephevans opened a new pull request #19903:
URL: https://github.com/apache/incubator-mxnet/pull/19903


   ## Description ##
   Forward port #19895 from v1.x to master. 
   
   This PR makes a number of changes to make it more stable:
   
   * Remove SafeDocker client which uses python docker package to run containers. Change to use "docker run" command directly using subprocess.call(), because the python-docker client does not support a gpus parameter which newer docker versions use and we don't get timeout issues when using the docker command directly. This will allow us to update our AMIs to use newer docker versions.
   * In order to support both docker variants simultaneously, we first try to use the --gpus all parameter to docker run, if it fails with error code 125 (which means docker run command itself failed,) then we retry using the old --runtime nvidia parameter.
   * Remove the extra custom codecov calls (which usually fail and have to retry multiple times, even though the initial codecov command works.)
   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #19903: Forward port #19895

Posted by GitBox <gi...@apache.org>.
mxnet-bot commented on pull request #19903:
URL: https://github.com/apache/incubator-mxnet/pull/19903#issuecomment-780007724


   Hey @josephevans , Thanks for submitting the PR 
   All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands: 
   - To trigger all jobs: @mxnet-bot run ci [all] 
   - To trigger specific jobs: @mxnet-bot run ci [job1, job2] 
   *** 
   **CI supported jobs**: [miscellaneous, edge, centos-cpu, windows-gpu, website, sanity, clang, unix-gpu, windows-cpu, centos-gpu, unix-cpu]
   *** 
   _Note_: 
    Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. 
   All CI tests must pass before the PR can be merged. 
   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-mxnet] leezu merged pull request #19903: Forward port #19895

Posted by GitBox <gi...@apache.org>.
leezu merged pull request #19903:
URL: https://github.com/apache/incubator-mxnet/pull/19903


   


----------------------------------------------------------------
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:
users@infra.apache.org