Skip to content

[ci] Disable GPU unit tests#13903

Closed
driazati wants to merge 1 commit intoapache:mainfrom
driazati:gpu_disable
Closed

[ci] Disable GPU unit tests#13903
driazati wants to merge 1 commit intoapache:mainfrom
driazati:gpu_disable

Conversation

@driazati
Copy link
Copy Markdown
Member

@driazati driazati commented Feb 2, 2023

This is a (hopefully) temporary bandaid to reduce cloud costs for OctoML
since they've spiked recently, mostly for GPU instances. This cuts down
on the GPU testing we do in order to save on those build minutes.

This is a (hopefully) temporary bandaid to reduce cloud costs for OctoML
since they've spiked recently, mostly for GPU instances. This cuts down
on the GPU testing we do in order to save on those build minutes.
@tvm-bot
Copy link
Copy Markdown
Collaborator

tvm-bot commented Feb 2, 2023

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 by @-ing them in a comment.

Generated by tvm-bot

@leandron
Copy link
Copy Markdown
Contributor

leandron commented Feb 2, 2023

Just a quick question. Can you clarify what are the tests being disabled? I imagine it is the diff between task_python_frontend_lite.sh and task_python_frontend.sh, but can you clarify what in practice was running before, and it is not running now?

@Mousius
Copy link
Copy Markdown
Member

Mousius commented Feb 2, 2023

@driazati are there any plans to allow other contributors to provide nodes? I remember I asked about this a long time ago but it wasn't a priority, now it might be worth re-visiting?

@driazati
Copy link
Copy Markdown
Member Author

driazati commented Feb 2, 2023

@driazati are there any plans to allow other contributors to provide nodes? I remember I asked about this a long time ago but it wasn't a priority, now it might be worth re-visiting?

On the technical side this isn't too complicated, it's mostly just a matter of making sure the Jenkins head can communicate with the machines and installing the client daemon. I'd be open to trying it if the machines are there but we'd probably also need some level of commitment to support those machines to make sure they're at least as reliable as our current runners

Just a quick question. Can you clarify what are the tests being disabled? I imagine it is the diff between task_python_frontend_lite.sh and task_python_frontend.sh, but can you clarify what in practice was running before, and it is not running now?

There's that and some more broad disabling of stuff in https://ci.tlcpack.ai/blue/organizations/jenkins/tvm-gpu/detail/main/244/pipeline vs https://ci.tlcpack.ai/blue/organizations/jenkins/tvm-gpu/detail/PR-13903/1/pipeline/101

@driazati driazati closed this Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants