x/tobiko
Code Issues Proposed changes

Increase timeout for ironic power-on and power-off operations

Browse Source 
Some test failed recently due to timeout powering-off a controller node
This patch should make that test stable and prevent timeouts in any
other tests using ironic to power-on/off overcloud nodes

Change-Id: If6262e5930dc41edd50215da3f71844651057eaf
This commit is contained in:
Eduardo Olivares 2021-09-16 17:42:48 +02:00
parent 0845507019
commit 60a8ad834f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tobiko/tripleo/_overcloud.py
View File

@ -76,7 +76,7 @@ def power_on_overcloud_node(server: typing.Union[nova.ServerType]):
if node_id is not None: if node_id is not None:
client = ironic.get_ironic_client(session=session) client = ironic.get_ironic_client(session=session)
try: try:
ironic.power_on_node(client=client, node=node_id) ironic.power_on_node(client=client, node=node_id, timeout=60)
return return
except ironic.WaitForNodePowerStateError: except ironic.WaitForNodePowerStateError:
LOG.exception(f"Failed powering on Ironic node: '{node_id}'") LOG.exception(f"Failed powering on Ironic node: '{node_id}'")
@ -93,7 +93,7 @@ def power_off_overcloud_node(server: typing.Union[nova.ServerType]) \
if node_id is not None: if node_id is not None:
client = ironic.get_ironic_client(session=session) client = ironic.get_ironic_client(session=session)
try: try:
ironic.power_off_node(client=client, node=node_id) ironic.power_off_node(client=client, node=node_id, timeout=60)
return return
except ironic.WaitForNodePowerStateError: except ironic.WaitForNodePowerStateError:
LOG.exception(f"Failed powering off Ironic node: '{node_id}'") LOG.exception(f"Failed powering off Ironic node: '{node_id}'")