diff options
author | Zuul <zuul@review.opendev.org> | 2022-06-27 12:10:31 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2022-06-27 12:10:31 +0000 |
commit | 312e1527ab9eaca7b74da7b23c160617aba8ffb3 (patch) | |
tree | 0308224056a45588f024d707133f9ef1035584a3 | |
parent | 63c19c6f93c0b80845b4cf918e62c1fd34747e90 (diff) | |
parent | b68fa6b2e13774dbde5d0b81f6bf0b0095536aa4 (diff) | |
download | ironic-python-agent-312e1527ab9eaca7b74da7b23c160617aba8ffb3.tar.gz |
Merge "Warn when smartctl not found"
-rw-r--r-- | ironic_python_agent/hardware.py | 3 |
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): |