Skip to content

Commit 065949b

Browse files
dgrisonnetdinhxuanvu
authored andcommitted
upgrade.go: wait some time after node upgrade
For the "Cluster should remain functional during upgrade" test, TRT noticed flakes from the step that verifies that deamonsets are running on all expected nodes afer an upgrade. This flake was caused by the verification of the deamonset happening too quickly after the upgrade. As soon as the last upgraded node becomes ready the check happens, but it doesn't always leave enough time for the deamonset to restart, thus causing the test to fail. Signed-off-by: Damien Grisonnet <[email protected]>
1 parent 1718cf2 commit 065949b

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

test/e2e/upgrade/upgrade.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,9 @@ var _ = g.Describe("[sig-arch][Feature:ClusterUpgrade]", func() {
188188
clusterUpgrade(f, client, dynamicClient, config, upgCtx.Versions[i]),
189189
fmt.Sprintf("during upgrade to %s", upgCtx.Versions[i].NodeImage))
190190
}
191+
// Sleep to give some time to the workloads on the last upgraded
192+
// node to restart.
193+
time.Sleep(30 * time.Second)
191194
},
192195
)
193196
})

0 commit comments

Comments
 (0)