Merge lp:~jelmer/launchpad/oops-on-pool-overwrite-error into lp:launchpad
Proposed by
Jelmer Vernooij
Status: | Merged |
---|---|
Approved by: | Jelmer Vernooij |
Approved revision: | no longer in the source branch. |
Merged at revision: | 11191 |
Proposed branch: | lp:~jelmer/launchpad/oops-on-pool-overwrite-error |
Merge into: | lp:launchpad |
Prerequisite: | lp:~jelmer/launchpad/robust-process-accepted |
Diff against target: |
126 lines (+39/-20) 3 files modified
lib/lp/archiveuploader/tests/test_uploadprocessor.py (+10/-0) lib/lp/soyuz/model/publishing.py (+19/-20) lib/lp/soyuz/tests/test_publishing.py (+10/-0) |
To merge this branch: | bzr merge lp:~jelmer/launchpad/oops-on-pool-overwrite-error |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brad Crittenden (community) | code | Approve | |
Review via email: mp+23328@code.launchpad.net |
Commit message
File an OOPS rather than just logging an error when encountering a file with a different checksum in process-accepted.
Description of the change
Previously process-accepted would just log errors when it encountered files with a different checksum. This patch makes it file OOPSes instead.
To post a comment you must log in.
Thanks for the change and the clarification on IRC. Please do consider using our tools to help create better merge proposal messages.
In your test you have at least one comment that needs final punctuation.