mseth10 commented on a change in pull request #19890:
URL: https://github.com/apache/incubator-mxnet/pull/19890#discussion_r575470041
##########
File path: ci/safe_docker_run.py
##########
@@ -117,6 +119,9 @@ def run(self, *args, **kwargs) -> int:
ret = 0
try:
# Race condition:
+ # add a random sleep to (a) give docker time to flush disk buffer
after pulling image
+ # and (b) minimize race conditions between jenkins runs on same
host
+ time.sleep(random.randint(2,10))
Review comment:
how does random help? vs let's say a fixed wait of 5 seconds?
----------------------------------------------------------------
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]
