Skip to content

Conversation

@driazati
Copy link
Member

@driazati driazati commented Nov 3, 2022

This PR makes it so any URLs accessed in tests in CI must be hosted in
S3. This improves reliability as we've seen even files on GitHub
sometimes serve 503s even when everything else is working fine. This
raises an error if any unallowed URL is detected and adds the remaining
few.

@tvm-bot
Copy link
Collaborator

tvm-bot commented Nov 3, 2022

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

@driazati driazati requested a review from areusch November 16, 2022 20:36
This PR makes it so any URLs accessed in tests in CI must be hosted in
S3. This improves reliability as we've seen even files on GitHub
sometimes serve 503s even when everything else is working fine. This
raises an error if any unallowed URL is detected and adds the remaining
few.
@driazati driazati merged commit d31a1fb into apache:main Dec 3, 2022
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.

3 participants