leandron commented on PR #11809: URL: https://github.com/apache/tvm/pull/11809#issuecomment-1168539081
Looking at the alternatives being proposed until now, I feel that for *our current type of usage* is makes more sense to unify `ci_qemu` into `ci_cpu`, for two reasons: 1. [According to our Jenkinsfile](https://github.com/apache/tvm/blob/ce77d20c4615c655214d0bd38307813e539ba9a2/Jenkinsfile#L503-L511), both jobs that require `ci_cpu` and `ci_qemu` will run on the same target machines, so the expectation is that our Jenkins nodes will more commonly pull `ci_cpu + ci_qemu` in terms of storage 2. Creating a base image with common dependencies, would also generate a side problem, which is a _time dependency_ to rebuild our images (we can't build all of them at the same time), and the need to create a generic label e.g. `latest` or `last-successful` for this base image to be picked up in the dependant images. I'd like to avoid this if possible. Given our current Docker update practices that don't require updating all images at once, I feel we better to consolidate the dependencies in one single image until we can devise better rules for Docker updating. -- 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]
