-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add @slow decorator to run tests on main
#10057
Conversation
e7ab449
to
2091729
Compare
Hi @driazati, Can you recreate your RFC in |
done! apache/tvm-rfcs#55 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple questions and one minor thing
4e78a0d
to
38c7ba9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apache/tvm-rfcs#55 is now merged, I'm going to leave this with @areusch to merge if happy 😸
@driazati it looks like it's merge cornflakes this morning 😿 if you rebase I can kick off a branch build for you. |
This adds the infrastructure discussed in https://discuss.tvm.apache.org/t/rfc-ci-skip-slow-tests-on-prs/11910, but without affecting any tests. As we investigate reasons behind [slow tests](https://gist.github.com/driazati/e009f09ff44c6bc91c4d95a8e17fd6f1) in CI, this decorator will allow us to move these to run only on `main` and not PRs after checking with all concerned parties.
* Add @slow decorator to run tests on `main` This adds the infrastructure discussed in https://discuss.tvm.apache.org/t/rfc-ci-skip-slow-tests-on-prs/11910, but without affecting any tests. As we investigate reasons behind [slow tests](https://gist.github.com/driazati/e009f09ff44c6bc91c4d95a8e17fd6f1) in CI, this decorator will allow us to move these to run only on `main` and not PRs after checking with all concerned parties. * cleanup Co-authored-by: driazati <driazati@users.noreply.github.com>
This adds the infrastructure discussed in https://discuss.tvm.apache.org/t/rfc-ci-skip-slow-tests-on-prs/11910, but without affecting any tests. As we investigate reasons behind slow tests in CI, this decorator will allow us to move these to run only on
main
and not PRs after checking with all concerned parties.To run slow tests on a PR, the PR body needs to have
@ci run slow tests
(or similar, seeshould_run_slow_tests.py
) in it somewhere. Skipping slow tests is a positive-only check, so they have to be explicitly skipped rather than run (meaning that for local runs no workflow change is necessary, slow tests will run as before)