Skip to content

Commit 2e6a91e

Browse files
update sno
1 parent 8220df8 commit 2e6a91e

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

test/extended/machine_config/machine_config_node.go

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -277,15 +277,15 @@ func ValidateMCNConditionTransitionsOnRebootlessUpdateSNO(oc *exutil.CLI, nodeDi
277277
clientSet, clientErr := machineconfigclient.NewForConfig(oc.KubeFramework().ClientConfig())
278278
o.Expect(clientErr).NotTo(o.HaveOccurred(), "Error creating client set for test.")
279279

280-
// // Remove node disruption policy on test completion or failure
281-
// defer func() {
282-
// // Apply empty MachineConfiguration fixture to remove previously set NodeDisruptionPolicy
283-
// framework.Logf("Removing node disruption policy.")
284-
// ApplyBootImageFixture(oc, nodeDisruptionEmptyFixture)
285-
// }()
280+
// Remove node disruption policy on test completion or failure
281+
defer func() {
282+
// Apply empty MachineConfiguration fixture to remove previously set NodeDisruptionPolicy
283+
framework.Logf("Removing node disruption policy.")
284+
ApplyBootImageFixture(oc, nodeDisruptionEmptyFixture)
285+
}()
286286

287-
// // Apply a node disruption policy to allow for rebootless update
288-
// ApplyBootImageFixture(oc, nodeDisruptionFixture)
287+
// Apply a node disruption policy to allow for rebootless update
288+
ApplyBootImageFixture(oc, nodeDisruptionFixture)
289289

290290
// Delete applied MC on test completion or failure
291291
defer func() {
@@ -295,9 +295,9 @@ func ValidateMCNConditionTransitionsOnRebootlessUpdateSNO(oc *exutil.CLI, nodeDi
295295
o.Expect(deleteMCErr).NotTo(o.HaveOccurred(), fmt.Sprintf("Could not delete MachineConfig '%v'.", mcName))
296296

297297
// Wait for master MCP to be ready
298-
// TODO: see if there needs to be a wait before checking this update status
299-
framework.Logf("Waiting for %v MCP to be updated with %v ready machines.", master, 1)
300-
WaitForMCPToBeReady(oc, clientSet, worker, 1)
298+
time.Sleep(15 * time.Second) //wait to not catch the updated state before the deleted mc triggers an update
299+
framework.Logf("Waiting for %v MCP to be updated with %v ready machines.", poolName, 1)
300+
WaitForMCPToBeReady(oc, clientSet, poolName, 1)
301301
}()
302302

303303
// Apply MC targeting worker node

0 commit comments

Comments
 (0)