Merge lp:~barryprice/swoffsite/trunk into lp:swoffsite

Proposed by Barry Price
Status: Merged
Approved by: James Simpson
Approved revision: 52
Merged at revision: 52
Proposed branch: lp:~barryprice/swoffsite/trunk
Merge into: lp:swoffsite
Diff against target: 11 lines (+1/-0)
1 file modified
swoffsite/mirror.py (+1/-0)
To merge this branch: bzr merge lp:~barryprice/swoffsite/trunk
Reviewer Review Type Date Requested Status
James Simpson Approve
Canonical IS Reviewers Pending
Review via email: mp+456393@code.launchpad.net

Commit message

Log when we stall for insufficient tmp space LP:2044897

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
James Simpson (jsimpso) wrote :

LGTM

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 52

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'swoffsite/mirror.py'
2--- swoffsite/mirror.py 2023-10-11 03:43:54 +0000
3+++ swoffsite/mirror.py 2023-11-28 02:40:57 +0000
4@@ -264,6 +264,7 @@
5 # Now we have the lock, wait until there is enough temp
6 # space to continue.
7 while not _has_tmp_space(size*1.5):
8+ log.warn('Insufficient tmp space to continue, sleeping 10s')
9 time.sleep(10)
10
11 log.info('Sending {}/{} to S3'.format(container, name))

Subscribers

People subscribed via source and target branches

to all changes: