From 20dc04e5e2b23f888ff79a3873d35fd4c170c8d8 Mon Sep 17 00:00:00 2001 From: Annie Lezil Date: Mon, 19 Dec 2016 11:09:52 -0800 Subject: Reboot and Poweroff fails with coreos IPA image The CoreOS IPA images do not support poweroff/reboot due to running in a chroot. For this case, we fall back to forcing poweroff or reboot via sysrq commands Change-Id: I75d68b6308beba299d043e43a5fa1671b6ef3ada Closes-Bug: #1628367 --- .../tests/unit/extensions/test_standby.py | 35 ++++++++++++++++++++-- 1 file changed, 32 insertions(+), 3 deletions(-) (limited to 'ironic_python_agent/tests') diff --git a/ironic_python_agent/tests/unit/extensions/test_standby.py b/ironic_python_agent/tests/unit/extensions/test_standby.py index b76dd596..6a98dd31 100644 --- a/ironic_python_agent/tests/unit/extensions/test_standby.py +++ b/ironic_python_agent/tests/unit/extensions/test_standby.py @@ -679,7 +679,34 @@ class TestStandbyExtension(test_base.BaseTestCase): self.agent_extension._run_shutdown_command('poweroff') calls = [mock.call('sync'), - mock.call('poweroff', check_exit_code=[0])] + mock.call('poweroff', use_standard_locale=True, + check_exit_code=[0])] + execute_mock.assert_has_calls(calls) + + @mock.patch('ironic_python_agent.utils.execute', autospec=True) + def test_run_shutdown_command_valid_poweroff_sysrq(self, execute_mock): + execute_mock.side_effect = [('', ''), ('', + 'Running in chroot, ignoring request.'), + ('', '')] + + self.agent_extension._run_shutdown_command('poweroff') + calls = [mock.call('sync'), + mock.call('poweroff', use_standard_locale=True, + check_exit_code=[0]), + mock.call("echo o > /proc/sysrq-trigger", shell=True)] + execute_mock.assert_has_calls(calls) + + @mock.patch('ironic_python_agent.utils.execute', autospec=True) + def test_run_shutdown_command_valid_reboot_sysrq(self, execute_mock): + execute_mock.side_effect = [('', ''), ('', + 'Running in chroot, ignoring request.'), + ('', '')] + + self.agent_extension._run_shutdown_command('reboot') + calls = [mock.call('sync'), + mock.call('reboot', use_standard_locale=True, + check_exit_code=[0]), + mock.call("echo b > /proc/sysrq-trigger", shell=True)] execute_mock.assert_has_calls(calls) @mock.patch('ironic_python_agent.utils.execute', autospec=True) @@ -689,7 +716,8 @@ class TestStandbyExtension(test_base.BaseTestCase): success_result = self.agent_extension.run_image() success_result.join() calls = [mock.call('sync'), - mock.call('reboot', check_exit_code=[0])] + mock.call('reboot', use_standard_locale=True, + check_exit_code=[0])] execute_mock.assert_has_calls(calls) self.assertEqual('SUCCEEDED', success_result.command_status) @@ -711,7 +739,8 @@ class TestStandbyExtension(test_base.BaseTestCase): success_result.join() calls = [mock.call('sync'), - mock.call('poweroff', check_exit_code=[0])] + mock.call('poweroff', use_standard_locale=True, + check_exit_code=[0])] execute_mock.assert_has_calls(calls) self.assertEqual('SUCCEEDED', success_result.command_status) -- cgit v1.2.1