Delayed copy publication sometimes crashes when reading changes file content
Bug #761439 reported by
William Grant
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
William Grant | ||
Bug Description
As seen in OOPS-1930PPAPUBLISH1, delayed copy publication sometimes fails to read the changes file. In this case the files had been unembargoed in a previous delayed copy almost a whole second earlier.
The crash leaves the PackageUpload Accepted, and subsequent attempts to process it fail like OOPS-1930PPAPUBLISH10.
Related branches
lp:~wgrant/launchpad/bug-761439
- Steve Kowalik (community): Approve
-
Diff: 90 lines (+47/-0)2 files modifiedlib/lp/soyuz/scripts/processaccepted.py (+9/-0)
lib/lp/soyuz/tests/test_processaccepted.py (+38/-0)
| tags: | added: oops |
Revision history for this message
| William Grant (wgrant) wrote | #1 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| Changed in launchpad: | |
| milestone: | none → 11.05 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
process-accepted completely fails to do any kind of transaction management. PackageUploads that result in an exception do not have their transactions aborted, and no commit is done until the end, so LFAs created by early uploads are unavailable to later ones.