OOPS processing Mantis bugwatches
Bug #218384 reported by
Diogo Matsubara
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
Tim Penhey | ||
Bug Description
As seen in OOPS-834CCW59, when something goes wrong processing Mantis bugwatches, instead of logging the error, we're raising an OOPS trying to call an non-existent self.warning() method. The exception is similar to bug 194039
Exception type AttributeError
Exception value 'Mantis' object has no attribute 'warning'
Related branches
lp:~thumper/launchpad/fix-mantis-warnings-bug-218384
- Robert Collins (community): Approve
- William Grant: Approve (code*)
-
Diff: 464 lines (+221/-81)7 files modifiedlib/lp/bugs/doc/externalbugtracker-trac-lp-plugin.txt (+2/-2)
lib/lp/bugs/doc/externalbugtracker-trac.txt (+4/-4)
lib/lp/bugs/externalbugtracker/base.py (+3/-3)
lib/lp/bugs/externalbugtracker/mantis.py (+72/-68)
lib/lp/bugs/externalbugtracker/tests/test_externalbugtracker.py (+28/-2)
lib/lp/bugs/externalbugtracker/tests/test_mantis.py (+110/-0)
lib/lp/bugs/tests/externalbugtracker.py (+2/-2)
| Changed in malone: | |
| importance: | Undecided → High |
Revision history for this message
| Björn Tillenius (bjornt) wrote | #1 |
| Changed in malone: | |
| importance: | High → Medium |
| milestone: | none → 1.2.5 |
| status: | New → Confirmed |
| Changed in malone: | |
| milestone: | 1.2.5 → 1.2.6 |
| Changed in malone: | |
| milestone: | 1.2.6 → none |
| tags: | added: story-reliable-bug-syncing |
| Changed in launchpad: | |
| importance: | Medium → Critical |
| Changed in launchpad: | |
| assignee: | nobody → Tim Penhey (thumper) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| Changed in launchpad: | |
| milestone: | none → 11.03 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: bad-commit-12487 qa-bad removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| tags: |
added: qa-needstesting removed: qa-bad |
| tags: |
added: qa-untestable removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Reducing the priority to medium, since this seems to affect only a small number of bug watches.