openstack
/
ironic-python-agent
Code Issues Proposed changes

Correct invalid docstrings; s/Found/Error/ 

It's clear the docstrings intended to indicate ProtectedDeviceError, but
instead indicate ProtectedDeviceFound.

This clears up the documentation to assist deployers trying to implement
hardware managers.

Change-Id: Iea534f58aeec60b3862099c1d89be829654a54c1
This commit is contained in:
Jay Faulkner 2024-03-06 08:32:48 -08:00
parent df7eccd7f1
commit a2f997e19b
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ironic_python_agent/hardware.py
View File

@ -964,7 +964,7 @@ class HardwareManager(object, metaclass=abc.ABCMeta):
:param node: Ironic node object
:param ports: list of Ironic port objects
:raises: ProtectedDeviceFound if a device has been identified which
:raises: ProtectedDeviceError if a device has been identified which
may require manual intervention due to the contents and
operational risk which exists as it could also be a sign
of an environmental misconfiguration.
@ -1748,7 +1748,7 @@ class GenericHardwareManager(HardwareManager):
:param ports: list of Ironic port objects
:raises BlockDeviceEraseError: when there's an error erasing the
block device
:raises: ProtectedDeviceFound if a device has been identified which
:raises: ProtectedDeviceError if a device has been identified which
may require manual intervention due to the contents and
operational risk which exists as it could also be a sign
of an environmental misconfiguration.
@ -1779,7 +1779,7 @@ class GenericHardwareManager(HardwareManager):
:param ports: list of Ironic port objects
:raises BlockDeviceEraseError: when there's an error erasing the
block device
:raises: ProtectedDeviceFound if a device has been identified which
:raises: ProtectedDeviceError if a device has been identified which
may require manual intervention due to the contents and
operational risk which exists as it could also be a sign
of an environmental misconfiguration.