diff options
author | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-12-13 10:23:54 +0100 |
---|---|---|
committer | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-12-13 10:25:27 +0100 |
commit | d342b07dd6f8c5478f37a31b14e77bba6bef55e2 (patch) | |
tree | aa89e55a35021455887e6c51dfbe7c1d7a11d86f /ironic/tests/unit/conductor/test_manager.py | |
parent | f6f6ce1a315b7ba11557d95036116741a4347ce2 (diff) | |
download | ironic-d342b07dd6f8c5478f37a31b14e77bba6bef55e2.tar.gz |
Adoption: do not validate boot interface when local booting
We validate the boot interface during adoption because of:
a) potential rebuilding,
b) non-local boot.
Rebuild proved a rarely used feature, and local boot is the default
nowadays, so it makes less sense to unconditionally validate the boot
interface during adoption. We will run the validation anyway the next
time we need to do something with booting.
Similarly, do not record is_whole_disk_image None if it cannot be
reliably determined.
Change-Id: I95252aea808c48ea2d94569449c871f0d483caaa
Diffstat (limited to 'ironic/tests/unit/conductor/test_manager.py')
-rw-r--r-- | ironic/tests/unit/conductor/test_manager.py | 53 |
1 files changed, 47 insertions, 6 deletions
diff --git a/ironic/tests/unit/conductor/test_manager.py b/ironic/tests/unit/conductor/test_manager.py index da1d7574d..81984e88b 100644 --- a/ironic/tests/unit/conductor/test_manager.py +++ b/ironic/tests/unit/conductor/test_manager.py @@ -7286,8 +7286,47 @@ class DoNodeAdoptionTestCase(mgr_utils.ServiceSetUpMixin, db_base.DbTestCase): mock_prepare.assert_called_once_with(task.driver.deploy, task) mock_take_over.assert_called_once_with(task.driver.deploy, task) self.assertFalse(mock_start_console.called) - self.assertTrue(mock_boot_validate.called) - self.assertIn('is_whole_disk_image', task.node.driver_internal_info) + mock_boot_validate.assert_not_called() + self.assertNotIn('is_whole_disk_image', task.node.driver_internal_info) + + @mock.patch('ironic.drivers.modules.fake.FakePower.validate', + autospec=True) + @mock.patch('ironic.drivers.modules.fake.FakeBoot.validate', autospec=True) + @mock.patch('ironic.drivers.modules.fake.FakeConsole.start_console', + autospec=True) + @mock.patch('ironic.drivers.modules.fake.FakeDeploy.take_over', + autospec=True) + @mock.patch('ironic.drivers.modules.fake.FakeDeploy.prepare', + autospec=True) + def test__do_adoption_with_netboot(self, + mock_prepare, + mock_take_over, + mock_start_console, + mock_boot_validate, + mock_power_validate): + """Test a successful node adoption""" + self._start_service() + node = obj_utils.create_test_node( + self.context, driver='fake-hardware', + provision_state=states.ADOPTING, + instance_info={ + 'capabilities': {'boot_option': 'netboot'}, + 'image_source': 'image', + }) + task = task_manager.TaskManager(self.context, node.uuid) + + self.service._do_adoption(task) + node.refresh() + + self.assertEqual(states.ACTIVE, node.provision_state) + self.assertIsNone(node.last_error) + self.assertFalse(node.console_enabled) + mock_prepare.assert_called_once_with(task.driver.deploy, task) + mock_take_over.assert_called_once_with(task.driver.deploy, task) + self.assertFalse(mock_start_console.called) + mock_boot_validate.assert_called_once_with(task.driver.boot, task) + self.assertTrue(task.node.driver_internal_info.get( + 'is_whole_disk_image')) @mock.patch('ironic.drivers.modules.fake.FakeBoot.validate', autospec=True) @mock.patch('ironic.drivers.modules.fake.FakeConsole.start_console', @@ -7328,8 +7367,7 @@ class DoNodeAdoptionTestCase(mgr_utils.ServiceSetUpMixin, db_base.DbTestCase): mock_prepare.assert_called_once_with(task.driver.deploy, task) mock_take_over.assert_called_once_with(task.driver.deploy, task) self.assertFalse(mock_start_console.called) - self.assertTrue(mock_boot_validate.called) - self.assertIn('is_whole_disk_image', task.node.driver_internal_info) + mock_boot_validate.assert_not_called() self.assertEqual(states.NOSTATE, node.power_state) @mock.patch('ironic.drivers.modules.fake.FakeBoot.validate', autospec=True) @@ -7353,7 +7391,10 @@ class DoNodeAdoptionTestCase(mgr_utils.ServiceSetUpMixin, db_base.DbTestCase): self._start_service() node = obj_utils.create_test_node( self.context, driver='fake-hardware', - provision_state=states.ADOPTING) + provision_state=states.ADOPTING, + instance_info={ + 'capabilities': {'boot_option': 'netboot'}, + }) task = task_manager.TaskManager(self.context, node.uuid) self.service._do_adoption(task) @@ -7365,7 +7406,7 @@ class DoNodeAdoptionTestCase(mgr_utils.ServiceSetUpMixin, db_base.DbTestCase): self.assertFalse(mock_prepare.called) self.assertFalse(mock_take_over.called) self.assertFalse(mock_start_console.called) - self.assertTrue(mock_boot_validate.called) + mock_boot_validate.assert_called_once_with(task.driver.boot, task) @mock.patch('ironic.conductor.manager.ConductorManager._spawn_worker', autospec=True) |