publish-ftpmaster cleanup fails
Bug #754453 reported by
Jeroen T. Vermeulen
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Jeroen T. Vermeulen | ||
Bug Description
The new publish-ftpmaster script's cleanup routine can fail while moving the "dists" directory if another directory is already in the way.
In the old script this failure would have been hidden by "mv" moving one directory into the other instead of replacing it as intended.
Traceback William got: http://
Related branches
lp://staging/~jtv/launchpad/db-bug-752279
- William Grant: Approve (code*)
-
Diff: 826 lines (+281/-255)5 files modifiedcronscripts/publishing/distro-parts/ubuntu/publish-distro.d/10-sign-releases (+0/-1)
cronscripts/publishing/distro-parts/ubuntu/publish-distro.d/20-remove-uncompressed-listings (+1/-1)
cronscripts/publishing/distro-parts/ubuntu/publish-distro.d/README.txt (+2/-2)
lib/lp/archivepublisher/scripts/publish_ftpmaster.py (+181/-126)
lib/lp/archivepublisher/tests/test_publish_ftpmaster.py (+97/-125)
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Jeroen T. Vermeulen (jtv) |
| milestone: | none → 11.05 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Jeroen T. Vermeulen (jtv) wrote | #2 |
| 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 r10430 <http://