You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/05/06 23:44:56 UTC

[GitHub] [tvm] jroesch commented on issue #7995: Update all ci- containers to reflect main

jroesch commented on issue #7995:
URL: https://github.com/apache/tvm/issues/7995#issuecomment-833950188


   So last time I tried to update the images but ran into issues where tests no longer passed under updated GPU and CPU images (see docker pull tlcpack/ci-gpu:v0.73-t3).
   
   I had chased down the image building, there are a few important things. 
   
   First disable caching in order to make sure you pull the freshest `apt` as if you have built before you can have a stale layer which pulled an apt-registry which is no longer valid. 
   
   Second there were some updates to the drivers which caused breakage, specifically this test no longer works because it incorrectly believes that CUDA is on (even though there is no GPU): 
   
   https://github.com/apache/tvm/blob/main/tests/cpp/build_module_test.cc#L84
   
   These needs to be patched to actually check for the GPU's existence. 
   
   There were also changes intended to fix Rust CI which should included in master today allowing us to turn Rust CI back on.
   


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