diff --git a/doc/source/usage.rst b/doc/source/usage.rst index 2ee84f5fc..01bbe4fba 100644 --- a/doc/source/usage.rst +++ b/doc/source/usage.rst @@ -204,7 +204,7 @@ Here are some plugins that can be additionally enabled: ``example`` example plugin logging it's input and output. -``raid_device`` (deprecated name ``root_device_hint``) +``raid_device`` gathers block devices from ramdisk and exposes root device in multiple runs. ``extra_hardware`` diff --git a/ironic_inspector/plugins/raid_device.py b/ironic_inspector/plugins/raid_device.py index d7cb6e487..027fadd9c 100644 --- a/ironic_inspector/plugins/raid_device.py +++ b/ironic_inspector/plugins/raid_device.py @@ -102,10 +102,3 @@ class RaidDeviceDetection(base.ProcessingHook): node_info.patch([{'op': 'add', 'path': '/extra/block_devices', 'value': {'serials': current_devices}}]) - - -class RootDeviceHintHook(RaidDeviceDetection): - def __init__(self): - LOG.warning(_LW('Using the root_device_hint alias for the ' - 'raid_device plugin is deprecated')) - super(RaidDeviceDetection, self).__init__() diff --git a/ironic_inspector/test/unit/test_plugins_raid_device.py b/ironic_inspector/test/unit/test_plugins_raid_device.py index a9a48fbdf..61785cc4e 100644 --- a/ironic_inspector/test/unit/test_plugins_raid_device.py +++ b/ironic_inspector/test/unit/test_plugins_raid_device.py @@ -23,12 +23,9 @@ class TestRaidDeviceDetection(test_base.NodeTest): hook = raid_device.RaidDeviceDetection() def test_loadable_by_name(self): - names = ('raid_device', 'root_device_hint') - base.CONF.set_override('processing_hooks', ','.join(names), - 'processing') - for name in names: - ext = base.processing_hooks_manager()[name] - self.assertIsInstance(ext.obj, raid_device.RaidDeviceDetection) + base.CONF.set_override('processing_hooks', 'raid_device', 'processing') + ext = base.processing_hooks_manager()['raid_device'] + self.assertIsInstance(ext.obj, raid_device.RaidDeviceDetection) def test_missing_local_gb(self): introspection_data = {} diff --git a/releasenotes/notes/no-root_device_hint-0e7676d481d503bb.yaml b/releasenotes/notes/no-root_device_hint-0e7676d481d503bb.yaml new file mode 100644 index 000000000..0f87039fa --- /dev/null +++ b/releasenotes/notes/no-root_device_hint-0e7676d481d503bb.yaml @@ -0,0 +1,3 @@ +--- +upgrade: + - Removed the deprecated "root_device_hint" alias for the "raid_device" hook. diff --git a/setup.cfg b/setup.cfg index b56a3e27b..2557d1efb 100644 --- a/setup.cfg +++ b/setup.cfg @@ -32,8 +32,6 @@ ironic_inspector.hooks.processing = extra_hardware = ironic_inspector.plugins.extra_hardware:ExtraHardwareHook raid_device = ironic_inspector.plugins.raid_device:RaidDeviceDetection capabilities = ironic_inspector.plugins.capabilities:CapabilitiesHook - # Deprecated name for raid_device, don't confuse with root_disk_selection - root_device_hint = ironic_inspector.plugins.raid_device:RootDeviceHintHook ironic_inspector.hooks.node_not_found = example = ironic_inspector.plugins.example:example_not_found_hook enroll = ironic_inspector.plugins.discovery:enroll_node_not_found_hook