driazati opened a new pull request, #11673:
URL: https://github.com/apache/tvm/pull/11673

   This activates the code from #11329 behind a couple Jenkins flags:
   * `USE_AUTOUPDATED_DOCKER_IMAGES` - whether or not to use the dynamically 
determined docker images in the build
   * `PUSH_DOCKER_IMAGES` - whether or not to push built images to `tlcpack` on 
`main`
   
   These are there so we can quickly toggle this behavior off it breaks 
anything. Once this is on though, any docker changes in a PR should be 
reflected entirely in that build, so there would be no need for a separate PR 
to update the Docker images.
   
   We will have to see if this works in practice, as the docker image builds do 
download quite a bit of data (which can flake) and add some runtime overhead 
(about 30m), so when an update lands all PRs will end up needing to rebuild 
until the merged commit finishes.
   
   Thanks for contributing to TVM!   Please refer to guideline 
https://tvm.apache.org/docs/contribute/ for useful information and tips. After 
the pull request is submitted, please request code reviews from 
[Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers)
 by @ them in the pull request thread.
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to