Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[ci] Create comment bot to highlight CI runtime changes #12781

Closed
driazati opened this issue Sep 14, 2022 · 0 comments
Closed

[ci] Create comment bot to highlight CI runtime changes #12781

driazati opened this issue Sep 14, 2022 · 0 comments
Labels
actionable has an immediately do-able work plan and a detailed description dev:ci type:ci Relates to TVM CI infrastructure

Comments

@driazati
Copy link
Member

driazati commented Sep 14, 2022 •
edited by github-actions bot
Loading

It's difficult to know whether a PR will affect CI runtime without doing a purposeful investigation or hitting timeouts. Similar to the tests_bot (https://github.com/apache/tvm/blob/main/.github/workflows/tests_bot.yml) we could have a bot that just comments the runtime change of the last Jenkins build vs the median of the prior N builds from main

cc @Mousius @areusch @gigiblender
@driazati driazati added type:ci Relates to TVM CI infrastructure actionable has an immediately do-able work plan and a detailed description labels Sep 14, 2022
@driazati driazati mentioned this issue Sep 15, 2022
Closed
driazati added a commit to driazati/tvm that referenced this issue Sep 15, 2022
@driazati
[ci] Comment runtime changes on PRs
b36c225 
This implements apache#12781 but needs to wait for apache#12695 before it can merge
driazati added a commit to driazati/tvm that referenced this issue Sep 16, 2022
@driazati
[ci] Comment runtime changes on PRs
04f4ec5 
This implements apache#12781 but needs to wait for apache#12695 before it can merge
driazati added a commit to driazati/tvm that referenced this issue Sep 16, 2022
@driazati
[ci] Comment runtime changes on PRs
a43bf39 
This implements apache#12781 but needs to wait for apache#12695 before it can merge
driazati added a commit to driazati/tvm that referenced this issue Sep 16, 2022
@driazati
[ci] Comment runtime changes on PRs
a379ebb 
This implements apache#12781 but needs to wait for apache#12695 before it can merge
@areusch areusch added the needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it label Oct 19, 2022
@janetsc janetsc added dev:ci and removed needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it labels Oct 19, 2022
@tqchen tqchen closed this as completed Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
actionable has an immediately do-able work plan and a detailed description dev:ci type:ci Relates to TVM CI infrastructure
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@areusch @tqchen @driazati @janetsc