-
-
Notifications
You must be signed in to change notification settings - Fork 624
Test failure in test-long etc. because of singular #40120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
This might be fixed by just bumping singular version. But why do Last times singular was updated
it doesn't seem that pull requests prior to these (e.g. #37790) breaks when singular is updated though. So… |
Actually #40033 already updates it to 4.4.1. This appears to be first introduced in that pull request (https://github.com/sagemath/sage/actions/runs/14761095232/job/41441717381), not sure what went wrong. |
It's claimed in #39973 that it's just a caching issue, but a full build ought to be made for each release, and because #40033 is already merged and https://github.com/sagemath/sage/actions/runs/15095151507/job/42428827874 is built, it ought to work? Why not? |
My current best guess for what happens: the docker image needed are no longer pushed. Also explains the massive slowdown seen by test-new earlier (and maybe also test-long now). If I understood correctly, currently in However Looking at the last release before that pull request is merged, What would a reasonable course of action be in this case?
@kwankyu @tobiasdiez @dimpase @vbraun help? Currently test-long are not ran at all. |
sagemathgh-40157: Add back ubuntu-noble-standard to ci-linux Fixes sagemath#40120 — I believe. As explained there, the root cause is that `ci-linux.yml` no longer build from scratch on `ubuntu-noble-standard` to generate the base docker image for the incremental builds to base on. In any case this appears to be harmless enough. From some [testing](https://github.com/user202729/sage/actions/workflows/ci- linux.yml) this works _some_ of the time (the debug statements are harmless)… which is not great, but the **current** `standard` runs aren't completely reliable either. ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#40157 Reported by: user202729 Reviewer(s): Travis Scrimshaw
sagemathgh-40157: Add back ubuntu-noble-standard to ci-linux Fixes sagemath#40120 — I believe. As explained there, the root cause is that `ci-linux.yml` no longer build from scratch on `ubuntu-noble-standard` to generate the base docker image for the incremental builds to base on. In any case this appears to be harmless enough. From some [testing](https://github.com/user202729/sage/actions/workflows/ci- linux.yml) this works _some_ of the time (the debug statements are harmless)… which is not great, but the **current** `standard` runs aren't completely reliable either. ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#40157 Reported by: user202729 Reviewer(s): Travis Scrimshaw
Uh oh!
There was an error while loading. Please reload this page.
Steps To Reproduce
Log on CI https://github.com/sagemath/sage/actions/runs/15081532937/job/42398821050?pr=39973 :
Also mentioned in #40085 and #39973 (comment)
The text was updated successfully, but these errors were encountered: