Merge lp:~wgrant/launchpad/faster-and-more-general-getBuildQueueSizes into lp:launchpad
Proposed by
William Grant
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Julian Edwards | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 11102 | ||||
Proposed branch: | lp:~wgrant/launchpad/faster-and-more-general-getBuildQueueSizes | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
121 lines (+30/-28) 3 files modified
lib/lp/buildmaster/doc/builder.txt (+18/-0) lib/lp/buildmaster/model/builder.py (+9/-27) lib/lp/soyuz/stories/soyuz/xx-buildfarm-index.txt (+3/-1) |
||||
To merge this branch: | bzr merge lp:~wgrant/launchpad/faster-and-more-general-getBuildQueueSizes | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Abel Deuring (community) | code | Approve | |
Review via email: mp+28476@code.launchpad.net |
Commit message
Generalise BuilderSet.
Description of the change
This branch generalises BuilderSet.
As mentioned in bug #592573, this misses jobs which are architecture-
There is no lint.
To post a comment you must log in.
{{{ /code.edge. launchpad. net/~wgrant/ launchpad/ faster- and-more- general- getBuildQueueSi zes/+merge/ 28476, if anyone's interested...
09:20 < wgrant> Nice simple buildfarm-related branch at https:/
09:21 < noodles775> Nice... more red than green :)
09:22 < noodles775> And much simpler query. Is it worth adding tests that ensure the other build types are considered (or possible yet? or worth waiting to land this when it is possible?)
09:23 < wgrant> It's not possible yet.
09:24 < wgrant> Hm.
09:24 < wgrant> Actually, it might be.
09:24 < wgrant> Since this is just BuildQueue, not BuildFarmJob.
09:25 < noodles775> Right - which means it's implied anyway (and doesn't need to be tested... as the BFJ table isn't even referenced)?
09:26 < wgrant> If it works for one it will work for the others. It would still be nice to test, though.
09:26 < wgrant> Let's see how easy the tests are to change.
09:27 < noodles775> And would ensure the implementation doesn't change in the future to exclude them... great, thanks wgrant.
}}}