Ordered the travis builds approximately from slower to faster ones, to bring their end times closes together (hopefully resulting in a faster result overall).
Reorder travis builds for faster response #5429
pull sipa wants to merge 1 commits into bitcoin:master from sipa:reorder changing 1 files +6 −6-
sipa commented at 9:46 AM on December 5, 2014: member
- laanwj added the label Tests on Dec 5, 2014
-
-
theuni commented at 5:31 PM on December 10, 2014: member
This is fine by me, though there are a few other things to consider:
- builds aren't necessarily launched in order. Sometimes travis randomly builds out of order.
- attempting to build the fastest first, when combined with fast-finish (which we use) means that github shows a failure much more quickly.
- it may also be desirable to build the fast builds first in order to begin building subsequent PRs as quickly as possible. For example: pr1 is already building while pr2 is submitted. pr2 can start building as soon as there's a free slot. If we're attempting to have all builds finish at the same time, pr2 wouldn't start until ~all pr1 builds finished.
That said, I have no problem with trying different ordering strategies.
-
sipa commented at 5:33 PM on December 10, 2014: member
- Yes, it's not entirely in order, but it seems to be more or less in order at least.
- Maybe we want the shortest one first, and then from long to short?
- Unsure.
-
theuni commented at 5:42 PM on December 10, 2014: member
- Sounds like a good strategy.
-
fanquake commented at 1:33 AM on January 16, 2015: member
Needs rebase
-
laanwj commented at 9:03 AM on March 13, 2015: member
Yes, needs rebase
-
Reorder travis builds for faster response 5abe2cf110
- sipa force-pushed on Mar 13, 2015
-
sipa commented at 2:15 PM on March 13, 2015: member
Redone.
The first one is the fastest now (ARM), the following ones are ordered from slow to fast.
-
jonasschnelli commented at 2:31 PM on March 13, 2015: contributor
ACK.
Before PR:
After PR:
- sipa merged this on Mar 24, 2015
- sipa closed this on Mar 24, 2015
- sipa referenced this in commit 03106221d4 on Mar 24, 2015
- MarcoFalke locked this on Sep 8, 2021
