PPA deletion leaves unremoved publications
Bug #685076 reported by
William Grant
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
William Grant | ||
Bug Description
Deleting a PPA sets all of its publications to Deleted, disables the archive and its publication, and the publisher then deletes the whole archive tree from disk. But the publications are left alive, with no date of removal or even scheduled removal date. The publisher doesn't consider disabled archives, so it never condemns the deleted publications and they remain live in the DB forever.
This has librarian disk space consequences, as the lack of dateremoved will make any involved files ineligible for expiration.
Related branches
lp:~wgrant/launchpad/archive-defer-deletion
- Steve Kowalik (community): Approve (code)
-
Diff: 324 lines (+110/-52)7 files modifiedlib/lp/archivepublisher/publishing.py (+29/-0)
lib/lp/archivepublisher/tests/test_publisher.py (+32/-0)
lib/lp/soyuz/doc/archive-deletion.txt (+9/-31)
lib/lp/soyuz/interfaces/publishing.py (+5/-3)
lib/lp/soyuz/model/archive.py (+13/-10)
lib/lp/soyuz/model/publishing.py (+16/-8)
lib/lp/soyuz/tests/test_publishing.py (+6/-0)
| Changed in soyuz: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| tags: | added: soyuz-publisher |
| tags: |
added: soyuz-publish removed: soyuz-publisher |
Revision history for this message
| Robert Collins (lifeless) wrote | #1 |
Revision history for this message
| Robert Collins (lifeless) wrote | #2 |
| Changed in launchpad: | |
| importance: | High → Low |
Revision history for this message
| Julian Edwards (julian-edwards) wrote | #3 |
| Changed in launchpad: | |
| assignee: | nobody → William Grant (wgrant) |
| status: | Triaged → In Progress |
Revision history for this message
| William Grant (wgrant) wrote | #4 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| 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.
Do we have an estimate of the overhead this has?