Skip to content

Commit 6536010

Browse files
authored
test: fix TestStressMemoryUsageHelm timeout (#1867) (#1872)
This updates the WatchForAllSyncs calls in the helm stress test to use the default timeout instead of a hard coded timeout. The default timeout is set programmatically, and is longer for autopilot (10 minutes instead of the hard coded 5 minutes). This is intended to fix timeout errors with this test on autopilot.
1 parent 3db2f77 commit 6536010

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

e2e/testcases/stress_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -566,7 +566,7 @@ func TestStressMemoryUsageHelm(t *testing.T) {
566566

567567
// Validate that the resources sync without the reconciler running out of
568568
// memory, getting OOMKilled, and crash looping.
569-
nt.Must(nt.WatchForAllSyncs(nomostest.WithTimeout(5 * time.Minute)))
569+
nt.Must(nt.WatchForAllSyncs())
570570

571571
nt.T.Logf("Verify the number of Anvil objects")
572572
for i := 1; i <= crdCount; i++ {
@@ -598,7 +598,7 @@ func TestStressMemoryUsageHelm(t *testing.T) {
598598
})
599599
// Validate that the resources sync without the reconciler running out of
600600
// memory, getting OOMKilled, and crash looping.
601-
nt.Must(nt.WatchForAllSyncs(nomostest.WithTimeout(5 * time.Minute)))
601+
nt.Must(nt.WatchForAllSyncs())
602602
}
603603

604604
func TestStressResourceGroup(t *testing.T) {

0 commit comments

Comments
 (0)