Merge lp:~rockstar/launchpad/upgrade-branch-rc into lp:launchpad/db-devel
Proposed by
Paul Hummer
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~rockstar/launchpad/upgrade-branch-rc |
Merge into: | lp:launchpad/db-devel |
Diff against target: |
102 lines (+26/-1) 5 files modified
database/schema/security.cfg (+1/-1) lib/lp/code/model/branch.py (+5/-0) lib/lp/code/model/branchjob.py (+2/-0) lib/lp/code/model/tests/test_branch.py (+15/-0) lib/lp/code/model/tests/test_branchjob.py (+3/-0) |
To merge this branch: | bzr merge lp:~rockstar/launchpad/upgrade-branch-rc |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gary Poster (community) | rc | Approve | |
Tim Penhey (community) | Approve | ||
Review via email: mp+18049@code.launchpad.net |
To post a comment you must log in.
Hi-
This branch fixes two release-critical issues with the branch upgrade job upgrade_ pending didn't check to see if
stuff. The first issue is that IBranch.
the upgrade had failed or was finished. In those cases, it should return
false. The other issue is that the BranchUpgradeJob wasn't triggering a mirror
after it got upgraded, so the branch still thought it needed upgrading.
The patch is the tests for those fixes and then the fixes.
Cheers,
Paul