openstack/ironic
Code Issues Proposed changes

Mock random generator for BackoffLoopingCall in IPMI unittests

Browse Source 
Set power state methods now use BackoffLoopingCall to wait for the
desired power state. It uses random.SystemRandom gauss distribution
to determine the amount of time to wait for the next check. This
change adds a method to mock the result of the pseudo-random sleep
interval generation, so that the value to wait between power state
checks is 1.

Closes-Bug: 1702859
Change-Id: I9270a187fa0f413ba8a8a14f859b0fd65c439b6e
This commit is contained in:
Vladyslav Drok 2017-07-07 13:25:17 +03:00
parent b1b22e885b
commit 54d5335edd
1 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
ironic/tests/unit/drivers/modules/test_ipmitool.py
View File

@ -22,6 +22,7 @@
import contextlib import contextlib
import os import os
import random
import stat import stat
import subprocess import subprocess
import tempfile import tempfile
@ -350,6 +351,19 @@ class IPMIToolPrivateMethodTestCase(db_base.DbTestCase):
driver_info=INFO_DICT) driver_info=INFO_DICT)
self.info = ipmi._parse_driver_info(self.node) self.info = ipmi._parse_driver_info(self.node)
# power actions use oslo_service.BackoffLoopingCall,
# mock random.SystemRandom gauss distribution
self._mock_system_random_distribution()
def _mock_system_random_distribution(self):
# random.SystemRandom with gauss distribution is used by oslo_service's
# BackoffLoopingCall, it multiplies default interval (equals to 1) by
# 2 * return_value, so if you want BackoffLoopingCall to "sleep" for
# 1 second, return_value should be 0.5.
m = mock.patch.object(random.SystemRandom, 'gauss', return_value=0.5)
m.start()
self.addCleanup(m.stop)
def _test__make_password_file(self, mock_sleep, input_password, def _test__make_password_file(self, mock_sleep, input_password,
exception_to_raise=None): exception_to_raise=None):
pw_file = None pw_file = None
@ -1259,7 +1273,7 @@ class IPMIToolPrivateMethodTestCase(db_base.DbTestCase):
self.assertRaises(exception.PowerStateFailure, self.assertRaises(exception.PowerStateFailure,
ipmi._power_on, task, self.info, timeout=2) ipmi._power_on, task, self.info, timeout=2)
self.assertEqual(mock_exec.call_args_list, expected) self.assertEqual(expected, mock_exec.call_args_list)
@mock.patch.object(ipmi, '_exec_ipmitool', autospec=True) @mock.patch.object(ipmi, '_exec_ipmitool', autospec=True)
@mock.patch('eventlet.greenthread.sleep', autospec=True) @mock.patch('eventlet.greenthread.sleep', autospec=True)
@ -1279,8 +1293,8 @@ class IPMIToolPrivateMethodTestCase(db_base.DbTestCase):
with task_manager.acquire(self.context, self.node.uuid) as task: with task_manager.acquire(self.context, self.node.uuid) as task:
state = ipmi._soft_power_off(task, self.info) state = ipmi._soft_power_off(task, self.info)
self.assertEqual(mock_exec.call_args_list, expected) self.assertEqual(expected, mock_exec.call_args_list)
self.assertEqual(state, states.POWER_OFF) self.assertEqual(states.POWER_OFF, state)
@mock.patch.object(ipmi, '_exec_ipmitool', autospec=True) @mock.patch.object(ipmi, '_exec_ipmitool', autospec=True)
@mock.patch('eventlet.greenthread.sleep', autospec=True) @mock.patch('eventlet.greenthread.sleep', autospec=True)
@ -1302,7 +1316,7 @@ class IPMIToolPrivateMethodTestCase(db_base.DbTestCase):
self.assertRaises(exception.PowerStateFailure, self.assertRaises(exception.PowerStateFailure,
ipmi._soft_power_off, task, self.info, timeout=2) ipmi._soft_power_off, task, self.info, timeout=2)
self.assertEqual(mock_exec.call_args_list, expected) self.assertEqual(expected, mock_exec.call_args_list)
@mock.patch.object(ipmi, '_power_status', autospec=True) @mock.patch.object(ipmi, '_power_status', autospec=True)
@mock.patch.object(ipmi, '_exec_ipmitool', autospec=True) @mock.patch.object(ipmi, '_exec_ipmitool', autospec=True)