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/02/12 01:01:19 UTC
[GitHub] [tvm] areusch opened a new pull request #7450: Report JUnit test results for all TVM Python tests
areusch opened a new pull request #7450:
URL: https://github.com/apache/tvm/pull/7450
This change enables JUnitXML output from pytest for all TVM python testing. The goal is to enable us to better understand which TVM tests are slow, how a change might impact them, and the runtime variance we see in the TVM CI.
See the example test report produced from the [staging run](https://ci.tlcpack.ai/job/tvm/job/ci-docker-staging/59/testReport/) on this PR.
Note that a [previous staging run](https://ci.tlcpack.ai/job/tvm/job/ci-docker-staging/58/) did timeout in the frontend tests, but by a long shot, so I think there's a chance it was unrelated to this PR.
@tqchen @junrushao1994
----------------------------------------------------------------
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
[GitHub] [tvm] areusch commented on pull request #7450: Report JUnit test results for all TVM Python tests
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #7450:
URL: https://github.com/apache/tvm/pull/7450#issuecomment-778292349
@leandron yeah that was enabled in #7407
----------------------------------------------------------------
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
[GitHub] [tvm] areusch commented on pull request #7450: Report JUnit test results for all TVM Python tests
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #7450:
URL: https://github.com/apache/tvm/pull/7450#issuecomment-779965536
I had thought I pushed `6a77cdd` to `ci-docker-staging` on friday, but looks like not. let's do that now and wait to merge til we see the result
----------------------------------------------------------------
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
[GitHub] [tvm] junrushao1994 merged pull request #7450: Report JUnit test results for all TVM Python tests
Posted by GitBox <gi...@apache.org>.
junrushao1994 merged pull request #7450:
URL: https://github.com/apache/tvm/pull/7450
----------------------------------------------------------------
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
[GitHub] [tvm] areusch commented on pull request #7450: Report JUnit test results for all TVM Python tests
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #7450:
URL: https://github.com/apache/tvm/pull/7450#issuecomment-780340915
@leandron please explicitly approve if you're ok w/ this.
----------------------------------------------------------------
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
[GitHub] [tvm] areusch commented on pull request #7450: Report JUnit test results for all TVM Python tests
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #7450:
URL: https://github.com/apache/tvm/pull/7450#issuecomment-780340777
ok this is ready to merge. I tested with the prefix option, the output is [here](https://ci.tlcpack.ai/job/tvm/job/ci-docker-staging/60/testReport/).
I made one final change (remove the `-` in the prefix to make it look like a python namespace). I don't think we need to run the one-character change in 205871e through staging CI, but we can if that's policy.
----------------------------------------------------------------
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