Skip to content

Commit

Permalink
extraConfigBfb: fix race condition before coldboot
Browse files Browse the repository at this point in the history
When cold booting BF-2 host to apply nic mode config, we need to make
sure the systemd file to switch is first in place.

Signed-off-by: Salvatore Daniele <[email protected]>
  • Loading branch information
SalDaniele committed Oct 9, 2024
1 parent 081734a commit f32620c
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion extraConfigBFB.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,9 @@ def helper(h: host.HostWithBF2) -> Optional[host.Result]:

client.oc("delete -f manifests/nicmode/switch.yaml")
client.oc("create -f manifests/nicmode/switch.yaml")

logger.info("Waiting for mcp to update")
client.wait_for_mcp("sriov", "switch.yaml")
# Workaround for https://issues.redhat.com/browse/OCPBUGS-29882 caused by the BF-2 firmware failing to update without cold boot

logger.info("Cold booting.....")
Expand All @@ -89,5 +92,5 @@ def helper(h: host.HostWithBF2) -> Optional[host.Result]:
f = executor.submit(helper, h)
futures[e.name] = f

logger.info("Waiting for mcp to update")
logger.info("Waiting for nodes to recover from cold boot")
client.wait_for_mcp("sriov", "switch.yaml")

0 comments on commit f32620c

Please sign in to comment.