From: Anil Belur <abelur@linuxfoundation.org>
Date: Thu, 2 Sep 2021 02:55:57 +0000 (+1000)
Subject: Fix: Incorrect indent skips calling deploy_s3()
X-Git-Tag: v0.35.10^2
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=55e31c4a98c54e97dc81e3a7f5c10d072b7c52c4;p=releng%2Flftools.git

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 <abelur@linuxfoundation.org>
---

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.