Some valid failures in process-upload should not OOPS
Bug #673590 reported by
Ursula Junque
on 2010-11-10
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Launchpad itself |
Low
|
Julian Edwards | ||
Bug Description
As seen in OOPS-1774FTPMASTER1:
EarlyReturnUp
This is a valid failure but should not oops.
Related branches
lp:~julian-edwards/launchpad/reduce-oops-count-on-uploads-bug-673590
- Robert Collins (community): Approve on 2010-11-23
- Julian Edwards (community): Approve (code) on 2010-11-19
-
Diff: 57 lines (+18/-7)2 files modifiedlib/lp/archiveuploader/nascentupload.py (+17/-6)
lib/lp/archiveuploader/tests/nascentupload.txt (+1/-1)
lp:~julian-edwards/launchpad/upload-early-return-oops
- Julian Edwards (community): Approve on 2010-11-22
-
Diff: 15 lines (+2/-2)1 file modifiedlib/lp/archiveuploader/uploadprocessor.py (+2/-2)
Julian Edwards (julian-edwards)
on 2010-11-10
| Changed in soyuz: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| tags: | added: soyuz-upload |
| Julian Edwards (julian-edwards) wrote : | #1 |
| Julian Edwards (julian-edwards) wrote : | #2 |
Meant to say - any LaunchpadCronScript auto-files OOPSes when logging at "warn" or above.
Julian Edwards (julian-edwards)
on 2010-11-19
| Changed in soyuz: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Julian Edwards (julian-edwards) |
| milestone: | none → 10.12 |
Julian Edwards (julian-edwards)
on 2010-11-22
| Changed in soyuz: | |
| status: | In Progress → Fix Committed |
Fixed in stable r11947 <http://
| tags: | added: qa-needstesting |
Julian Edwards (julian-edwards)
on 2010-11-22
| tags: |
added: qa-ok removed: qa-needstesting |
| Julian Edwards (julian-edwards) wrote : | #4 |
See also OOPS-1785PPA71
The fix for this one is another logger change.
r11959 in stable (http://
| Changed in soyuz: | |
| status: | Fix Committed → In Progress |
| Robert Collins (lifeless) wrote : | #6 |
Not deployed to the uploader machines yet, sorry for the noise.
| Changed in soyuz: | |
| status: | In Progress → Fix Released |
| status: | Fix Released → Fix Committed |
Curtis Hovey (sinzui)
on 2010-12-08
| Changed in soyuz: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Some of the other OOPSes like OOPS-1782PPA118 are caused by the log level being an "error" in do_accept when it catches exceptions and logs them.
This needs to change so that if it's a QueueInconsiste