diff options
author | Julia Kreger <juliaashleykreger@gmail.com> | 2023-02-28 13:28:49 -0800 |
---|---|---|
committer | Julia Kreger <juliaashleykreger@gmail.com> | 2023-03-14 19:30:38 +0000 |
commit | 4481031d71339336bc1d3832e818018e4403e292 (patch) | |
tree | 45847dd37d519dde5a9520dd3f2733b3d0af0702 /ironic/tests/unit/conductor/test_utils.py | |
parent | 3e3d4e1cbc7b886e8bbd058d89425b3f3703f006 (diff) | |
download | ironic-4481031d71339336bc1d3832e818018e4403e292.tar.gz |
Wipe Agent Token when cleaning timeout occcurs
In a relatively odd turn of events, should cleaning
have started, but then timed out due to lost communications
or a hard failure of the machine, an agent token could
previously be orphaned preventing re-cleaning.
We now explicitly remove the token in this case.
Change-Id: I236cdf6ddb040284e9fd1fa10136ad17ef665638
(cherry picked from commit 47b5909486c336352c536eb2cadd121afea8cf12)
Diffstat (limited to 'ironic/tests/unit/conductor/test_utils.py')
-rw-r--r-- | ironic/tests/unit/conductor/test_utils.py | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/ironic/tests/unit/conductor/test_utils.py b/ironic/tests/unit/conductor/test_utils.py index a29da21a7..e190358d2 100644 --- a/ironic/tests/unit/conductor/test_utils.py +++ b/ironic/tests/unit/conductor/test_utils.py @@ -1172,6 +1172,9 @@ class ErrorHandlersTestCase(db_base.DbTestCase): self.node.set_driver_internal_info('skip_current_clean_step', True) self.node.set_driver_internal_info('clean_step_index', 0) self.node.set_driver_internal_info('agent_url', 'url') + self.node.set_driver_internal_info('agent_secret_token', 'foo') + self.node.set_driver_internal_info('agent_secret_token_pregenerated', + False) msg = 'error bar' last_error = "last error" @@ -1184,6 +1187,9 @@ class ErrorHandlersTestCase(db_base.DbTestCase): self.assertNotIn('cleaning_polling', self.node.driver_internal_info) self.assertNotIn('skip_current_clean_step', self.node.driver_internal_info) + self.assertNotIn('agent_secret_token', self.node.driver_internal_info) + self.assertNotIn('agent_secret_token_pregenerated', + self.node.driver_internal_info) self.assertEqual(last_error, self.node.last_error) self.assertTrue(self.node.maintenance) self.assertEqual(last_error, self.node.maintenance_reason) |