-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
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.
LGTM
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.
Approving, but I'd appreciate if you could bisect and find out which PR introduced the issue and find out how it was possible to merge it while having missing dependencies. Something seems off.
@marcoabreu I’m also wondering how this would ever be possible. I guest it’s due to some internal update of the CI machines. |
The environment is fully dockerized, so changes on the CI machines shouldn't impact it as far as I know |
this could be a ccache cache issue. I can't reproduce locally. TVM lib seems corrupted. |
If this change is not fixing the failure, could we revert it if the additional deps are not needed? |
@larroy Yes, we should revert it |
This reverts commit a11b7ea.
Description
It looks that this issue #16815 is blocking the CI.
Revised based on the error message in http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-16902/4/pipeline
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments