-
Notifications
You must be signed in to change notification settings - Fork 13.3k
CI: Trim some tests from i686-gnu #59054
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
@bors p=50 |
Yeah, this seems fine to me. @bors r+ p=50 |
📌 Commit 635b2c95446a54c819c4efcc1eb3123de4bcfe80 has been approved by |
cc @rust-lang/infra |
@bors r- |
This removes some tests from the i686-gnu job. This job clocks in at 2hr 56min, and removing these should cut about 10 to 15 minutes, giving a little more breathing room. I suspect these don't need to be tested on every platform.
635b2c9
to
609316a
Compare
My apologies on the typo. I also removed src/doc because it is an implicit dependency from linkchecker. |
@bors r+ p=50 |
📌 Commit 609316a has been approved by |
⌛ Testing commit 609316a with merge 827ca46317788c731a7dc7cd265596de48debeeb... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-travis |
@bors retry What's up with the docker cache???? |
CI: Trim some tests from i686-gnu This removes some tests from the i686-gnu job. This job clocks in at 2hr 56min, and removing these should cut about 10 to 15 minutes, giving a little more breathing room. I suspect these don't need to be tested on every platform.
This PR changed a Docker image so all of them will be rebuilt. |
☀️ Test successful - checks-travis, status-appveyor |
@pietroalbini This PR only changed the Dockerfile of i686-gnu, it shouldn't affect dist-x86_64-linux. The current hash calculation is probably too conservative. |
Due to dependencies on other folders, we essentially have to be as pessimistic as possible. |
This removes some tests from the i686-gnu job. This job clocks in at 2hr 56min, and removing these should cut about 10 to 15 minutes, giving a little more breathing room. I suspect these don't need to be tested on every platform.