From 65b76aa3cd847fdf8f8a312188b8b2e00371fb3c Mon Sep 17 00:00:00 2001 From: Pádraig Brady Date: Mon, 30 Jul 2012 13:43:27 +0100 Subject: reduce debugging from utils.trycmd() In the case where the command fails, we're returning the err anyway and the caller can log if required. In the discard_warnings=True case, then the caller has explicitly asked to suppress the warnings, so don't log this noise either. Fixes bug: 1030609 Change-Id: I075c21aa6508182ae6dd030cf41ec31916559957 --- nova/utils.py | 2 -- 1 file changed, 2 deletions(-) (limited to 'nova') diff --git a/nova/utils.py b/nova/utils.py index f1de74549..e5b4fe2d2 100644 --- a/nova/utils.py +++ b/nova/utils.py @@ -223,12 +223,10 @@ def trycmd(*args, **kwargs): failed = False except exception.ProcessExecutionError, exn: out, err = '', str(exn) - LOG.debug(err) failed = True if not failed and discard_warnings and err: # Handle commands that output to stderr but otherwise succeed - LOG.debug(err) err = '' return out, err -- cgit