areusch commented on code in PR #11470:
URL: https://github.com/apache/tvm/pull/11470#discussion_r888373343


##########
docker/install/ubuntu_install_zephyr.sh:
##########
@@ -38,10 +39,12 @@ sudo apt-key add kitware-archive-latest.asc
 
 echo deb https://apt.kitware.com/ubuntu/ bionic main\
      >> /etc/apt/sources.list.d/kitware.list
-sudo apt-get update
+sudo apt-get update --fix-missing

Review Comment:
   okay looking over all of this i agree there are some challenges then. I 
don't think we should take an approach that requires us to run apt-get update 
for each layer. we do need to run it:
   - one time before the first apt-get install for each docker image
   - any time a script adds to /etc/apt/sources.list.d
   
   i think we could work around this either by:
   - not removing /var/lib/apt/lists (i just built ci_cpu and it's 40M. that 
seems small relative to any accumulated packages)
   - attempting to mount that dir in a docker volume (this doesn't actually 
look super-feasible from docker build, but perhaps a workaround exists with 
e.g. buildx or something)
   - doing something a bit crazy like making a tar of /var/lib/apt/lists and 
uploading that to the build host each time. i actually think we should not do 
this one, but if a simpler similar thing exists and the lists are larger than 
40M, maybe it's worth it. 
   
   feel free to investigate a bit here. the main thing i'm interested in is 
removing any cached debian packages



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