From 55e31c4a98c54e97dc81e3a7f5c10d072b7c52c4 Mon Sep 17 00:00:00 2001 From: Anil Belur Date: Thu, 2 Sep 2021 12:55:57 +1000 Subject: [PATCH] Fix: Incorrect indent skips calling deploy_s3() When Unix style glob patterns are passed through the CLI option the deploy_s3() function skips the step to push logs to the S3 bucket. Issue-Id: RELENG-3866 Change-Id: I58d6ef8dab41213fdbc431294ddd21854adc60dc Signed-off-by: Anil Belur --- lftools/cli/deploy.py | 2 +- releasenotes/notes/fix-deploy-s3-indent-76a7ad7eb5ca3c21.yaml | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/fix-deploy-s3-indent-76a7ad7eb5ca3c21.yaml diff --git a/lftools/cli/deploy.py b/lftools/cli/deploy.py index ff01576d..27b1b192 100644 --- a/lftools/cli/deploy.py +++ b/lftools/cli/deploy.py @@ -148,7 +148,7 @@ def s3(ctx, s3_bucket, s3_path, build_url, workspace, pattern): """Deploy logs and archives to a S3 bucket.""" if not pattern: pattern = None - deploy_sys.deploy_s3(s3_bucket, s3_path, build_url, workspace, pattern) + deploy_sys.deploy_s3(s3_bucket, s3_path, build_url, workspace, pattern) log.info("Logs upload to S3 complete.") diff --git a/releasenotes/notes/fix-deploy-s3-indent-76a7ad7eb5ca3c21.yaml b/releasenotes/notes/fix-deploy-s3-indent-76a7ad7eb5ca3c21.yaml new file mode 100644 index 00000000..c2accb99 --- /dev/null +++ b/releasenotes/notes/fix-deploy-s3-indent-76a7ad7eb5ca3c21.yaml @@ -0,0 +1,7 @@ +--- +fixes: + - | + Fix: Indent line correctly to call deploy s3 function. + + When Unix style glob patterns are passed through the CLI option + the deploy_s3() function skips the step to push logs to the S3 bucket. -- 2.16.6