-
Notifications
You must be signed in to change notification settings - Fork 42
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
categorize sandbox creation failure #92
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.
Any chance of finding out if the test failures are related to the code changes?
It also would be nice to have a test, but I see that it's
|
I ran the tests locally under WSL 2 in Windows 11
there all tests, including the two failing here, pass. Edit: though my toolchain is outdated
Edit: updated toolchain and tests still pass locally
|
Attempting to run the test on my other machine runnign linux mint I can reproduce these errors, but I also get several more. One thing to note is that I had to run
I think the problem (with the tests failing here) might be that the tests are build outside of docker which has a recent version of glibc, but then run in docker with an older version of glibc. Edit: The docker image appears to have glibc 2.35
The tests failing for me locally on linux mint are:
I am currently still investigating. For the tests that fail locally but not here in CI I get this error
|
I now made a checkout of the master branch and the same tests are failing so this appears to be unrelated to this PR.
|
It might be possibel to setup a filesystem storage quota for docker and then have rustwide try to create a container that is expected to be larger than the quota, but I wouldn't know how to go about doing this.
there is the |
Found the reason why my linux mint machine had more errors than CI, |
I think the source of the glibc error is that in the last CI run prior to this PR and for this PR its Which causes the tests to be build against a newer glibc version than is available in the Solutions would be to either
|
14a4e70
to
5dffc77
Compare
🤦 forgot the approval before merge :) So let's see this as approved. @Skgland do you have more things to add? Otherwise I would create a release. |
No, no further additions from my side at the moment. A few chores that could be done come to mind like updating/upgrading dependencies and rust edition, but nothing I require or want to do right now. |
released in 0.19.0 |
This is so that crater can categorize this as a spurious failure similar to SandboxOOM.
See rust-lang/crater#700 (comment)