From 89ed14752babb83dd6835cc341b1c6df7e1e2ff0 Mon Sep 17 00:00:00 2001 From: Ian Wienand Date: Fri, 1 Nov 2019 16:06:22 +1100 Subject: [PATCH] test-swift-log-upload: fix dry_run parameter 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 --- roles/test-upload-logs-swift/library/zuul_swift_upload.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/test-upload-logs-swift/library/zuul_swift_upload.py b/roles/test-upload-logs-swift/library/zuul_swift_upload.py index 0e90f8053..e5f272a37 100755 --- a/roles/test-upload-logs-swift/library/zuul_swift_upload.py +++ b/roles/test-upload-logs-swift/library/zuul_swift_upload.py @@ -649,8 +649,8 @@ class Uploader(): def __init__(self, cloud, container, prefix=None, delete_after=None, public=True, dry_run=False): + self.dry_run = dry_run if dry_run: - self.dry_run = True self.url = 'http://dry-run-url.com/a/path/' return