openstack
/
ironic
Code Issues Proposed changes

Followup idrac configuration mold steps 

Followup to change: I10dbc30614d8654dda540ffaa5e61e7a9f03c7b3

Change-Id: Ie7cb2c831854be288553dc634d5367739519e0e1
This commit is contained in:
Aija Jauntēva 2021-03-31 11:19:20 -04:00
parent ff4c370d4b
commit ae30139af2
2 changed files with 14 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
ironic/drivers/modules/drac/management.py
View File

@ -359,7 +359,7 @@ class DracRedfishManagement(redfish_management.RedfishManagement):
if not system.managers:
raise exception.DracOperationError(
error=(_("No managers found for %(node)s"),
error=(_("No managers found for %(node)s") %
{'node': task.node.uuid}))
for manager in system.managers:
@ -400,7 +400,7 @@ class DracRedfishManagement(redfish_management.RedfishManagement):
configuration)
else:
raise exception.DracOperationError(
error=(_("No configuration exported for node %(node)s"),
error=(_("No configuration exported for node %(node)s") %
{'node': task.node.uuid}))
@base.deploy_step(priority=0, argsinfo=IMPORT_CONFIGURATION_ARGSINFO)
@ -428,7 +428,7 @@ class DracRedfishManagement(redfish_management.RedfishManagement):
if not configuration:
raise exception.DracOperationError(
error=(_("No configuration found for node %(node)s by name "
"%(configuration_name)s"),
"%(configuration_name)s") %
{'node': task.node.uuid,
'configuration_name': import_configuration_location}))
@ -437,7 +437,7 @@ class DracRedfishManagement(redfish_management.RedfishManagement):
raise exception.DracOperationError(
error=(_("Invalid configuration for node %(node)s "
"in %(configuration_name)s. Supports only "
"idrac-redfish, but found %(interface)s"),
"idrac-redfish, but found %(interface)s") %
{'node': task.node.uuid,
'configuration_name': import_configuration_location,
'interface': interface}))
@ -446,7 +446,7 @@ class DracRedfishManagement(redfish_management.RedfishManagement):
if not system.managers:
raise exception.DracOperationError(
error=(_("No managers found for %(node)s"),
error=(_("No managers found for %(node)s") %
{'node': task.node.uuid}))
for manager in system.managers:
@ -492,7 +492,7 @@ class DracRedfishManagement(redfish_management.RedfishManagement):
continue
raise exception.DracOperationError(
error=(_("Failed to import configuration for node %(node)s"),
error=(_("Failed to import configuration for node %(node)s") %
{'node': task.node.uuid}))
@base.clean_step(priority=0,

24
ironic/tests/unit/drivers/modules/drac/test_management.py
View File

@ -847,8 +847,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
def test_export_configuration_no_managers(self, mock_get_system):
task = mock.Mock(node=self.node, context=self.context)
fake_system = mock.Mock(managers=[])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = []
self.assertRaises(exception.DracOperationError,
self.management.export_configuration, task, 'edge')
@ -861,8 +860,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
fake_manager1 = mock.Mock()
fake_manager1.get_oem_extension.side_effect = (
sushy.exceptions.OEMExtensionNotFoundError)
fake_system = mock.Mock(managers=[fake_manager1])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = [fake_manager1]
self.assertRaises(exception.RedfishError,
self.management.export_configuration, task, 'edge')
@ -883,8 +881,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
sushy.exceptions.SushyError)
fake_manager2 = mock.Mock()
fake_manager2.get_oem_extension.return_value = fake_manager_oem2
fake_system = mock.Mock(managers=[fake_manager1, fake_manager2])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = [fake_manager1, fake_manager2]
self.assertRaises(exception.DracOperationError,
self.management.export_configuration,
@ -899,8 +896,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
fake_manager_oem1.export_system_configuration.status_code = 500
fake_manager1 = mock.Mock()
fake_manager1.get_oem_extension.return_value = fake_manager_oem1
fake_system = mock.Mock(managers=[fake_manager1])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = [fake_manager1]
self.assertRaises(exception.DracOperationError,
self.management.export_configuration, task, 'edge')
@ -926,8 +922,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
configuration)
fake_manager2 = mock.Mock()
fake_manager2.get_oem_extension.return_value = fake_manager_oem2
fake_system = mock.Mock(managers=[fake_manager1, fake_manager2])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = [fake_manager1, fake_manager2]
self.management.export_configuration(task, 'edge')
mock_save_configuration.assert_called_once_with(
@ -1002,8 +997,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
sushy.exceptions.SushyError)
fake_manager2 = mock.Mock()
fake_manager2.get_oem_extension.return_value = fake_manager_oem2
fake_system = mock.Mock(managers=[fake_manager1, fake_manager2])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = [fake_manager1, fake_manager2]
mock_get_configuration.return_value = json.loads(
'{"oem": {"interface": "idrac-redfish", '
'"data": {"prop1": "value1", "prop2": 2}}}')
@ -1020,8 +1014,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
fake_manager_oem1 = mock.Mock()
fake_manager1 = mock.Mock()
fake_manager1.get_oem_extension.return_value = fake_manager_oem1
fake_system = mock.Mock(managers=[fake_manager1])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = [fake_manager1]
mock_get_configuration.return_value = json.loads(
'{"oem": {"interface": "idrac-wsman", '
'"data": {"prop1": "value1", "prop2": 2}}}')
@ -1054,8 +1047,7 @@ class DracRedfishManagementTestCase(test_utils.BaseDracTest):
fake_manager_oem2 = mock.Mock()
fake_manager2 = mock.Mock()
fake_manager2.get_oem_extension.return_value = fake_manager_oem2
fake_system = mock.Mock(managers=[fake_manager1, fake_manager2])
mock_get_system.return_value = fake_system
mock_get_system.return_value.managers = [fake_manager1, fake_manager2]
mock_get_configuration.return_value = json.loads(
'{"oem": {"interface": "idrac-redfish", '
'"data": {"prop1": "value1", "prop2": 2}}}')