Skip to content

Conversation

@Lunderberg
Copy link
Contributor

The mxnet project was archived in 2023-09 (link). Downloads from the mxnet AWS
links (e.g. https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/models/resnet18_v1-a0666292.zip) no longer can be performed. This commit disables TVM CI steps that depend on downloading models from these locations.

The mxnet project was archived in
2023-09 ([link](https://attic.apache.org/projects/mxnet.html)).
Downloads from the mxnet AWS
links (e.g. `https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/models/resnet18_v1-a0666292.zip`)
no longer can be performed.  This commit disables TVM CI steps that
depend on downloading models from these locations.
@Lunderberg
Copy link
Contributor Author

Failure to download from mxnet AWS links began in the evening of 2024-02-08, for multiple unrelated PRs. (example, example). Testing again this morning, but based on the archiving of the mxnet project, it may be permanent. If so, this commit can be used to avoid CI breakage.

@tqchen
Copy link
Member

tqchen commented Feb 8, 2024

mxnet is getting deprecated, so we should remove the testcases/example that depends on it

@tqchen
Copy link
Member

tqchen commented Feb 8, 2024

opened a related tracking issue #16547

@Lunderberg
Copy link
Contributor Author

That was my guess, and agreed that it doesn't make sense to maintain integration with a deprecated framework.

Marking this PR as ready for review, so that it can unblock CI. It still attempts to download files, but should bail out on error rather than failing CI altogether. A follow-up PR can then remove the test cases altogether.

@Lunderberg Lunderberg marked this pull request as ready for review February 8, 2024 16:16
@Lunderberg Lunderberg changed the title [Draft][CI] Disable testing that downloads from mxnet [CI] Disable testing that downloads from mxnet Feb 8, 2024
Copy link
Member

@yongwww yongwww left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to have it to unblock the CI.

@Lunderberg
Copy link
Contributor Author

Looks like CI is passing, except for the ARM tests, which are stalled waiting for ARM-GRAVITON3 instances. 1/8 has run since I checked this morning, so hopefully they'll get queued up sooner rather than later.

@yongwww
Copy link
Member

yongwww commented Feb 9, 2024

arm tests are stuck #16549

@Lunderberg
Copy link
Contributor Author

Thank you for mentioning that. Looks like the ARM CI was related to another of my PRs, which I recently rebased onto main. I've unstuck the ARM tests, and CI has now passed here.

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