From: Thanh Ha <thanh.ha@linuxfoundation.org>
Date: Thu, 18 Oct 2018 17:39:59 +0000 (-0400)
Subject: Fix get-credentials subcommand
X-Git-Tag: v0.18.0^2
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F62%2F13062%2F1;p=releng%2Flftools.git

Fix get-credentials subcommand

When we refactored the Jenkins module in
I2168adf9bc992b719da6c0350a446830015e6df6 we forgot to refactor
the get-credentials command so it broke.

Change-Id: I1ee04d2753327557e4d7e5e017a3d6ca7741fef0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
---

diff --git a/lftools/cli/jenkins/__init__.py b/lftools/cli/jenkins/__init__.py
index 681d1f1d..25905cad 100644
--- a/lftools/cli/jenkins/__init__.py
+++ b/lftools/cli/jenkins/__init__.py
@@ -45,7 +45,7 @@ def jenkins_cli(ctx, server, user, password):
 @click.pass_context
 def get_credentials(ctx):
     """Print all available Credentials."""
-    server = ctx.obj['server']
+    jenkins = ctx.obj['jenkins']
     groovy_script = """
 import com.cloudbees.plugins.credentials.*
 
@@ -63,7 +63,7 @@ for (c in creds) {
     } catch (MissingPropertyException) {}
 }
 """
-    result = server.run_script(groovy_script)
+    result = jenkins.server.run_script(groovy_script)
     log.info(result)
 
 
diff --git a/releasenotes/notes/fix-get-credentials-6759fee7366c5602.yaml b/releasenotes/notes/fix-get-credentials-6759fee7366c5602.yaml
new file mode 100644
index 00000000..dc916d9a
--- /dev/null
+++ b/releasenotes/notes/fix-get-credentials-6759fee7366c5602.yaml
@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    The get-credentials command is now fixed since it was was broken after
+    refactoring done in Gerrit patch I2168adf9bc992b719da6c0350a446830015e6df6.