PackageCopyJob.getActiveJobs() should check job status.
Bug #782096 reported by
Gavin Panella
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Jeroen T. Vermeulen | ||
Bug Description
Currently it returns all jobs, pending, completed or failed, not just "active" jobs. Perhaps it should be renamed also?
Related branches
lp:~jtv/launchpad/db-bug-782096
- j.c.sackett (community): Approve
-
Diff: 364 lines (+219/-10)6 files modifiedlib/lp/registry/browser/distroseries.py (+20/-0)
lib/lp/registry/browser/tests/test_distroseries.py (+27/-0)
lib/lp/registry/templates/distroseries-localdifferences.pt (+13/-7)
lib/lp/soyuz/interfaces/packagecopyjob.py (+15/-0)
lib/lp/soyuz/model/packagecopyjob.py (+43/-2)
lib/lp/soyuz/tests/test_packagecopyjob.py (+101/-1)
Revision history for this message
| Jeroen T. Vermeulen (jtv) wrote | #1 |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Jeroen T. Vermeulen (jtv) |
| milestone: | none → 11.06 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| Changed in launchpad: | |
| milestone: | 11.06 → none |
| 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.
It should also sort by job age to avoid starvation.