diff options
| author | Pádraig Brady <pbrady@redhat.com> | 2012-07-30 13:43:27 +0100 |
|---|---|---|
| committer | Pádraig Brady <pbrady@redhat.com> | 2012-07-30 13:48:45 +0100 |
| commit | 65b76aa3cd847fdf8f8a312188b8b2e00371fb3c (patch) | |
| tree | c9e407c770f053a7b9853145d130230597441ae8 | |
| parent | 90de9055355322f0c49366fe3001fe6f3cbe4b03 (diff) | |
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
| -rw-r--r-- | nova/utils.py | 2 |
1 files changed, 0 insertions, 2 deletions
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 |