f10958a542
Currently, if an attempt is made to fetch MAC address information using OOB inspection on a Redfish-managed node and EthernetInterfaces attribute is missing on the node, inspection fails due to a MissingAttributeError exception being raised by sushy. This change adds catching and handling this exception. Change-Id: I6f16da05e19c7efc966128fdf79f13546f51b5a6
1611 lines
72 KiB
Python
1611 lines
72 KiB
Python
# Copyright 2017 Red Hat, Inc.
|
|
# All Rights Reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
import datetime
|
|
from unittest import mock
|
|
|
|
from oslo_utils import importutils
|
|
from oslo_utils import units
|
|
|
|
from ironic.common import boot_devices
|
|
from ironic.common import boot_modes
|
|
from ironic.common import components
|
|
from ironic.common import exception
|
|
from ironic.common import indicator_states
|
|
from ironic.common import states
|
|
from ironic.conductor import task_manager
|
|
from ironic.conductor import utils as manager_utils
|
|
from ironic.conf import CONF
|
|
from ironic.drivers.modules import deploy_utils
|
|
from ironic.drivers.modules.redfish import boot as redfish_boot
|
|
from ironic.drivers.modules.redfish import firmware_utils
|
|
from ironic.drivers.modules.redfish import management as redfish_mgmt
|
|
from ironic.drivers.modules.redfish import utils as redfish_utils
|
|
from ironic.tests.unit.db import base as db_base
|
|
from ironic.tests.unit.db import utils as db_utils
|
|
from ironic.tests.unit.objects import utils as obj_utils
|
|
|
|
sushy = importutils.try_import('sushy')
|
|
|
|
INFO_DICT = db_utils.get_test_redfish_info()
|
|
|
|
|
|
class RedfishManagementTestCase(db_base.DbTestCase):
|
|
|
|
def setUp(self):
|
|
super(RedfishManagementTestCase, self).setUp()
|
|
self.config(enabled_hardware_types=['redfish'],
|
|
enabled_power_interfaces=['redfish'],
|
|
enabled_boot_interfaces=['redfish-virtual-media'],
|
|
enabled_management_interfaces=['redfish'],
|
|
enabled_inspect_interfaces=['redfish'],
|
|
enabled_bios_interfaces=['redfish'])
|
|
self.node = obj_utils.create_test_node(
|
|
self.context, driver='redfish', driver_info=INFO_DICT)
|
|
|
|
self.system_uuid = 'ZZZ--XXX-YYY'
|
|
self.chassis_uuid = 'XXX-YYY-ZZZ'
|
|
self.drive_uuid = 'ZZZ-YYY-XXX'
|
|
|
|
def init_system_mock(self, system_mock, **properties):
|
|
|
|
system_mock.reset()
|
|
|
|
system_mock.boot.mode = 'uefi'
|
|
|
|
system_mock.memory_summary.size_gib = 2
|
|
|
|
system_mock.processors.summary = '8', 'MIPS'
|
|
|
|
system_mock.simple_storage.disks_sizes_bytes = (
|
|
1 * units.Gi, units.Gi * 3, units.Gi * 5)
|
|
system_mock.storage.volumes_sizes_bytes = (
|
|
2 * units.Gi, units.Gi * 4, units.Gi * 6)
|
|
|
|
system_mock.ethernet_interfaces.summary = {
|
|
'00:11:22:33:44:55': sushy.STATE_ENABLED,
|
|
'66:77:88:99:AA:BB': sushy.STATE_DISABLED,
|
|
}
|
|
|
|
return system_mock
|
|
|
|
@mock.patch.object(redfish_mgmt, 'sushy', None)
|
|
def test_loading_error(self):
|
|
self.assertRaisesRegex(
|
|
exception.DriverLoadError,
|
|
'Unable to import the sushy library',
|
|
redfish_mgmt.RedfishManagement)
|
|
|
|
def test_get_properties(self):
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
properties = task.driver.get_properties()
|
|
for prop in redfish_utils.COMMON_PROPERTIES:
|
|
self.assertIn(prop, properties)
|
|
|
|
@mock.patch.object(redfish_utils, 'parse_driver_info', autospec=True)
|
|
def test_validate(self, mock_parse_driver_info):
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
task.driver.management.validate(task)
|
|
mock_parse_driver_info.assert_called_once_with(task.node)
|
|
|
|
def test_get_supported_boot_devices(self):
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
supported_boot_devices = (
|
|
task.driver.management.get_supported_boot_devices(task))
|
|
self.assertEqual(list(redfish_mgmt.BOOT_DEVICE_MAP_REV),
|
|
supported_boot_devices)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_device(self, mock_get_system):
|
|
fake_system = mock.Mock()
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
expected_values = [
|
|
(boot_devices.PXE, sushy.BOOT_SOURCE_TARGET_PXE),
|
|
(boot_devices.DISK, sushy.BOOT_SOURCE_TARGET_HDD),
|
|
(boot_devices.CDROM, sushy.BOOT_SOURCE_TARGET_CD),
|
|
(boot_devices.BIOS, sushy.BOOT_SOURCE_TARGET_BIOS_SETUP)
|
|
]
|
|
|
|
for target, expected in expected_values:
|
|
task.driver.management.set_boot_device(task, target)
|
|
|
|
# Asserts
|
|
fake_system.set_system_boot_options.assert_has_calls(
|
|
[mock.call(expected,
|
|
enabled=sushy.BOOT_SOURCE_ENABLED_ONCE)])
|
|
mock_get_system.assert_called_with(task.node)
|
|
self.assertNotIn('redfish_boot_device',
|
|
task.node.driver_internal_info)
|
|
|
|
# Reset mocks
|
|
fake_system.set_system_boot_options.reset_mock()
|
|
mock_get_system.reset_mock()
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_device_persistency(self, mock_get_system):
|
|
fake_system = mock.Mock()
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
expected_values = [
|
|
(True, sushy.BOOT_SOURCE_ENABLED_CONTINUOUS),
|
|
(False, sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
]
|
|
|
|
for target, expected in expected_values:
|
|
task.driver.management.set_boot_device(
|
|
task, boot_devices.PXE, persistent=target)
|
|
|
|
fake_system.set_system_boot_options.assert_has_calls(
|
|
[mock.call(sushy.BOOT_SOURCE_TARGET_PXE,
|
|
enabled=expected)])
|
|
mock_get_system.assert_called_with(task.node)
|
|
self.assertNotIn('redfish_boot_device',
|
|
task.node.driver_internal_info)
|
|
|
|
# Reset mocks
|
|
fake_system.set_system_boot_options.reset_mock()
|
|
mock_get_system.reset_mock()
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_device_persistency_no_change(self, mock_get_system):
|
|
fake_system = mock.Mock()
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
expected_values = [
|
|
(True, sushy.BOOT_SOURCE_ENABLED_CONTINUOUS),
|
|
(False, sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
]
|
|
|
|
for target, expected in expected_values:
|
|
fake_system.boot.get.return_value = expected
|
|
|
|
task.driver.management.set_boot_device(
|
|
task, boot_devices.PXE, persistent=target)
|
|
|
|
fake_system.set_system_boot_options.assert_has_calls(
|
|
[mock.call(sushy.BOOT_SOURCE_TARGET_PXE, enabled=None)])
|
|
mock_get_system.assert_called_with(task.node)
|
|
|
|
# Reset mocks
|
|
fake_system.set_system_boot_options.reset_mock()
|
|
mock_get_system.reset_mock()
|
|
|
|
@mock.patch.object(sushy, 'Sushy', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_device_fail(self, mock_get_system, mock_sushy):
|
|
fake_system = mock.Mock()
|
|
fake_system.set_system_boot_options.side_effect = (
|
|
sushy.exceptions.SushyError()
|
|
)
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
self.assertRaisesRegex(
|
|
exception.RedfishError, 'Redfish set boot device',
|
|
task.driver.management.set_boot_device, task, boot_devices.PXE)
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
sushy.BOOT_SOURCE_TARGET_PXE,
|
|
enabled=sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
self.assertNotIn('redfish_boot_device',
|
|
task.node.driver_internal_info)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_device_fail_no_change(self, mock_get_system):
|
|
fake_system = mock.Mock()
|
|
fake_system.set_system_boot_options.side_effect = (
|
|
sushy.exceptions.SushyError()
|
|
)
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
expected_values = [
|
|
(True, sushy.BOOT_SOURCE_ENABLED_CONTINUOUS),
|
|
(False, sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
]
|
|
|
|
for target, expected in expected_values:
|
|
fake_system.boot.get.return_value = expected
|
|
|
|
self.assertRaisesRegex(
|
|
exception.RedfishError, 'Redfish set boot device',
|
|
task.driver.management.set_boot_device, task,
|
|
boot_devices.PXE, persistent=target)
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
sushy.BOOT_SOURCE_TARGET_PXE, enabled=None)
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
self.assertNotIn('redfish_boot_device',
|
|
task.node.driver_internal_info)
|
|
|
|
# Reset mocks
|
|
fake_system.set_system_boot_options.reset_mock()
|
|
mock_get_system.reset_mock()
|
|
|
|
@mock.patch.object(sushy, 'Sushy', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_device_persistence_fallback(self, mock_get_system,
|
|
mock_sushy):
|
|
fake_system = mock.Mock()
|
|
fake_system.set_system_boot_options.side_effect = [
|
|
sushy.exceptions.SushyError(),
|
|
None,
|
|
None
|
|
]
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.driver.management.set_boot_device(
|
|
task, boot_devices.PXE, persistent=True)
|
|
fake_system.set_system_boot_options.assert_has_calls([
|
|
mock.call(sushy.BOOT_SOURCE_TARGET_PXE,
|
|
enabled=sushy.BOOT_SOURCE_ENABLED_CONTINUOUS),
|
|
mock.call(sushy.BOOT_SOURCE_TARGET_PXE,
|
|
enabled=sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
])
|
|
mock_get_system.assert_called_with(task.node)
|
|
|
|
task.node.refresh()
|
|
self.assertEqual(
|
|
boot_devices.PXE,
|
|
task.node.driver_internal_info['redfish_boot_device'])
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_device_persistency_vendor(self, mock_get_system):
|
|
fake_system = mock_get_system.return_value
|
|
fake_system.boot.get.return_value = \
|
|
sushy.BOOT_SOURCE_ENABLED_CONTINUOUS
|
|
|
|
values = [
|
|
('SuperMicro', sushy.BOOT_SOURCE_ENABLED_CONTINUOUS),
|
|
('SomeVendor', None)
|
|
]
|
|
|
|
for vendor, expected in values:
|
|
properties = self.node.properties
|
|
properties['vendor'] = vendor
|
|
self.node.properties = properties
|
|
self.node.save()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.driver.management.set_boot_device(
|
|
task, boot_devices.PXE, persistent=True)
|
|
if vendor == 'SuperMicro':
|
|
fake_system.set_system_boot_options.assert_has_calls(
|
|
[mock.call(sushy.BOOT_SOURCE_TARGET_PXE,
|
|
enabled=expected),
|
|
mock.call(mode=sushy.BOOT_SOURCE_MODE_UEFI)])
|
|
else:
|
|
fake_system.set_system_boot_options.assert_has_calls(
|
|
[mock.call(sushy.BOOT_SOURCE_TARGET_PXE,
|
|
enabled=expected)])
|
|
|
|
# Reset mocks
|
|
fake_system.set_system_boot_options.reset_mock()
|
|
mock_get_system.reset_mock()
|
|
|
|
def test_restore_boot_device(self):
|
|
fake_system = mock.Mock()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.node.driver_internal_info['redfish_boot_device'] = (
|
|
boot_devices.DISK
|
|
)
|
|
|
|
task.driver.management.restore_boot_device(task, fake_system)
|
|
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
sushy.BOOT_SOURCE_TARGET_HDD,
|
|
enabled=sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
# The stored boot device is kept intact
|
|
self.assertEqual(
|
|
boot_devices.DISK,
|
|
task.node.driver_internal_info['redfish_boot_device'])
|
|
|
|
def test_restore_boot_device_compat(self):
|
|
fake_system = mock.Mock()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
# Previously we used sushy constants
|
|
task.node.driver_internal_info['redfish_boot_device'] = "hdd"
|
|
|
|
task.driver.management.restore_boot_device(task, fake_system)
|
|
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
sushy.BOOT_SOURCE_TARGET_HDD,
|
|
enabled=sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
# The stored boot device is kept intact
|
|
self.assertEqual(
|
|
"hdd",
|
|
task.node.driver_internal_info['redfish_boot_device'])
|
|
|
|
def test_restore_boot_device_noop(self):
|
|
fake_system = mock.Mock()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.driver.management.restore_boot_device(task, fake_system)
|
|
|
|
self.assertFalse(fake_system.set_system_boot_options.called)
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'warning', autospec=True)
|
|
def test_restore_boot_device_failure(self, mock_log):
|
|
fake_system = mock.Mock()
|
|
fake_system.set_system_boot_options.side_effect = (
|
|
sushy.exceptions.SushyError()
|
|
)
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.node.driver_internal_info['redfish_boot_device'] = (
|
|
boot_devices.DISK
|
|
)
|
|
|
|
task.driver.management.restore_boot_device(task, fake_system)
|
|
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
sushy.BOOT_SOURCE_TARGET_HDD,
|
|
enabled=sushy.BOOT_SOURCE_ENABLED_ONCE)
|
|
self.assertTrue(mock_log.called)
|
|
# The stored boot device is kept intact
|
|
self.assertEqual(
|
|
boot_devices.DISK,
|
|
task.node.driver_internal_info['redfish_boot_device'])
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_boot_device(self, mock_get_system):
|
|
boot_attribute = {
|
|
'target': sushy.BOOT_SOURCE_TARGET_PXE,
|
|
'enabled': sushy.BOOT_SOURCE_ENABLED_CONTINUOUS
|
|
}
|
|
fake_system = mock.Mock(boot=boot_attribute)
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
response = task.driver.management.get_boot_device(task)
|
|
expected = {'boot_device': boot_devices.PXE,
|
|
'persistent': True}
|
|
self.assertEqual(expected, response)
|
|
|
|
def test_get_supported_boot_modes(self):
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
supported_boot_modes = (
|
|
task.driver.management.get_supported_boot_modes(task))
|
|
self.assertEqual(list(redfish_mgmt.BOOT_MODE_MAP_REV),
|
|
supported_boot_modes)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_mode(self, mock_get_system):
|
|
boot_attribute = {
|
|
'target': sushy.BOOT_SOURCE_TARGET_PXE,
|
|
'enabled': sushy.BOOT_SOURCE_ENABLED_CONTINUOUS,
|
|
'mode': sushy.BOOT_SOURCE_MODE_BIOS,
|
|
}
|
|
fake_system = mock.Mock(boot=boot_attribute)
|
|
fake_system = mock.Mock()
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
expected_values = [
|
|
(boot_modes.LEGACY_BIOS, sushy.BOOT_SOURCE_MODE_BIOS),
|
|
(boot_modes.UEFI, sushy.BOOT_SOURCE_MODE_UEFI)
|
|
]
|
|
|
|
for mode, expected in expected_values:
|
|
task.driver.management.set_boot_mode(task, mode=mode)
|
|
|
|
# Asserts
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
mode=expected)
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
|
|
# Reset mocks
|
|
fake_system.set_system_boot_options.reset_mock()
|
|
mock_get_system.reset_mock()
|
|
|
|
@mock.patch.object(sushy, 'Sushy', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_mode_fail(self, mock_get_system, mock_sushy):
|
|
boot_attribute = {
|
|
'target': sushy.BOOT_SOURCE_TARGET_PXE,
|
|
'enabled': sushy.BOOT_SOURCE_ENABLED_CONTINUOUS,
|
|
'mode': sushy.BOOT_SOURCE_MODE_BIOS,
|
|
}
|
|
fake_system = mock.Mock(boot=boot_attribute)
|
|
fake_system.set_system_boot_options.side_effect = (
|
|
sushy.exceptions.SushyError)
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
self.assertRaisesRegex(
|
|
exception.RedfishError, 'Setting boot mode',
|
|
task.driver.management.set_boot_mode, task, boot_modes.UEFI)
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
mode=sushy.BOOT_SOURCE_MODE_UEFI)
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
|
|
@mock.patch.object(sushy, 'Sushy', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_boot_mode_unsupported(self, mock_get_system, mock_sushy):
|
|
boot_attribute = {
|
|
'target': sushy.BOOT_SOURCE_TARGET_PXE,
|
|
'enabled': sushy.BOOT_SOURCE_ENABLED_CONTINUOUS,
|
|
}
|
|
fake_system = mock.Mock(boot=boot_attribute)
|
|
error = sushy.exceptions.BadRequestError('PATCH', '/', mock.Mock())
|
|
fake_system.set_system_boot_options.side_effect = error
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
self.assertRaisesRegex(
|
|
exception.UnsupportedDriverExtension,
|
|
'does not support set_boot_mode',
|
|
task.driver.management.set_boot_mode, task, boot_modes.UEFI)
|
|
fake_system.set_system_boot_options.assert_called_once_with(
|
|
mode=sushy.BOOT_SOURCE_MODE_UEFI)
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_boot_mode(self, mock_get_system):
|
|
boot_attribute = {
|
|
'target': sushy.BOOT_SOURCE_TARGET_PXE,
|
|
'enabled': sushy.BOOT_SOURCE_ENABLED_CONTINUOUS,
|
|
'mode': sushy.BOOT_SOURCE_MODE_BIOS,
|
|
}
|
|
fake_system = mock.Mock(boot=boot_attribute)
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
response = task.driver.management.get_boot_mode(task)
|
|
expected = boot_modes.LEGACY_BIOS
|
|
self.assertEqual(expected, response)
|
|
|
|
def test__get_sensors_fan(self):
|
|
attributes = {
|
|
"identity": "XXX-YYY-ZZZ",
|
|
"name": "CPU Fan",
|
|
"status": {
|
|
"state": "enabled",
|
|
"health": "OK"
|
|
},
|
|
"reading": 6000,
|
|
"reading_units": "RPM",
|
|
"lower_threshold_fatal": 2000,
|
|
"min_reading_range": 0,
|
|
"max_reading_range": 10000,
|
|
"serial_number": "SN010203040506",
|
|
"physical_context": "CPU"
|
|
}
|
|
|
|
mock_chassis = mock.MagicMock(identity='ZZZ-YYY-XXX')
|
|
|
|
mock_fan = mock.MagicMock(**attributes)
|
|
mock_fan.name = attributes['name']
|
|
mock_fan.status = mock.MagicMock(**attributes['status'])
|
|
mock_chassis.thermal.fans = [mock_fan]
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
sensors = task.driver.management._get_sensors_fan(mock_chassis)
|
|
|
|
expected = {
|
|
'XXX-YYY-ZZZ@ZZZ-YYY-XXX': {
|
|
'identity': 'XXX-YYY-ZZZ',
|
|
'max_reading_range': 10000,
|
|
'min_reading_range': 0,
|
|
'physical_context': 'CPU',
|
|
'reading': 6000,
|
|
'reading_units': 'RPM',
|
|
'serial_number': 'SN010203040506',
|
|
'health': 'OK',
|
|
'state': 'enabled'
|
|
}
|
|
}
|
|
|
|
self.assertEqual(expected, sensors)
|
|
|
|
def test__get_sensors_temperatures(self):
|
|
attributes = {
|
|
"identity": "XXX-YYY-ZZZ",
|
|
"name": "CPU Temp",
|
|
"status": {
|
|
"state": "enabled",
|
|
"health": "OK"
|
|
},
|
|
"reading_celsius": 62,
|
|
"upper_threshold_non_critical": 75,
|
|
"upper_threshold_critical": 90,
|
|
"upperThresholdFatal": 95,
|
|
"min_reading_range_temp": 0,
|
|
"max_reading_range_temp": 120,
|
|
"physical_context": "CPU",
|
|
"sensor_number": 1
|
|
}
|
|
|
|
mock_chassis = mock.MagicMock(identity='ZZZ-YYY-XXX')
|
|
|
|
mock_temperature = mock.MagicMock(**attributes)
|
|
mock_temperature.name = attributes['name']
|
|
mock_temperature.status = mock.MagicMock(**attributes['status'])
|
|
mock_chassis.thermal.temperatures = [mock_temperature]
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
sensors = task.driver.management._get_sensors_temperatures(
|
|
mock_chassis)
|
|
|
|
expected = {
|
|
'XXX-YYY-ZZZ@ZZZ-YYY-XXX': {
|
|
'identity': 'XXX-YYY-ZZZ',
|
|
'max_reading_range_temp': 120,
|
|
'min_reading_range_temp': 0,
|
|
'physical_context': 'CPU',
|
|
'reading_celsius': 62,
|
|
'sensor_number': 1,
|
|
'health': 'OK',
|
|
'state': 'enabled'
|
|
}
|
|
}
|
|
|
|
self.assertEqual(expected, sensors)
|
|
|
|
def test__get_sensors_power(self):
|
|
attributes = {
|
|
'identity': 0,
|
|
'name': 'Power Supply 0',
|
|
'power_capacity_watts': 1450,
|
|
'last_power_output_watts': 650,
|
|
'line_input_voltage': 220,
|
|
'input_ranges': {
|
|
'minimum_voltage': 185,
|
|
'maximum_voltage': 250,
|
|
'minimum_frequency_hz': 47,
|
|
'maximum_frequency_hz': 63,
|
|
'output_wattage': 1450
|
|
},
|
|
'serial_number': 'SN010203040506',
|
|
"status": {
|
|
"state": "enabled",
|
|
"health": "OK"
|
|
}
|
|
}
|
|
|
|
mock_chassis = mock.MagicMock(identity='ZZZ-YYY-XXX')
|
|
mock_power = mock_chassis.power
|
|
mock_power.identity = 'Power'
|
|
mock_psu = mock.MagicMock(**attributes)
|
|
mock_psu.name = attributes['name']
|
|
mock_psu.status = mock.MagicMock(**attributes['status'])
|
|
mock_psu.input_ranges = mock.MagicMock(**attributes['input_ranges'])
|
|
mock_power.power_supplies = [mock_psu]
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
sensors = task.driver.management._get_sensors_power(mock_chassis)
|
|
|
|
expected = {
|
|
'0:Power@ZZZ-YYY-XXX': {
|
|
'health': 'OK',
|
|
'last_power_output_watts': 650,
|
|
'line_input_voltage': 220,
|
|
'maximum_frequency_hz': 63,
|
|
'maximum_voltage': 250,
|
|
'minimum_frequency_hz': 47,
|
|
'minimum_voltage': 185,
|
|
'output_wattage': 1450,
|
|
'power_capacity_watts': 1450,
|
|
'serial_number': 'SN010203040506',
|
|
'state': 'enabled'
|
|
}
|
|
}
|
|
|
|
self.assertEqual(expected, sensors)
|
|
|
|
def test__get_sensors_data_drive(self):
|
|
attributes = {
|
|
'name': '32ADF365C6C1B7BD',
|
|
'manufacturer': 'IBM',
|
|
'model': 'IBM 350A',
|
|
'capacity_bytes': 3750000000,
|
|
'status': {
|
|
'health': 'OK',
|
|
'state': 'enabled'
|
|
}
|
|
}
|
|
|
|
mock_system = mock.MagicMock(identity='ZZZ-YYY-XXX')
|
|
mock_drive = mock.MagicMock(**attributes)
|
|
mock_drive.name = attributes['name']
|
|
mock_drive.status = mock.MagicMock(**attributes['status'])
|
|
mock_storage = mock.MagicMock()
|
|
mock_storage.devices = [mock_drive]
|
|
mock_storage.identity = 'XXX-YYY-ZZZ'
|
|
mock_system.simple_storage.get_members.return_value = [mock_storage]
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
sensors = task.driver.management._get_sensors_drive(mock_system)
|
|
|
|
expected = {
|
|
'32ADF365C6C1B7BD:XXX-YYY-ZZZ@ZZZ-YYY-XXX': {
|
|
'capacity_bytes': 3750000000,
|
|
'health': 'OK',
|
|
'name': '32ADF365C6C1B7BD',
|
|
'model': 'IBM 350A',
|
|
'state': 'enabled'
|
|
}
|
|
}
|
|
|
|
self.assertEqual(expected, sensors)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_sensors_data(self, mock_system):
|
|
mock_chassis = mock.MagicMock()
|
|
mock_system.return_value.chassis = [mock_chassis]
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
sensors = task.driver.management.get_sensors_data(task)
|
|
|
|
expected = {
|
|
'Fan': {},
|
|
'Temperature': {},
|
|
'Power': {},
|
|
'Drive': {}
|
|
}
|
|
|
|
self.assertEqual(expected, sensors)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_inject_nmi(self, mock_get_system):
|
|
fake_system = mock.Mock()
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.driver.management.inject_nmi(task)
|
|
fake_system.reset_system.assert_called_once_with(sushy.RESET_NMI)
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
|
|
@mock.patch.object(sushy, 'Sushy', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_inject_nmi_fail(self, mock_get_system, mock_sushy):
|
|
fake_system = mock.Mock()
|
|
fake_system.reset_system.side_effect = (
|
|
sushy.exceptions.SushyError)
|
|
mock_get_system.return_value = fake_system
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
self.assertRaisesRegex(
|
|
exception.RedfishError, 'Redfish inject NMI',
|
|
task.driver.management.inject_nmi, task)
|
|
fake_system.reset_system.assert_called_once_with(
|
|
sushy.RESET_NMI)
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_supported_indicators(self, mock_get_system):
|
|
fake_chassis = mock.Mock(
|
|
uuid=self.chassis_uuid,
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
fake_drive = mock.Mock(
|
|
uuid=self.drive_uuid,
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
fake_system = mock.Mock(
|
|
uuid=self.system_uuid,
|
|
chassis=[fake_chassis],
|
|
simple_storage=mock.MagicMock(drives=[fake_drive]),
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
|
|
mock_get_system.return_value = fake_system
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
|
|
supported_indicators = (
|
|
task.driver.management.get_supported_indicators(task))
|
|
|
|
expected = {
|
|
components.CHASSIS: {
|
|
'XXX-YYY-ZZZ': {
|
|
"readonly": False,
|
|
"states": [
|
|
indicator_states.BLINKING,
|
|
indicator_states.OFF,
|
|
indicator_states.ON
|
|
]
|
|
}
|
|
},
|
|
components.SYSTEM: {
|
|
'ZZZ--XXX-YYY': {
|
|
"readonly": False,
|
|
"states": [
|
|
indicator_states.BLINKING,
|
|
indicator_states.OFF,
|
|
indicator_states.ON
|
|
]
|
|
}
|
|
},
|
|
components.DISK: {
|
|
'ZZZ-YYY-XXX': {
|
|
"readonly": False,
|
|
"states": [
|
|
indicator_states.BLINKING,
|
|
indicator_states.OFF,
|
|
indicator_states.ON
|
|
]
|
|
}
|
|
}
|
|
}
|
|
|
|
self.assertEqual(expected, supported_indicators)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_indicator_state(self, mock_get_system):
|
|
fake_chassis = mock.Mock(
|
|
uuid=self.chassis_uuid,
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
fake_drive = mock.Mock(
|
|
uuid=self.drive_uuid,
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
fake_system = mock.Mock(
|
|
uuid=self.system_uuid,
|
|
chassis=[fake_chassis],
|
|
simple_storage=mock.MagicMock(drives=[fake_drive]),
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
|
|
mock_get_system.return_value = fake_system
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.driver.management.set_indicator_state(
|
|
task, components.SYSTEM, self.system_uuid, indicator_states.ON)
|
|
|
|
fake_system.set_indicator_led.assert_called_once_with(
|
|
sushy.INDICATOR_LED_LIT)
|
|
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_indicator_state(self, mock_get_system):
|
|
fake_chassis = mock.Mock(
|
|
uuid=self.chassis_uuid,
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
fake_drive = mock.Mock(
|
|
uuid=self.drive_uuid,
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
fake_system = mock.Mock(
|
|
uuid=self.system_uuid,
|
|
chassis=[fake_chassis],
|
|
simple_storage=mock.MagicMock(drives=[fake_drive]),
|
|
indicator_led=sushy.INDICATOR_LED_LIT)
|
|
|
|
mock_get_system.return_value = fake_system
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
|
|
state = task.driver.management.get_indicator_state(
|
|
task, components.SYSTEM, self.system_uuid)
|
|
|
|
mock_get_system.assert_called_once_with(task.node)
|
|
|
|
self.assertEqual(indicator_states.ON, state)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_detect_vendor(self, mock_get_system):
|
|
mock_get_system.return_value.manufacturer = "Fake GmbH"
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
response = task.driver.management.detect_vendor(task)
|
|
self.assertEqual("Fake GmbH", response)
|
|
|
|
@mock.patch.object(deploy_utils, 'build_agent_options',
|
|
spec_set=True, autospec=True)
|
|
@mock.patch.object(redfish_boot.RedfishVirtualMediaBoot, 'prepare_ramdisk',
|
|
spec_set=True, autospec=True)
|
|
@mock.patch.object(manager_utils, 'node_power_action', autospec=True)
|
|
@mock.patch.object(deploy_utils, 'get_async_step_return_state',
|
|
autospec=True)
|
|
@mock.patch.object(deploy_utils, 'set_async_step_flags', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
def test_update_firmware(self, mock_get_update_service,
|
|
mock_set_async_step_flags,
|
|
mock_get_async_step_return_state,
|
|
mock_node_power_action, mock_prepare,
|
|
build_mock):
|
|
build_mock.return_value = {'a': 'b'}
|
|
mock_task_monitor = mock.Mock()
|
|
mock_task_monitor.task_monitor_uri = '/task/123'
|
|
mock_update_service = mock.Mock()
|
|
mock_update_service.simple_update.return_value = mock_task_monitor
|
|
mock_get_update_service.return_value = mock_update_service
|
|
CONF.set_override('firmware_source', 'http', 'redfish')
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.node.save = mock.Mock()
|
|
|
|
task.driver.management.update_firmware(
|
|
task,
|
|
[{'url': 'http://test1',
|
|
'checksum': 'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434d'},
|
|
{'url': 'http://test2',
|
|
'checksum': '9f6227549221920e312fed2cfc6586ee832cc546'}])
|
|
|
|
mock_get_update_service.assert_called_once_with(task.node)
|
|
mock_update_service.simple_update.assert_called_once_with(
|
|
'http://test1')
|
|
self.assertIsNotNone(task.node
|
|
.driver_internal_info['firmware_updates'])
|
|
self.assertEqual(
|
|
[{'task_monitor': '/task/123', 'url': 'http://test1',
|
|
'checksum': 'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434d'},
|
|
{'url': 'http://test2',
|
|
'checksum': '9f6227549221920e312fed2cfc6586ee832cc546'}],
|
|
task.node.driver_internal_info['firmware_updates'])
|
|
self.assertIsNone(
|
|
task.node.driver_internal_info.get('firmware_cleanup'))
|
|
mock_set_async_step_flags.assert_called_once_with(
|
|
task.node, reboot=True, skip_current_step=True, polling=True)
|
|
mock_get_async_step_return_state.assert_called_once_with(
|
|
task.node)
|
|
mock_node_power_action.assert_called_once_with(task, states.REBOOT)
|
|
|
|
@mock.patch.object(redfish_mgmt.RedfishManagement, '_stage_firmware_file',
|
|
autospec=True)
|
|
@mock.patch.object(deploy_utils, 'build_agent_options',
|
|
spec_set=True, autospec=True)
|
|
@mock.patch.object(redfish_boot.RedfishVirtualMediaBoot, 'prepare_ramdisk',
|
|
spec_set=True, autospec=True)
|
|
@mock.patch.object(manager_utils, 'node_power_action', autospec=True)
|
|
@mock.patch.object(deploy_utils, 'get_async_step_return_state',
|
|
autospec=True)
|
|
@mock.patch.object(deploy_utils, 'set_async_step_flags', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
def test_update_firmware_stage(
|
|
self, mock_get_update_service, mock_set_async_step_flags,
|
|
mock_get_async_step_return_state, mock_node_power_action,
|
|
mock_prepare, build_mock, mock_stage):
|
|
build_mock.return_value = {'a': 'b'}
|
|
mock_task_monitor = mock.Mock()
|
|
mock_task_monitor.task_monitor_uri = '/task/123'
|
|
mock_update_service = mock.Mock()
|
|
mock_update_service.simple_update.return_value = mock_task_monitor
|
|
mock_get_update_service.return_value = mock_update_service
|
|
mock_stage.return_value = ('http://staged/test1', 'http')
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.node.save = mock.Mock()
|
|
|
|
task.driver.management.update_firmware(
|
|
task,
|
|
[{'url': 'http://test1',
|
|
'checksum': 'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434d'},
|
|
{'url': 'http://test2',
|
|
'checksum': '9f6227549221920e312fed2cfc6586ee832cc546'}])
|
|
|
|
mock_get_update_service.assert_called_once_with(task.node)
|
|
mock_update_service.simple_update.assert_called_once_with(
|
|
'http://staged/test1')
|
|
self.assertIsNotNone(task.node
|
|
.driver_internal_info['firmware_updates'])
|
|
self.assertEqual(
|
|
[{'task_monitor': '/task/123', 'url': 'http://test1',
|
|
'checksum': 'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434d'},
|
|
{'url': 'http://test2',
|
|
'checksum': '9f6227549221920e312fed2cfc6586ee832cc546'}],
|
|
task.node.driver_internal_info['firmware_updates'])
|
|
self.assertIsNotNone(
|
|
task.node.driver_internal_info['firmware_cleanup'])
|
|
self.assertEqual(
|
|
['http'], task.node.driver_internal_info['firmware_cleanup'])
|
|
mock_set_async_step_flags.assert_called_once_with(
|
|
task.node, reboot=True, skip_current_step=True, polling=True)
|
|
mock_get_async_step_return_state.assert_called_once_with(
|
|
task.node)
|
|
mock_node_power_action.assert_called_once_with(task, states.REBOOT)
|
|
|
|
@mock.patch.object(redfish_mgmt.RedfishManagement, '_stage_firmware_file',
|
|
autospec=True)
|
|
@mock.patch.object(deploy_utils, 'build_agent_options',
|
|
spec_set=True, autospec=True)
|
|
@mock.patch.object(redfish_boot.RedfishVirtualMediaBoot, 'prepare_ramdisk',
|
|
spec_set=True, autospec=True)
|
|
@mock.patch.object(manager_utils, 'node_power_action', autospec=True)
|
|
@mock.patch.object(deploy_utils, 'get_async_step_return_state',
|
|
autospec=True)
|
|
@mock.patch.object(deploy_utils, 'set_async_step_flags', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
def test_update_firmware_stage_both(
|
|
self, mock_get_update_service, mock_set_async_step_flags,
|
|
mock_get_async_step_return_state, mock_node_power_action,
|
|
mock_prepare, build_mock, mock_stage):
|
|
build_mock.return_value = {'a': 'b'}
|
|
mock_task_monitor = mock.Mock()
|
|
mock_task_monitor.task_monitor_uri = '/task/123'
|
|
mock_update_service = mock.Mock()
|
|
mock_update_service.simple_update.return_value = mock_task_monitor
|
|
mock_get_update_service.return_value = mock_update_service
|
|
mock_stage.return_value = ('http://staged/test1', 'http')
|
|
info = self.node.driver_internal_info
|
|
info['firmware_cleanup'] = ['swift']
|
|
self.node.driver_internal_info = info
|
|
self.node.save()
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.node.save = mock.Mock()
|
|
|
|
task.driver.management.update_firmware(
|
|
task,
|
|
[{'url': 'http://test1',
|
|
'checksum': 'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434d'},
|
|
{'url': 'http://test2',
|
|
'checksum': '9f6227549221920e312fed2cfc6586ee832cc546'}])
|
|
|
|
mock_get_update_service.assert_called_once_with(task.node)
|
|
mock_update_service.simple_update.assert_called_once_with(
|
|
'http://staged/test1')
|
|
self.assertIsNotNone(task.node
|
|
.driver_internal_info['firmware_updates'])
|
|
self.assertEqual(
|
|
[{'task_monitor': '/task/123', 'url': 'http://test1',
|
|
'checksum': 'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434d'},
|
|
{'url': 'http://test2',
|
|
'checksum': '9f6227549221920e312fed2cfc6586ee832cc546'}],
|
|
task.node.driver_internal_info['firmware_updates'])
|
|
self.assertIsNotNone(
|
|
task.node.driver_internal_info['firmware_cleanup'])
|
|
self.assertEqual(
|
|
['swift', 'http'],
|
|
task.node.driver_internal_info['firmware_cleanup'])
|
|
mock_set_async_step_flags.assert_called_once_with(
|
|
task.node, reboot=True, skip_current_step=True, polling=True)
|
|
mock_get_async_step_return_state.assert_called_once_with(
|
|
task.node)
|
|
mock_node_power_action.assert_called_once_with(task, states.REBOOT)
|
|
|
|
def test_update_firmware_invalid_args(self):
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
self.assertRaises(
|
|
exception.InvalidParameterValue,
|
|
task.driver.management.update_firmware,
|
|
task, [{'urlX': 'test1'}, {'url': 'test2'}])
|
|
|
|
@mock.patch.object(task_manager, 'acquire', autospec=True)
|
|
def test__query_firmware_update_failed(self, mock_acquire):
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
mock_manager = mock.Mock()
|
|
node_list = [(self.node.uuid, 'redfish', '', driver_internal_info)]
|
|
mock_manager.iter_nodes.return_value = node_list
|
|
task = mock.Mock(node=self.node,
|
|
driver=mock.Mock(management=management))
|
|
mock_acquire.return_value = mock.MagicMock(
|
|
__enter__=mock.MagicMock(return_value=task))
|
|
management._clear_firmware_updates = mock.Mock()
|
|
|
|
management._query_firmware_update_failed(mock_manager,
|
|
self.context)
|
|
|
|
management._clear_firmware_updates.assert_called_once_with(self.node)
|
|
|
|
@mock.patch.object(task_manager, 'acquire', autospec=True)
|
|
def test__query_firmware_update_failed_no_firmware_upd(self, mock_acquire):
|
|
driver_internal_info = {'something': 'else'}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
mock_manager = mock.Mock()
|
|
node_list = [(self.node.uuid, 'redfish', '', driver_internal_info)]
|
|
mock_manager.iter_nodes.return_value = node_list
|
|
task = mock.Mock(node=self.node,
|
|
driver=mock.Mock(management=management))
|
|
mock_acquire.return_value = mock.MagicMock(
|
|
__enter__=mock.MagicMock(return_value=task))
|
|
management._clear_firmware_updates = mock.Mock()
|
|
|
|
management._query_firmware_update_failed(mock_manager,
|
|
self.context)
|
|
|
|
management._clear_firmware_updates.assert_not_called()
|
|
|
|
@mock.patch.object(task_manager, 'acquire', autospec=True)
|
|
def test__query_firmware_update_status(self, mock_acquire):
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
mock_manager = mock.Mock()
|
|
node_list = [(self.node.uuid, 'redfish', '', driver_internal_info)]
|
|
mock_manager.iter_nodes.return_value = node_list
|
|
task = mock.Mock(node=self.node,
|
|
driver=mock.Mock(management=management))
|
|
mock_acquire.return_value = mock.MagicMock(
|
|
__enter__=mock.MagicMock(return_value=task))
|
|
management._check_node_firmware_update = mock.Mock()
|
|
|
|
management._query_firmware_update_status(mock_manager,
|
|
self.context)
|
|
|
|
management._check_node_firmware_update.assert_called_once_with(task)
|
|
|
|
@mock.patch.object(task_manager, 'acquire', autospec=True)
|
|
def test__query_firmware_update_status_no_firmware_upd(self, mock_acquire):
|
|
driver_internal_info = {'something': 'else'}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
mock_manager = mock.Mock()
|
|
node_list = [(self.node.uuid, 'redfish', '', driver_internal_info)]
|
|
mock_manager.iter_nodes.return_value = node_list
|
|
task = mock.Mock(node=self.node,
|
|
driver=mock.Mock(management=management))
|
|
mock_acquire.return_value = mock.MagicMock(
|
|
__enter__=mock.MagicMock(return_value=task))
|
|
management._check_node_firmware_update = mock.Mock()
|
|
|
|
management._query_firmware_update_status(mock_manager,
|
|
self.context)
|
|
|
|
management._check_node_firmware_update.assert_not_called()
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'warning', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
def test__check_node_firmware_update_redfish_conn_error(
|
|
self, mock_get_update_services, mock_log):
|
|
mock_get_update_services.side_effect = exception.RedfishConnectionError
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
|
|
management = redfish_mgmt.RedfishManagement()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._check_node_firmware_update(task)
|
|
|
|
self.assertTrue(mock_log.called)
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'debug', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
def test__check_node_firmware_update_wait_elapsed(
|
|
self, mock_get_update_service, mock_log):
|
|
mock_update_service = mock.Mock()
|
|
mock_get_update_service.return_value = mock_update_service
|
|
|
|
wait_start_time = datetime.datetime.utcnow() -\
|
|
datetime.timedelta(minutes=15)
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1',
|
|
'wait_start_time':
|
|
wait_start_time.isoformat(),
|
|
'wait': 1}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
management._continue_firmware_updates = mock.Mock()
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._check_node_firmware_update(task)
|
|
|
|
self.assertTrue(mock_log.called)
|
|
management._continue_firmware_updates.assert_called_once_with(
|
|
task,
|
|
mock_update_service,
|
|
[{'task_monitor': '/task/123', 'url': 'test1'}])
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'debug', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
def test__check_node_firmware_update_still_waiting(
|
|
self, mock_get_update_service, mock_log):
|
|
mock_update_service = mock.Mock()
|
|
mock_get_update_service.return_value = mock_update_service
|
|
|
|
wait_start_time = datetime.datetime.utcnow() -\
|
|
datetime.timedelta(minutes=1)
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1',
|
|
'wait_start_time':
|
|
wait_start_time.isoformat(),
|
|
'wait': 600}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
management._continue_firmware_updates = mock.Mock()
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._check_node_firmware_update(task)
|
|
|
|
self.assertTrue(mock_log.called)
|
|
management._continue_firmware_updates.assert_not_called()
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'warning', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_task_monitor', autospec=True)
|
|
def test__check_node_firmware_update_task_monitor_not_found(
|
|
self, mock_task_monitor, mock_get_update_service, mock_log):
|
|
mock_task_monitor.side_effect = exception.RedfishError()
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
management._continue_firmware_updates = mock.Mock()
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._check_node_firmware_update(task)
|
|
|
|
self.assertTrue(mock_log.called)
|
|
management._continue_firmware_updates.assert_called_once_with(
|
|
task,
|
|
mock_get_update_service.return_value,
|
|
[{'task_monitor': '/task/123', 'url': 'test1'}])
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'debug', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_task_monitor', autospec=True)
|
|
def test__check_node_firmware_update_in_progress(self,
|
|
mock_get_task_monitor,
|
|
mock_get_update_service,
|
|
mock_log):
|
|
mock_get_task_monitor.return_value.is_processing = True
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._check_node_firmware_update(task)
|
|
|
|
self.assertTrue(mock_log.called)
|
|
|
|
@mock.patch.object(manager_utils, 'cleaning_error_handler', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_task_monitor', autospec=True)
|
|
def test__check_node_firmware_update_fail(self,
|
|
mock_get_task_monitor,
|
|
mock_get_update_service,
|
|
mock_cleaning_error_handler):
|
|
mock_sushy_task = mock.Mock()
|
|
mock_sushy_task.task_state = 'exception'
|
|
mock_message_unparsed = mock.Mock()
|
|
mock_message_unparsed.message = None
|
|
mock_message = mock.Mock()
|
|
mock_message.message = 'Firmware upgrade failed'
|
|
messages = mock.PropertyMock(side_effect=[[mock_message_unparsed],
|
|
[mock_message],
|
|
[mock_message]])
|
|
type(mock_sushy_task).messages = messages
|
|
mock_task_monitor = mock.Mock()
|
|
mock_task_monitor.is_processing = False
|
|
mock_task_monitor.get_task.return_value = mock_sushy_task
|
|
mock_get_task_monitor.return_value = mock_task_monitor
|
|
driver_internal_info = {'something': 'else',
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
management._continue_firmware_updates = mock.Mock()
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.upgrade_lock = mock.Mock()
|
|
task.process_event = mock.Mock()
|
|
|
|
management._check_node_firmware_update(task)
|
|
|
|
task.upgrade_lock.assert_called_once_with()
|
|
self.assertEqual({'something': 'else'},
|
|
task.node.driver_internal_info)
|
|
mock_cleaning_error_handler.assert_called_once()
|
|
management._continue_firmware_updates.assert_not_called()
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'info', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_update_service', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_task_monitor', autospec=True)
|
|
def test__check_node_firmware_update_done(self,
|
|
mock_get_task_monitor,
|
|
mock_get_update_service,
|
|
mock_log):
|
|
mock_task = mock.Mock()
|
|
mock_task.task_state = sushy.TASK_STATE_COMPLETED
|
|
mock_task.task_status = sushy.HEALTH_OK
|
|
mock_message = mock.Mock()
|
|
mock_message.message = 'Firmware update done'
|
|
mock_task.messages = [mock_message]
|
|
mock_task_monitor = mock.Mock()
|
|
mock_task_monitor.is_processing = False
|
|
mock_task_monitor.get_task.return_value = mock_task
|
|
mock_get_task_monitor.return_value = mock_task_monitor
|
|
driver_internal_info = {
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123',
|
|
'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
management = redfish_mgmt.RedfishManagement()
|
|
management._continue_firmware_updates = mock.Mock()
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._check_node_firmware_update(task)
|
|
|
|
self.assertTrue(mock_log.called)
|
|
management._continue_firmware_updates.assert_called_once_with(
|
|
task,
|
|
mock_get_update_service.return_value,
|
|
[{'task_monitor': '/task/123',
|
|
'url': 'test1'}])
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'debug', autospec=True)
|
|
def test__continue_firmware_updates_wait(self, mock_log):
|
|
mock_update_service = mock.Mock()
|
|
|
|
management = redfish_mgmt.RedfishManagement()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._continue_firmware_updates(
|
|
task,
|
|
mock_update_service,
|
|
[{'task_monitor': '/task/123',
|
|
'url': 'test1',
|
|
'wait': 10,
|
|
'wait_start_time': '20200901123045'},
|
|
{'url': 'test2'}])
|
|
|
|
self.assertTrue(mock_log.called)
|
|
# Wait start time has changed
|
|
self.assertNotEqual(
|
|
'20200901123045',
|
|
task.node.driver_internal_info['firmware_updates']
|
|
[0]['wait_start_time'])
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'info', autospec=True)
|
|
@mock.patch.object(manager_utils, 'notify_conductor_resume_clean',
|
|
autospec=True)
|
|
def test__continue_firmware_updates_last_update(
|
|
self,
|
|
mock_notify_conductor_resume_clean,
|
|
mock_log):
|
|
mock_update_service = mock.Mock()
|
|
driver_internal_info = {
|
|
'something': 'else',
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123', 'url': 'test1'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
self.node.save()
|
|
|
|
management = redfish_mgmt.RedfishManagement()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
management._continue_firmware_updates(
|
|
task,
|
|
mock_update_service,
|
|
[{'task_monitor': '/task/123', 'url': 'test1'}])
|
|
|
|
self.assertTrue(mock_log.called)
|
|
mock_notify_conductor_resume_clean.assert_called_once_with(task)
|
|
self.assertEqual({'something': 'else'},
|
|
task.node.driver_internal_info)
|
|
|
|
@mock.patch.object(redfish_mgmt.LOG, 'debug', autospec=True)
|
|
@mock.patch.object(manager_utils, 'node_power_action', autospec=True)
|
|
def test__continue_firmware_updates_more_updates(self,
|
|
mock_node_power_action,
|
|
mock_log):
|
|
mock_task_monitor = mock.Mock()
|
|
mock_task_monitor.task_monitor_uri = '/task/987'
|
|
mock_update_service = mock.Mock()
|
|
mock_update_service.simple_update.return_value = mock_task_monitor
|
|
driver_internal_info = {
|
|
'something': 'else',
|
|
'firmware_updates': [
|
|
{'task_monitor': '/task/123', 'url': 'http://test1'},
|
|
{'url': 'http://test2'}]}
|
|
self.node.driver_internal_info = driver_internal_info
|
|
CONF.set_override('firmware_source', 'http', 'redfish')
|
|
|
|
management = redfish_mgmt.RedfishManagement()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=False) as task:
|
|
task.node.save = mock.Mock()
|
|
|
|
management._continue_firmware_updates(
|
|
task,
|
|
mock_update_service,
|
|
[{'task_monitor': '/task/123', 'url': 'http://test1'},
|
|
{'url': 'http://test2'}])
|
|
|
|
self.assertTrue(mock_log.called)
|
|
mock_update_service.simple_update.assert_called_once_with(
|
|
'http://test2')
|
|
self.assertIsNotNone(
|
|
task.node.driver_internal_info['firmware_updates'])
|
|
self.assertEqual(
|
|
[{'url': 'http://test2', 'task_monitor': '/task/987'}],
|
|
task.node.driver_internal_info['firmware_updates'])
|
|
task.node.save.assert_called_once_with()
|
|
mock_node_power_action.assert_called_once_with(task, states.REBOOT)
|
|
|
|
@mock.patch.object(firmware_utils, 'download_to_temp', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'verify_checksum', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'stage', autospec=True)
|
|
def test__stage_firmware_file_https(self, mock_stage, mock_verify_checksum,
|
|
mock_download_to_temp):
|
|
CONF.set_override('firmware_source', 'local', 'redfish')
|
|
firmware_update = {'url': 'https://test1', 'checksum': 'abc'}
|
|
node = mock.Mock()
|
|
mock_download_to_temp.return_value = '/tmp/test1'
|
|
mock_stage.return_value = ('http://staged/test1', 'http')
|
|
|
|
management = redfish_mgmt.RedfishManagement()
|
|
|
|
staged_url, needs_cleanup = management._stage_firmware_file(
|
|
node, firmware_update)
|
|
|
|
self.assertEqual(staged_url, 'http://staged/test1')
|
|
self.assertEqual(needs_cleanup, 'http')
|
|
mock_download_to_temp.assert_called_with(node, 'https://test1')
|
|
mock_verify_checksum.assert_called_with(node, 'abc', '/tmp/test1')
|
|
mock_stage.assert_called_with(node, 'local', '/tmp/test1')
|
|
|
|
@mock.patch.object(firmware_utils, 'download_to_temp', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'verify_checksum', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'stage', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'get_swift_temp_url', autospec=True)
|
|
def test__stage_firmware_file_swift(
|
|
self, mock_get_swift_temp_url, mock_stage, mock_verify_checksum,
|
|
mock_download_to_temp):
|
|
CONF.set_override('firmware_source', 'swift', 'redfish')
|
|
firmware_update = {'url': 'swift://container/bios.exe'}
|
|
node = mock.Mock()
|
|
mock_get_swift_temp_url.return_value = 'http://temp'
|
|
|
|
management = redfish_mgmt.RedfishManagement()
|
|
|
|
staged_url, needs_cleanup = management._stage_firmware_file(
|
|
node, firmware_update)
|
|
|
|
self.assertEqual(staged_url, 'http://temp')
|
|
self.assertIsNone(needs_cleanup)
|
|
mock_download_to_temp.assert_not_called()
|
|
mock_verify_checksum.assert_not_called()
|
|
mock_stage.assert_not_called()
|
|
|
|
@mock.patch.object(firmware_utils, 'cleanup', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'download_to_temp', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'verify_checksum', autospec=True)
|
|
@mock.patch.object(firmware_utils, 'stage', autospec=True)
|
|
def test__stage_firmware_file_error(self, mock_stage, mock_verify_checksum,
|
|
mock_download_to_temp, mock_cleanup):
|
|
node = mock.Mock()
|
|
firmware_update = {'url': 'https://test1'}
|
|
CONF.set_override('firmware_source', 'local', 'redfish')
|
|
firmware_update = {'url': 'https://test1'}
|
|
node = mock.Mock()
|
|
mock_download_to_temp.return_value = '/tmp/test1'
|
|
mock_stage.side_effect = exception.IronicException
|
|
|
|
management = redfish_mgmt.RedfishManagement()
|
|
self.assertRaises(exception.IronicException,
|
|
management._stage_firmware_file, node,
|
|
firmware_update)
|
|
mock_download_to_temp.assert_called_with(node, 'https://test1')
|
|
mock_verify_checksum.assert_called_with(node, None, '/tmp/test1')
|
|
mock_stage.assert_called_with(node, 'local', '/tmp/test1')
|
|
mock_cleanup.assert_called_with(node)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_secure_boot_state(self, mock_get_system):
|
|
fake_system = mock_get_system.return_value
|
|
fake_system.secure_boot.enabled = False
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
response = task.driver.management.get_secure_boot_state(task)
|
|
self.assertIs(False, response)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_secure_boot_state_not_implemented(self, mock_get_system):
|
|
# Yes, seriously, that's the only way to do it.
|
|
class NoSecureBoot(mock.Mock):
|
|
@property
|
|
def secure_boot(self):
|
|
raise sushy.exceptions.MissingAttributeError("boom")
|
|
|
|
mock_get_system.return_value = NoSecureBoot()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertRaises(exception.UnsupportedDriverExtension,
|
|
task.driver.management.get_secure_boot_state,
|
|
task)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_secure_boot_state(self, mock_get_system):
|
|
fake_system = mock_get_system.return_value
|
|
fake_system.secure_boot.enabled = False
|
|
fake_system.boot = {'mode': sushy.BOOT_SOURCE_MODE_UEFI}
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
task.driver.management.set_secure_boot_state(task, True)
|
|
fake_system.secure_boot.set_enabled.assert_called_once_with(True)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_secure_boot_state_boot_mode_unknown(self, mock_get_system):
|
|
fake_system = mock_get_system.return_value
|
|
fake_system.secure_boot.enabled = False
|
|
fake_system.boot = {}
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
task.driver.management.set_secure_boot_state(task, True)
|
|
fake_system.secure_boot.set_enabled.assert_called_once_with(True)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_secure_boot_state_boot_mode_no_change(self, mock_get_system):
|
|
fake_system = mock_get_system.return_value
|
|
fake_system.secure_boot.enabled = False
|
|
fake_system.boot = {'mode': sushy.BOOT_SOURCE_MODE_BIOS}
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
task.driver.management.set_secure_boot_state(task, False)
|
|
self.assertFalse(fake_system.secure_boot.set_enabled.called)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_secure_boot_state_boot_mode_incorrect(self, mock_get_system):
|
|
fake_system = mock_get_system.return_value
|
|
fake_system.secure_boot.enabled = False
|
|
fake_system.boot = {'mode': sushy.BOOT_SOURCE_MODE_BIOS}
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertRaisesRegex(
|
|
exception.RedfishError, 'requires UEFI',
|
|
task.driver.management.set_secure_boot_state, task, True)
|
|
self.assertFalse(fake_system.secure_boot.set_enabled.called)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_secure_boot_state_boot_mode_fails(self, mock_get_system):
|
|
fake_system = mock_get_system.return_value
|
|
fake_system.secure_boot.enabled = False
|
|
fake_system.secure_boot.set_enabled.side_effect = \
|
|
sushy.exceptions.SushyError
|
|
fake_system.boot = {'mode': sushy.BOOT_SOURCE_MODE_UEFI}
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertRaisesRegex(
|
|
exception.RedfishError, 'Failed to set secure boot',
|
|
task.driver.management.set_secure_boot_state, task, True)
|
|
fake_system.secure_boot.set_enabled.assert_called_once_with(True)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_set_secure_boot_state_not_implemented(self, mock_get_system):
|
|
# Yes, seriously, that's the only way to do it.
|
|
class NoSecureBoot(mock.Mock):
|
|
@property
|
|
def secure_boot(self):
|
|
raise sushy.exceptions.MissingAttributeError("boom")
|
|
|
|
mock_get_system.return_value = NoSecureBoot()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertRaises(exception.UnsupportedDriverExtension,
|
|
task.driver.management.set_secure_boot_state,
|
|
task, True)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_reset_secure_boot_to_default(self, mock_get_system):
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
task.driver.management.reset_secure_boot_keys_to_default(task)
|
|
sb = mock_get_system.return_value.secure_boot
|
|
sb.reset_keys.assert_called_once_with(
|
|
sushy.SECURE_BOOT_RESET_KEYS_TO_DEFAULT)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_reset_secure_boot_to_default_not_implemented(self,
|
|
mock_get_system):
|
|
class NoSecureBoot(mock.Mock):
|
|
@property
|
|
def secure_boot(self):
|
|
raise sushy.exceptions.MissingAttributeError("boom")
|
|
|
|
mock_get_system.return_value = NoSecureBoot()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertRaises(
|
|
exception.UnsupportedDriverExtension,
|
|
task.driver.management.reset_secure_boot_keys_to_default, task)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_clear_secure_boot(self, mock_get_system):
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
task.driver.management.clear_secure_boot_keys(task)
|
|
sb = mock_get_system.return_value.secure_boot
|
|
sb.reset_keys.assert_called_once_with(
|
|
sushy.SECURE_BOOT_RESET_KEYS_DELETE_ALL)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_clear_secure_boot_not_implemented(self, mock_get_system):
|
|
class NoSecureBoot(mock.Mock):
|
|
@property
|
|
def secure_boot(self):
|
|
raise sushy.exceptions.MissingAttributeError("boom")
|
|
|
|
mock_get_system.return_value = NoSecureBoot()
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertRaises(
|
|
exception.UnsupportedDriverExtension,
|
|
task.driver.management.clear_secure_boot_keys, task)
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_mac_addresses_success(self, mock_get_system):
|
|
self.init_system_mock(mock_get_system.return_value)
|
|
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertEqual(['00:11:22:33:44:55'],
|
|
task.driver.management.get_mac_addresses(task))
|
|
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_mac_addresses_no_ports_found(self, mock_get_system):
|
|
|
|
system_mock = self.init_system_mock(mock_get_system.return_value)
|
|
system_mock.ethernet_interfaces.summary = None
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertEqual([],
|
|
task.driver.management.get_mac_addresses(task))
|
|
|
|
@mock.patch.object(redfish_utils, 'get_enabled_macs', autospec=True)
|
|
@mock.patch.object(redfish_utils, 'get_system', autospec=True)
|
|
def test_get_mac_addresses_missing_attr(self, mock_get_system,
|
|
mock_get_enabled_macs):
|
|
redfish_utils.get_enabled_macs.side_effect = (sushy.exceptions.
|
|
MissingAttributeError)
|
|
with task_manager.acquire(self.context, self.node.uuid,
|
|
shared=True) as task:
|
|
self.assertIsNone(task.driver.management.get_mac_addresses(task))
|