Merge lp:~thumper/launchpad/stop-mirroring-hosted into lp:launchpad
Proposed by
Tim Penhey
Status: | Merged |
---|---|
Approved by: | Michael Hudson-Doyle |
Approved revision: | no longer in the source branch. |
Merged at revision: | 11250 |
Proposed branch: | lp:~thumper/launchpad/stop-mirroring-hosted |
Merge into: | lp:launchpad |
Diff against target: |
201 lines (+19/-30) 6 files modified
lib/lp/code/browser/tests/test_branch.py (+2/-2) lib/lp/code/model/branch.py (+4/-4) lib/lp/code/model/tests/test_branch.py (+9/-8) lib/lp/code/model/tests/test_branchmergeproposaljobs.py (+0/-1) lib/lp/code/model/tests/test_branchtarget.py (+1/-7) lib/lp/code/xmlrpc/tests/test_codehosting.py (+3/-8) |
To merge this branch: | bzr merge lp:~thumper/launchpad/stop-mirroring-hosted |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Hudson-Doyle | Approve | ||
Review via email: mp+31116@code.launchpad.net |
Commit message
Make the world explode if requestMirror, startMirror, or mirrorFailed are called on hosted branches.
Description of the change
This is the final branch in the stack that enforces not calling the branch mirror methods on hosted branches.
Exceptions are now raised if requestMirror, startMirror, or mirrorFailed are called on hosted branches.
Several tests that were doing this have been fixed.
To post a comment you must log in.
This looks basically fine. I'm not sure the comment in pending_writes is ideal now -- we shouldn't really have conversations in our source code :-) If you can fix that up a bit, please merge.