upstream/kubernetes
1
0
Fork 0
mirror of https://github.com/kubernetes/kubernetes.git synced 2026-02-03 20:40:26 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

e2e tests netpol: Set getProbeTimeoutSeconds to 3

Browse source 
Before tests sometimes failed randomly. For example:

> [It] [sig-network] Netpol NetworkPolicy between server and client should allow ingress access from updated namespace [Feature:NetworkPolicy]

Related: https://kubernetes.slack.com/archives/C09QYUH5W/p1770111316158039

Signed-off-by: Thomas Güttler <thomas.guettler@syself.com>

Update test/e2e/network/netpol/kubemanager.go

Co-authored-by: Antonio Ojea <antonio.ojea.garcia@gmail.com>
Signed-off-by: Thomas Güttler <thomas.guettler@syself.com>
This commit is contained in:
Thomas Güttler 2026-02-03 12:09:57 +01:00
parent bfafa32d90
commit 1c2e529295
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
test/e2e/network/netpol/kubemanager.go
View file

@ -321,13 +321,15 @@ func (k *kubeManager) getNamespace(ctx context.Context, ns string) (*v1.Namespac
return selectedNameSpace, nil
}
// getProbeTimeoutSeconds returns a timeout for how long the probe should work before failing a check, and takes windows heuristics into account, where requests can take longer sometimes.
// getProbeTimeoutSeconds returns a timeout for how long the probe should work before failing a
// check. The 3-second timeout not only ensures probe reliability across different platforms,
// but also serves as an assertion on the non-functional requirement that the dataplane must
// program network policies within an acceptable latency. This is especially critical for
// network policy features where slow dataplane programming could impact connectivity and security.
// The 3-second threshold was determined through empirical evidence as the maximum acceptable
// latency for network policy enforcement.
func getProbeTimeoutSeconds() int {
timeoutSeconds := 1
if framework.NodeOSDistroIs("windows") {
timeoutSeconds = 3
}
return timeoutSeconds
return 3
}
// getWorkers returns the number of workers suggested to run when testing.