Make BeakerLib logging less verbose

Logs from Beaker jobs are normally very brief, with the standard
output/error containing detailed information. Make ipa-run-tests
with BeakerLib plugin follow this convention.

Only include INFO and higher level messages in the Beaker logs.
Downgrade several message levels to DEBUG.
Log to console using Python logging instead of showing the Beaker logs.

Since ipa-run-tests sets up its own logging, Nose's own log
handling just causes duplicate messages. Disable it with --nologcapture.

2 files changed, 8 insertions, 8 deletions

diff --git a/ipatests/test_integration/host.py b/ipatests/test_integration/host.py
index 8109f51aa..9accdf035 100644
--- a/ipatests/test_integration/host.py
+++ b/ipatests/test_integration/host.py
@@ -99,7 +99,7 @@ class RemoteCommand(object):
             self.log.error('Exit code: %s', self.returncode)
             raise subprocess.CalledProcessError(self.returncode, self.argv)
         else:
-            self.log.info('Exit code: %s', self.returncode)
+            self.log.debug('Exit code: %s', self.returncode)
         return self.returncode
 
     def _start_pipe_thread(self, result_list, stream, name, do_log=True):
@@ -108,7 +108,7 @@ class RemoteCommand(object):
         def read_stream():
             for line in stream:
                 if do_log:
-                    log.info(line.rstrip('\n'))
+                    log.debug(line.rstrip('\n'))
                 result_list.append(line)
 
         thread = threading.Thread(target=read_stream)
@@ -285,7 +285,7 @@ class Host(object):
 
     def get_file_contents(self, filename):
         """Read the named remote file and return the contents as a string"""
-        self.log.info('READ %s', filename)
+        self.log.debug('READ %s', filename)
         with self.sftp.open(filename) as f:
             return f.read()
 
@@ -297,7 +297,7 @@ class Host(object):
 
     def file_exists(self, filename):
         """Return true if the named remote file exists"""
-        self.log.info('STAT %s', filename)
+        self.log.debug('STAT %s', filename)
         try:
             self.sftp.stat(filename)
         except IOError, e:
@@ -308,7 +308,7 @@ class Host(object):
         return True
 
     def get_file(self, remotepath, localpath):
-        self.log.info('GET %s', remotepath)
+        self.log.debug('GET %s', remotepath)
         self.sftp.get(remotepath, localpath)
 
     def put_file(self, localpath, remotepath):
diff --git a/ipatests/test_integration/tasks.py b/ipatests/test_integration/tasks.py
index d131e4df6..ecefdab70 100644
--- a/ipatests/test_integration/tasks.py
+++ b/ipatests/test_integration/tasks.py
@@ -64,7 +64,7 @@ def fix_etc_hosts(host):
                       flags=re.MULTILINE)
     # Add the host's info again
     contents += '\n%s %s %s\n' % (host.ip, host.hostname, host.shortname)
-    log.info('Writing the following to /etc/hosts:\n%s', contents)
+    log.debug('Writing the following to /etc/hosts:\n%s', contents)
     host.put_file_contents('/etc/hosts', contents)
 
 
@@ -85,7 +85,7 @@ def fix_resolv_conf(host):
         if other_host.role in ('master', 'replica'):
             lines.append('nameserver %s' % other_host.ip)
     contents = '\n'.join(lines)
-    log.info('Writing the following to /etc/resolv.conf:\n%s', contents)
+    log.debug('Writing the following to /etc/resolv.conf:\n%s', contents)
     host.put_file_contents('/etc/resolv.conf', contents)
 
 
@@ -109,7 +109,7 @@ def restore_hostname(host):
     try:
         hostname = host.get_file_contents(backupname)
     except IOError:
-        log.info('No hostname backed up on %s' % host.hostname)
+        log.debug('No hostname backed up on %s' % host.hostname)
     else:
         host.run_command(['hostname', hostname.strip()])
         host.run_command(['rm', backupname])