Opened at 2011-01-15T04:04:09Z
Closed at 2019-07-25T12:49:20Z
#1309 closed defect (fixed)
reorder buildbot steps for speed — at Version 3
Reported by: | davidsarah | Owned by: | somebody |
---|---|---|---|
Priority: | minor | Milestone: | soon (release n/a) |
Component: | dev-infrastructure | Version: | n/a |
Keywords: | buildbot | Cc: | |
Launchpad Bug: |
Description (last modified by exarkun)
zooko wrote on 1190#comment:43:
As David-Sarah pointed out in 1190#comment:42, we [were] seeing this VersionConflict on some buildslaves now because I moved the invocation of test-with-fake-dists.py (by the buildmaster) to happen before any build step. (This is necessary because the build step provides a good version of pycryptopp so once it has run then the test-with-fake-dists.py step can't determine how well our setup scripts handle the lack of a good version of pycryptopp...)
On the ticket1306 branch ([4935/ticket1306]), test-with-fake-dists was changed to use a "fakedependency" package, so that it no longer matters what versions of pycryptopp are installed or which version has been built. Once we push this to trunk, test-with-fake-dists can be moved to after 'test', which should still be a good test for that part of #1190, but will be much faster because it won't have to rebuild any of the genuine dependencies. It would also mean that failure to clean up properly after test-with-fake-dists can't affect the main build or test.
Change History (3)
comment:1 Changed at 2011-01-15T04:05:49Z by davidsarah
- Description modified (diff)
comment:2 Changed at 2011-05-11T18:56:22Z by zooko
comment:3 Changed at 2019-07-25T12:49:20Z by exarkun
- Description modified (diff)
- Resolution set to fixed
- Status changed from new to closed
Buildbot has been decommissioned and the CI that replaced it seems to be doing things in a pretty good order, with respect to completion time.
That test-with-fake-dists improvement was committed to trunk as e2b564024b5482fe, 089b27d9f8e726e7, e2b564024b5482fe.