zuul/zuul-jobs
Code Issues Proposed changes

test-swift-log-upload: fix dry_run parameter

Browse Source 
This was pointed out and fixed by jhesketh in
Ie4888606a8ca4ffe2eb99ddbbcd9d5cee8ceec44, but I forgot to add it to
the test role.  Sure enough, it fails as jhesketh predicted.

[1] https://review.opendev.org/#/c/592929/3..4/roles/upload-logs-swift/library/zuul_swift_upload.py

Change-Id: I8d25a234fb92cd1bcfcd0cb7ddf9b8222dbd5978
This commit is contained in:
Ian Wienand 2019-11-01 16:06:22 +11:00
parent 9a8cc2a1c0
commit 89ed14752b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
roles/test-upload-logs-swift/library/zuul_swift_upload.py
View File

@ -649,8 +649,8 @@ class Uploader():
def __init__(self, cloud, container, prefix=None, delete_after=None, def __init__(self, cloud, container, prefix=None, delete_after=None,
public=True, dry_run=False): public=True, dry_run=False):
self.dry_run = dry_run
if dry_run: if dry_run:
self.dry_run = True
self.url = 'http://dry-run-url.com/a/path/' self.url = 'http://dry-run-url.com/a/path/'
return return