From fcf7bbb5187c26b6191392ae412ad0b4fa00b80b Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Wed, 15 Nov 2017 17:20:18 -0500 Subject: [PATCH] Add raises note to disk_utils.get_disk_identifier get_disk_identifier can raise OSError when the underlying method that is called, and hexdump is missing. Updating the docstring to convey it appropriately. Change-Id: Ia57bf10e40f89e095154b0d88d40b63d835de03a Related-Bug: #1732470 --- ironic_lib/disk_utils.py | 1 + 1 file changed, 1 insertion(+) diff --git a/ironic_lib/disk_utils.py b/ironic_lib/disk_utils.py index 98c4a451..c8115512 100644 --- a/ironic_lib/disk_utils.py +++ b/ironic_lib/disk_utils.py @@ -140,6 +140,7 @@ def get_disk_identifier(dev): http://www.syslinux.org/wiki/index.php/Comboot/chain.c32#mbr: :param dev: Path for the already populated disk device. + :raises OSError: When the hexdump binary is unavailable. :returns: The Disk Identifier. """ disk_identifier = utils.execute('hexdump', '-s', '440', '-n', '4',