diff --git a/ironic_python_agent/raid_utils.py b/ironic_python_agent/raid_utils.py index d4000eb1b..5db963e9d 100644 --- a/ironic_python_agent/raid_utils.py +++ b/ironic_python_agent/raid_utils.py @@ -227,7 +227,7 @@ def create_raid_device(index, logical_disk): volume_name = logical_disk.get('volume_name') try: if volume_name is None: - volume_name = md_device + volume_name = 'md%d' % index LOG.debug("Creating md device %(dev)s with name %(name)s" "on %(comp)s", {'dev': md_device, 'name': volume_name, diff --git a/ironic_python_agent/tests/unit/test_hardware.py b/ironic_python_agent/tests/unit/test_hardware.py index 64407e517..fd80b9a6c 100644 --- a/ironic_python_agent/tests/unit/test_hardware.py +++ b/ironic_python_agent/tests/unit/test_hardware.py @@ -3637,10 +3637,10 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 2, '/dev/sda1', '/dev/sdb1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '0', '--name', '/dev/md1', + '--metadata=1', '--level', '0', '--name', 'md1', '--raid-devices', 2, '/dev/sda2', '/dev/sdb2')]) self.assertEqual(raid_config, result) @@ -3742,11 +3742,11 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 3, '/dev/sda1', '/dev/sdb1', '/dev/sdc1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '5', '--name', '/dev/md1', + '--metadata=1', '--level', '5', '--name', 'md1', '--raid-devices', 3, '/dev/sda2', '/dev/sdb2', '/dev/sdc2')]) self.assertEqual(raid_config, result) @@ -3860,11 +3860,11 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 4, '/dev/sda1', '/dev/sdb1', '/dev/sdc1', '/dev/sdd1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '6', '--name', '/dev/md1', + '--metadata=1', '--level', '6', '--name', 'md1', '--raid-devices', 4, '/dev/sda2', '/dev/sdb2', '/dev/sdc2', '/dev/sdd2')]) self.assertEqual(raid_config, result) @@ -3943,10 +3943,10 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 2, '/dev/sda1', '/dev/sdb1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '0', '--name', '/dev/md1', + '--metadata=1', '--level', '0', '--name', 'md1', '--raid-devices', 2, '/dev/sda2', '/dev/sdb2')]) self.assertEqual(raid_config, result) @@ -4030,10 +4030,10 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 2, '/dev/sda1', '/dev/sdb1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '0', '--name', '/dev/md1', + '--metadata=1', '--level', '0', '--name', 'md1', '--raid-devices', 2, '/dev/sda2', '/dev/sdb2')]) self.assertEqual(raid_config, result) @@ -4112,10 +4112,10 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 2, '/dev/sda1', '/dev/sdb1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '0', '--name', '/dev/md1', + '--metadata=1', '--level', '0', '--name', 'md1', '--raid-devices', 2, '/dev/sda2', '/dev/sdb2')]) self.assertEqual(raid_config, result) @@ -4196,10 +4196,10 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '0', '--name', '/dev/md0', + '--metadata=1', '--level', '0', '--name', 'md0', '--raid-devices', 2, '/dev/sda1', '/dev/sdb1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md1', + '--metadata=1', '--level', '1', '--name', 'md1', '--raid-devices', 2, '/dev/sda2', '/dev/sdb2')]) self.assertEqual(raid_config, result) @@ -4289,10 +4289,10 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 2, '/dev/sda1', '/dev/sdb1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '0', '--name', '/dev/md1', + '--metadata=1', '--level', '0', '--name', 'md1', '--raid-devices', 2, '/dev/sda2', '/dev/sdb2')]) self.assertEqual(raid_config, result) @@ -4637,11 +4637,11 @@ class TestGenericHardwareManager(base.IronicAgentTest): delay_on_retry=True), mock.call('udevadm', 'settle'), mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 2, '/dev/nvme0n1p1', '/dev/nvme1n1p1'), mock.call('mdadm', '--create', '/dev/md1', '--force', '--run', - '--metadata=1', '--level', '0', '--name', '/dev/md1', + '--metadata=1', '--level', '0', '--name', 'md1', '--raid-devices', 2, '/dev/nvme0n1p2', '/dev/nvme1n1p2') ]) self.assertEqual(raid_config, result) diff --git a/ironic_python_agent/tests/unit/test_raid_utils.py b/ironic_python_agent/tests/unit/test_raid_utils.py index fda7795ca..835021a74 100644 --- a/ironic_python_agent/tests/unit/test_raid_utils.py +++ b/ironic_python_agent/tests/unit/test_raid_utils.py @@ -57,7 +57,7 @@ class TestRaidUtils(base.IronicAgentTest): mock_execute.assert_called_once_with( 'mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 3, '/dev/sda1', '/dev/sdb1', '/dev/sdc1') @mock.patch.object(raid_utils, '_get_actual_component_devices', @@ -97,7 +97,7 @@ class TestRaidUtils(base.IronicAgentTest): expected_calls = [ mock.call('mdadm', '--create', '/dev/md0', '--force', '--run', - '--metadata=1', '--level', '1', '--name', '/dev/md0', + '--metadata=1', '--level', '1', '--name', 'md0', '--raid-devices', 3, '/dev/sda1', '/dev/sdb1', '/dev/sdc1'), mock.call('mdadm', '--add', '/dev/md0', '/dev/sdb1',