You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by "yongwww (via GitHub)" <gi...@apache.org> on 2023/04/18 00:49:49 UTC

[GitHub] [tvm] yongwww opened a new pull request, #14651: [WIP][CI] update all the images and test cases

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

   This is for testing the images built from https://github.com/apache/tvm/pull/14635


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] tvm-bot commented on pull request #14651: [WIP][CI] update all the images and test cases

Posted by "tvm-bot (via GitHub)" <gi...@apache.org>.
tvm-bot commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1512285558

   <!---bot-comment-->
   
   Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from [Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers) by @-ing them in a comment.
   
   <!--bot-comment-ccs-start-->
    * cc @Mousius, @areusch, @driazati, @gigiblender, @leandron <sub>See [#10317](https://github.com/apache/tvm/issues/10317) for details</sub><!--bot-comment-ccs-end-->
   
   <sub>Generated by [tvm-bot](https://github.com/apache/tvm/blob/main/ci/README.md#github-actions)</sub>


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] yongwww commented on pull request #14651: [CI] Modify test cases to accommodate the CI upgrades

Posted by "yongwww (via GitHub)" <gi...@apache.org>.
yongwww commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1535356893

   @tvm-bot  rerun


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] yongwww commented on pull request #14651: [CI] Modify test cases to accommodate the CI upgrades

Posted by "yongwww (via GitHub)" <gi...@apache.org>.
yongwww commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1535634995

   The docker images built from the ci upgrade https://github.com/apache/tvm/pull/14635 have been automatically uploaded to tlcpackstaging. In this pr, I have modified the docker-images.ini to point to those newly uploaded images, and CI is green now. Please take a look and help land it if there are no further change requests. cc: @tqchen @leandron @masahi @driazati


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] yongwww commented on pull request #14651: [CI] Modify test cases to accommodate the CI upgrades

Posted by "yongwww (via GitHub)" <gi...@apache.org>.
yongwww commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1526091903

   It is ready for review now, CI is supposed to be green soon. cc: @tqchen @masahi 


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] yongwww commented on pull request #14651: [DO NOT MERGE][CI] Modify test cases to accommodate the CI upgrades

Posted by "yongwww (via GitHub)" <gi...@apache.org>.
yongwww commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1533279172

   @leandron I am following [the steps](https://tvm.apache.org/docs/contribute/ci.html#updating-a-docker-image-tag) to upgrade the CI. I assume the upgraded images will be built and uploaded into tlcpackstaging after #14635 lands, then I just need to update ci/jenkins/docker-images.ini in this PR to point to the expected newly generated images. 


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] tqchen commented on pull request #14651: [DO NOT MERGE][CI] Modify test cases to accommodate the CI upgrades

Posted by "tqchen (via GitHub)" <gi...@apache.org>.
tqchen commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1533082675

   The CI is green now. Based on what I read, there is a inter dep between changing the docker name and properly build and upload the images to the dockerhub.
   
   I assume once we merged the docker part and images get uploaded, it would be as simple as change the images to the right one and it can be done quickly.


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] leandron merged pull request #14651: [CI] Modify test cases to accommodate the CI upgrades

Posted by "leandron (via GitHub)" <gi...@apache.org>.
leandron merged PR #14651:
URL: https://github.com/apache/tvm/pull/14651


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] tqchen commented on pull request #14651: [DO NOT MERGE][CI] Modify test cases to accommodate the CI upgrades

Posted by "tqchen (via GitHub)" <gi...@apache.org>.
tqchen commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1532311597

   @yongwww please resolve the conflict


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] yongwww commented on pull request #14651: [CI] Modify test cases to accommodate the CI upgrades

Posted by "yongwww (via GitHub)" <gi...@apache.org>.
yongwww commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1535490329

   @tvm-bot rerun


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] leandron commented on pull request #14651: [CI] Modify test cases to accommodate the CI upgrades

Posted by "leandron (via GitHub)" <gi...@apache.org>.
leandron commented on PR #14651:
URL: https://github.com/apache/tvm/pull/14651#issuecomment-1536079384

   Thanks @yongwww @tqchen, this is now merged, thanks for the great work!


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org