-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
[beta] Use time crate in bootstrap dist instead of date #48118
Conversation
The bot failed to assign a reviewer... @alexcrichton? |
@bors: r+ p=1 |
📌 Commit 0895099 has been approved by |
@bors p=20 |
⌛ Testing commit 0895099 with merge fa56e087a5e971f9372d5023e91abeb2be91080c... |
💔 Test failed - status-travis |
Heh, no space left on device?
The disk usage report, 14 GB used by
The size of the current beta HEAD is 13 GB https://api.travis-ci.org/v3/job/336884022/log.txt, so this is sort of the last straw.
|
I'm going to retry, I have a hard time believing this is actually constantly true because of this patch... @bors retry |
@Mark-Simulacrum The size is 10 GB six months ago, so it is likely a very old problem. We can patch compiletest to delete those generated *.err/*.out dump if the test is successful. |
Well, sure, and we probably should just not keep any artifacts around on CI that aren't going to be used, but I would somewhat expect this PR not to alter whether we fail or succeed. |
⌛ Testing commit 0895099 with merge c625ea3c88b436c1fd5716ec55b289718a89fc31... |
💔 Test failed - status-travis |
Well, I guess it looks like we're going to have trouble landing this without some space-saving patches...
|
@onur Could you try to ignore the diff --git a/src/test/run-pass/rustc-rust-log.rs b/src/test/run-pass/rustc-rust-log.rs
index 629387d4cb..a0e04beaa9 100644
--- a/src/test/run-pass/rustc-rust-log.rs
+++ b/src/test/run-pass/rustc-rust-log.rs
@@ -9,5 +9,6 @@
// except according to those terms.
// rustc-env:RUST_LOG=debug
+// ignore-musl - workaround to avoid consuming all CI storage, see #48118 I bet 90% of the space usage comes from this test. |
Done @kennytm |
@bors r=alexcrichton |
📌 Commit 309e0d6 has been approved by |
⌛ Testing commit 309e0d6 with merge fc74e576a09f533d7a49072eec9e744a13a92e95... |
💔 Test failed - status-travis |
Guess we're wrong, |
going to close this in favor of the general beta backport PR (#48146) |
Compiletest: delete the compiled program once its test is done. Reduces CI disk usage which prevents #48118 from succeeding.
Beta backport of #48013