diff options
author | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-06-11 16:04:59 +0200 |
---|---|---|
committer | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-06-14 11:52:06 +0000 |
commit | 2645240b6eac992014eea39d245545407ae75234 (patch) | |
tree | c43e1fccb4dae871e0c1d61d2b52128791c6d8d8 | |
parent | 474d2a91b06942a72c59d0f25ddd65fc255a3d16 (diff) | |
download | ironic-python-agent-2645240b6eac992014eea39d245545407ae75234.tar.gz |
Reduce logging verbosity when collecting logs
It's not uncommon that some commands fail when collecting logs.
We already log all failures in utils.execute, no need to duplicate
them with a non-fatal ERROR logging.
Change-Id: If151b3a3be979bd2b3ce01030e5d6242ad74eaa3
(cherry picked from commit 2fcf35e56d578d98d77ffc97b6441f83548d7704)
-rw-r--r-- | ironic_python_agent/utils.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ironic_python_agent/utils.py b/ironic_python_agent/utils.py index 637737dd..e2a1e5ce 100644 --- a/ironic_python_agent/utils.py +++ b/ironic_python_agent/utils.py @@ -342,7 +342,6 @@ def get_command_output(command): except (processutils.ProcessExecutionError, OSError) as e: error_msg = ('Failed to get the output of the command "%(command)s". ' 'Error: %(error)s' % {'command': command, 'error': e}) - LOG.error(error_msg) raise errors.CommandExecutionError(error_msg) return io.BytesIO(out) @@ -411,7 +410,7 @@ def collect_system_logs(journald_max_lines=None): try: io_dict[file_name] = get_command_output(command) except errors.CommandExecutionError: - pass + LOG.debug('Collecting logs from command %s has failed', command) io_dict = {} file_list = [] |