From: Thanh Ha <thanh.ha@linuxfoundation.org>
Date: Sat, 8 Sep 2018 18:27:34 +0000 (-0400)
Subject: Use logger instead of print in Jenkins CLI
X-Git-Tag: v0.18.0~11
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F07%2F12507%2F3;p=releng%2Flftools.git

Use logger instead of print in Jenkins CLI

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

diff --git a/lftools/cli/jenkins/__init__.py b/lftools/cli/jenkins/__init__.py
index bbf606f8..9b255e47 100644
--- a/lftools/cli/jenkins/__init__.py
+++ b/lftools/cli/jenkins/__init__.py
@@ -12,6 +12,7 @@
 __author__ = 'Trevor Bramwell'
 
 
+import logging
 import os
 
 import click
@@ -24,6 +25,8 @@ from lftools.cli.jenkins.jobs import jobs
 from lftools.cli.jenkins.nodes import nodes
 from lftools.cli.jenkins.plugins import plugins_init
 
+log = logging.getLogger(__name__)
+
 
 @click.group()
 @click.option(
@@ -81,7 +84,7 @@ for (c in creds) {
 }
 """
     result = server.run_script(groovy_script)
-    print(result)
+    log.info(result)
 
 
 @click.command()
@@ -94,7 +97,7 @@ def groovy(ctx, groovy_file):
 
     server = ctx.obj['server']
     result = server.run_script(data)
-    print(result)
+    log.info(result)
 
 
 @click.command()
@@ -109,9 +112,9 @@ def quiet_down(ctx, n):
             ctx.obj['server'].quiet_down()
         except HTTPError as m:
             if m.code == 405:
-                print("\n[%s]\nJenkins %s does not support Quiet Down "
-                      "without a CSRF Token. (CVE-2017-04-26)\nPlease "
-                      "file a bug with 'python-jenkins'" % (m, version))
+                log.error("\n[%s]\nJenkins %s does not support Quiet Down "
+                          "without a CSRF Token. (CVE-2017-04-26)\nPlease "
+                          "file a bug with 'python-jenkins'" % (m, version))
             else:
                 raise m
 
@@ -180,7 +183,7 @@ for (node in Jenkins.instance.computers) {
         result = server.run_script(force_script)
     else:
         result = server.run_script(groovy_script)
-    print(result)
+    log.info(result)
 
 
 jenkins_cli.add_command(plugins_init, name='plugins')