diff --git a/doc/source/drivers/ucs.rst b/doc/source/drivers/ucs.rst index f283e87ff5..7cb05f8fac 100644 --- a/doc/source/drivers/ucs.rst +++ b/doc/source/drivers/ucs.rst @@ -25,9 +25,9 @@ Prerequisites to manage Cisco UCS Managed B/C-series servers. Install ``UcsSdk`` [1]_ module on the Ironic conductor node. - Required version is 0.8.1.9:: + Required version is 0.8.2.2:: - $ pip install "UcsSdk==0.8.1.9" + $ pip install "UcsSdk==0.8.2.2" Tested Platforms ~~~~~~~~~~~~~~~~ diff --git a/driver-requirements.txt b/driver-requirements.txt index 5cca29cadb..530b23882b 100644 --- a/driver-requirements.txt +++ b/driver-requirements.txt @@ -10,7 +10,7 @@ pyghmi pysnmp python-scciclient python-seamicroclient>=0.4.0 -UcsSdk==0.8.1.9 +UcsSdk==0.8.2.2 # The drac and amt driver import a python module called "pywsman", however, # this does not exist on pypi. diff --git a/ironic/drivers/modules/ucs/management.py b/ironic/drivers/modules/ucs/management.py index 080ecf0c16..606e6f2dd3 100644 --- a/ironic/drivers/modules/ucs/management.py +++ b/ironic/drivers/modules/ucs/management.py @@ -36,8 +36,10 @@ LOG = logging.getLogger(__name__) UCS_TO_IRONIC_BOOT_DEVICE = { 'storage': boot_devices.DISK, + 'disk': boot_devices.DISK, 'pxe': boot_devices.PXE, - 'read-only-vm': boot_devices.CDROM + 'read-only-vm': boot_devices.CDROM, + 'cdrom': boot_devices.CDROM } @@ -65,7 +67,7 @@ class UcsManagement(base.ManagementInterface): in :mod:`ironic.common.boot_devices`. """ - return list(UCS_TO_IRONIC_BOOT_DEVICE.values()) + return list(set(UCS_TO_IRONIC_BOOT_DEVICE.values())) @ucs_helper.requires_ucs_client def set_boot_device(self, task, device, persistent=False, helper=None): diff --git a/ironic/tests/drivers/ucs/test_management.py b/ironic/tests/drivers/ucs/test_management.py index 1be2ef0285..a16c72bf55 100644 --- a/ironic/tests/drivers/ucs/test_management.py +++ b/ironic/tests/drivers/ucs/test_management.py @@ -66,7 +66,7 @@ class UcsManagementTestCase(db_base.DbTestCase): mock_helper.generate_ucsm_handle.return_value = (True, mock.Mock()) mock_mgmt = mock_ucs_mgmt.return_value mock_mgmt.get_boot_device.return_value = { - 'boot_device': 'storage', + 'boot_device': 'disk', 'persistent': False } with task_manager.acquire(self.context, self.node.uuid,