Merge lp://staging/~cjwatson/txpkgupload/drop-distro-files into lp://staging/~lazr-developers/txpkgupload/trunk

Proposed by Colin Watson
Status: Merged
Merged at revision: 19
Proposed branch: lp://staging/~cjwatson/txpkgupload/drop-distro-files
Merge into: lp://staging/~lazr-developers/txpkgupload/trunk
Diff against target: 826 lines (+95/-462)
8 files modified
etc/txpkgupload.yaml (+5/-1)
src/txpkgupload/glock.py (+0/-316)
src/txpkgupload/hooks.py (+23/-106)
src/txpkgupload/plugin.py (+22/-6)
src/txpkgupload/tests/test_plugin.py (+23/-12)
src/txpkgupload/tests/test_twistedsftp.py (+8/-6)
src/txpkgupload/twistedftp.py (+11/-11)
src/txpkgupload/twistedsftp.py (+3/-4)
To merge this branch: bzr merge lp://staging/~cjwatson/txpkgupload/drop-distro-files
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+247143@code.staging.launchpad.net

Commit message

Institute a temporary directory on the same filesystem as the upload root; stop creating obsolete .distro files; remove locking, since we now move directories into place atomically.

Description of the change

Institute a temporary directory on the same filesystem as the upload root; stop creating obsolete .distro files; remove locking, since we now move directories into place atomically.

Before deploying this, we need to make sure that the tmp-incoming directory is created with appropriate permissions, and it wouldn't hurt to land an lp-production-configs change making the new directory explicit.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: