diff options
author | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-06-11 16:04:59 +0200 |
---|---|---|
committer | Julia Kreger <juliaashleykreger@gmail.com> | 2021-06-14 13:34:59 +0000 |
commit | 76d061ea87e633874f19569512d11f3d54d8193d (patch) | |
tree | 6cb2c369c21e50d6ceabcd6bbcb320e2a236efd8 | |
parent | 8287b96628afc38a8667b7a38e16d2a715011a36 (diff) | |
download | ironic-python-agent-76d061ea87e633874f19569512d11f3d54d8193d.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 7b248d7e..b95ccf6f 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 = [] |