session pruner should be in tree
Bug #729019 reported by
Stuart Bishop
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Medium
|
Stuart Bishop | ||
Bug Description
Session pruning is done using a crufty script not in the Launchpad tree.
Related branches
lp://qastaging/~stub/launchpad/session-prune
- Graham Binns (community): Approve (code)
-
Diff: 206 lines (+106/-36)2 files modifiedlib/lp/scripts/garbo.py (+29/-16)
lib/lp/scripts/tests/test_garbo.py (+77/-20)
- Launchpad code reviewers: Pending requested
-
Diff: 624 lines (+424/-17)10 files modifieddatabase/schema/launchpad_session.sql (+25/-0)
lib/lp/scripts/garbo.py (+86/-8)
lib/lp/scripts/tests/test_garbo.py (+161/-5)
lib/lp/services/configure.zcml (+1/-0)
lib/lp/services/session/adapters.py (+40/-0)
lib/lp/services/session/configure.zcml (+12/-0)
lib/lp/services/session/interfaces.py (+15/-0)
lib/lp/services/session/model.py (+47/-0)
lib/lp/services/session/tests/test_session.py (+32/-0)
lib/lp/testing/tests/test_standard_test_template.py (+5/-4)
| Changed in launchpad: | |
| status: | New → In Progress |
| importance: | Undecided → Medium |
| assignee: | nobody → Stuart Bishop (stub) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| Changed in launchpad: | |
| milestone: | none → 11.03 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-untestable removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-untestable |
| Changed in launchpad: | |
| milestone: | 11.03 → 11.04 |
Revision history for this message
| Robert Collins (lifeless) wrote | #3 |
| tags: |
added: bad-commit-10259 qa-bad removed: qa-needstesting |
Revision history for this message
| Stuart Bishop (stub) wrote | #4 |
| tags: |
added: qa-needstesting removed: bad-commit-10259 qa-bad |
| 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 db-stable r10259 <http://