Skip to content

Commit cd4a559

Browse files
committed
add timeout for updating gateway listeners
1 parent ab84415 commit cd4a559

File tree

2 files changed

+17
-13
lines changed

2 files changed

+17
-13
lines changed

tests/framework/timeout.go

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -38,22 +38,26 @@ type TimeoutConfig struct {
3838

3939
// KubernetesClientTimeout represents the maximum time for Kubernetes client operations.
4040
KubernetesClientTimeout time.Duration
41+
42+
// GatewayListenerUpdateTimeout represents the maximum time for Gateway Listener count to be updated.
43+
GatewayListenerUpdateTimeout time.Duration
4144
}
4245

4346
// DefaultTimeoutConfig populates a TimeoutConfig with the default values.
4447
func DefaultTimeoutConfig() TimeoutConfig {
4548
return TimeoutConfig{
46-
CreateTimeout: 60 * time.Second,
47-
UpdateTimeout: 60 * time.Second,
48-
DeleteTimeout: 10 * time.Second,
49-
DeleteNamespaceTimeout: 150 * time.Second,
50-
GetTimeout: 10 * time.Second,
51-
ManifestFetchTimeout: 10 * time.Second,
52-
RequestTimeout: 30 * time.Second,
53-
ContainerRestartTimeout: 10 * time.Second,
54-
GetLeaderLeaseTimeout: 60 * time.Second,
55-
GetStatusTimeout: 60 * time.Second,
56-
TestForTrafficTimeout: 60 * time.Second,
57-
KubernetesClientTimeout: 10 * time.Second,
49+
CreateTimeout: 60 * time.Second,
50+
UpdateTimeout: 60 * time.Second,
51+
DeleteTimeout: 10 * time.Second,
52+
DeleteNamespaceTimeout: 150 * time.Second,
53+
GetTimeout: 10 * time.Second,
54+
ManifestFetchTimeout: 10 * time.Second,
55+
RequestTimeout: 30 * time.Second,
56+
ContainerRestartTimeout: 10 * time.Second,
57+
GetLeaderLeaseTimeout: 60 * time.Second,
58+
GetStatusTimeout: 60 * time.Second,
59+
TestForTrafficTimeout: 60 * time.Second,
60+
KubernetesClientTimeout: 10 * time.Second,
61+
GatewayListenerUpdateTimeout: 60 * time.Second,
5862
}
5963
}

tests/suite/scale_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1046,7 +1046,7 @@ var _ = Describe("Zero downtime scale test", Ordered, Label("nfr", "zero-downtim
10461046
return nil
10471047
},
10481048
).
1049-
WithTimeout(timeoutConfig.GetStatusTimeout).
1049+
WithTimeout(timeoutConfig.GatewayListenerUpdateTimeout).
10501050
WithPolling(1 * time.Second).
10511051
Should(Succeed())
10521052
}

0 commit comments

Comments
 (0)