Twisted job runner is more complex than needed
Bug #833888 reported by
Aaron Bentley
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
Aaron Bentley | ||
Bug Description
The TwistedJobRunner uses the ParallelLimited
Related branches
lp:~abentley/launchpad/simplify-twisted-runner
- j.c.sackett (community): Approve
-
Diff: 104 lines (+28/-33)2 files modifiedlib/lp/services/job/runner.py (+13/-33)
lib/lp/services/job/tests/test_runner.py (+15/-0)
lp:~abentley/launchpad/simplify-twisted-runner-2
- Deryck Hodge (community): Approve (code)
-
Diff: 142 lines (+55/-20)2 files modifiedlib/lp/services/job/runner.py (+28/-20)
lib/lp/services/job/tests/test_runner.py (+27/-0)
lp:~abentley/launchpad/simplify-twisted-runner-3
- Brad Crittenden (community): Approve (code)
-
Diff: 83 lines (+23/-14)1 file modifiedlib/lp/services/job/runner.py (+23/-14)
| Changed in launchpad: | |
| assignee: | nobody → Aaron Bentley (abentley) |
| status: | Triaged → In Progress |
| tags: | added: tech-debt |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: bad-commit-13801 qa-bad removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-bad |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: bad-commit-13831 qa-bad removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| tags: |
added: qa-needstesting removed: qa-bad |
| 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.
Fixed in stable r13801 <http://