summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZuul <zuul@review.opendev.org>2022-06-27 12:10:31 +0000
committerGerrit Code Review <review@openstack.org>2022-06-27 12:10:31 +0000
commit312e1527ab9eaca7b74da7b23c160617aba8ffb3 (patch)
tree0308224056a45588f024d707133f9ef1035584a3
parent63c19c6f93c0b80845b4cf918e62c1fd34747e90 (diff)
parentb68fa6b2e13774dbde5d0b81f6bf0b0095536aa4 (diff)
downloadironic-python-agent-312e1527ab9eaca7b74da7b23c160617aba8ffb3.tar.gz
Merge "Warn when smartctl not found"
-rw-r--r--ironic_python_agent/hardware.py3
1 files changed, 2 insertions, 1 deletions
diff --git a/ironic_python_agent/hardware.py b/ironic_python_agent/hardware.py
index 36b853ed..79f4a426 100644
--- a/ironic_python_agent/hardware.py
+++ b/ironic_python_agent/hardware.py
@@ -1800,10 +1800,11 @@ class GenericHardwareManager(HardwareManager):
'ATA commands via the `smartctl` utility with device '
'%s do not succeed.', block_device.name)
return False
- except OSError:
+ except OSError as e:
# Processutils can raise OSError if a path is not found,
# and it is okay that we tollerate that since it was the
# prior behavior.
+ LOG.warning('Unable to execute `smartctl` utility: %s', e)
return True
def _ata_erase(self, block_device):