Use logger instead of print in Jenkins CLI 07/12507/3
authorThanh Ha <thanh.ha@linuxfoundation.org>
Sat, 8 Sep 2018 18:27:34 +0000 (14:27 -0400)
committerThanh Ha <thanh.ha@linuxfoundation.org>
Mon, 10 Sep 2018 21:49:40 +0000 (17:49 -0400)
Change-Id: I6e99c69f75a0be5c0e54da5c21a2930652ad3f8e
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
lftools/cli/jenkins/__init__.py

index bbf606f..9b255e4 100644 (file)
@@ -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')