From 1fc4268aea7f2df7527b14f2521ff511cacbe935 Mon Sep 17 00:00:00 2001 From: Jordan Whited Date: Wed, 4 Sep 2024 09:54:32 -0700 Subject: [PATCH] cmd/stunstamp: increase probe jitter (#13362) We've added more probe targets recently which has resulted in more timeouts behind restrictive NATs in localized testing that don't like how many flows we are creating at once. Not so much an issue for datacenter or cloud-hosted deployments. Updates tailscale/corp#22114 Signed-off-by: Jordan Whited --- cmd/stunstamp/stunstamp.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/stunstamp/stunstamp.go b/cmd/stunstamp/stunstamp.go index f66fb7c05..1aaa7ced1 100644 --- a/cmd/stunstamp/stunstamp.go +++ b/cmd/stunstamp/stunstamp.go @@ -380,7 +380,7 @@ func probe(meta nodeMeta, cf *connAndMeasureFn, dstPort int) (*time.Duration, er Port: dstPort, } - time.Sleep(rand.N(200 * time.Millisecond)) // jitter across tx + time.Sleep(rand.N(400 * time.Millisecond)) // jitter across tx rtt, err := cf.fn(cf.conn, meta.hostname, netip.AddrPortFrom(meta.addr, uint16(dstPort))) if err != nil { if isTemporaryOrTimeoutErr(err) {