-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[Test Failure] Clojure Integration #14415
Comments
Hey, this is the MXNet Label Bot. |
Yep, this was due to the Scala NDArray arange test failing. Going to go ahead and get this one closed. @mxnet-label-bot [Clojure, Test, Flaky, Scala] |
@mxnet-label-bot add [Clojure, Test, Flaky, Scala] |
Saw this again: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-14359/13/pipeline. @andrewfayres Any updates on the fix for this? |
That's a different failure.
Looks like an issue with the origami repo. It had an outage last week and may be experience another one. There's an open issue about mitigating the clojure integ test reliance on this #14394. |
@andrewfayres Please address this ASAP as I'm experiencing another occurrence on my other PR here: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-14445/2/pipeline. Thanks! |
@haojin2 not related to this issue and I don't own this code The CI failure you're seeing is due to clojure integ tests having a dependency on the origami repo which is apparently experiencing an outage. This happened last week and @gigasquid opened a PR (#14379) to disable those tests. The repo came back online before the PR got merged so she closed it instead. I did go ahead and open a new PR (#14448) which has the same changes and will disable the integ tests. Feel free to take a look and review it. I'll post a message in the slack mxnet-clojure channel to hopefully get a committer to look. |
@andrewfayres Thanks for the details, please lemme know when the workaround has been merged so that I can re-trigger my builds of all my PRs. Have a great weekend |
Merged it now. Let's wait for somebody to fix it. |
This has been resolved with the long term solution discussed in #14394 - so closing this one |
@gigasquid Seems like this test is failing again now but caused by a different reason here:
|
@gigasquid Could you help with identifying the cause so that we could fix this ASAP? Thanks! |
looking into it |
The failing test is a bit too brittle - it is verifying the saved model of the mnist. A new attribute has been added so the test is failing |
@gigasquid okay I see the cause. But no hurries on the fix for the test, I think @reminisce has also made changes to make that test working now. Thanks a lot for your prompt reply! I'll also close the issue now. |
Description
Seems the scala package tests are failing the Clojure Integration tests on CI
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/master/405/pipeline
Seems related to #14402
The text was updated successfully, but these errors were encountered: