From b1faa3d2fbbc465e55d8bc9b562d24c37feae8d6 Mon Sep 17 00:00:00 2001 From: Carlos Amedee Date: Fri, 27 Sep 2024 11:56:50 -0400 Subject: [PATCH] internal/gomote: extend gomote max timeout This extends the maximum amount of time a gomote instance can run. It was previously 24 hours. It has now been set to approximately a week. Fixes golang/go#69658 Change-Id: I01d7f10a5ba96b3213dbb5fdfeff66cbc7aa688b Reviewed-on: https://go-review.googlesource.com/c/build/+/616062 Reviewed-by: Dmitri Shuralyov Reviewed-by: David Chase LUCI-TryBot-Result: Go LUCI Reviewed-by: Dmitri Shuralyov Reviewed-by: Michael Knyszek --- internal/gomote/swarming.go | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/internal/gomote/swarming.go b/internal/gomote/swarming.go index 33c3e0cbde..f8796a1f21 100644 --- a/internal/gomote/swarming.go +++ b/internal/gomote/swarming.go @@ -987,7 +987,9 @@ func (ss *SwarmingServer) newSwarmingTask(ctx context.Context, name string, dime Value: name, }, }, - ExecutionTimeoutSecs: 86400, + // The swarming limits state it must be between 30s and 601140s. This information is returned + // as part of an error message when you attempt a request with a value outside of these boundaries. + ExecutionTimeoutSecs: 601140, }, ExpirationSecs: 86400, WaitForCapacity: false, @@ -1090,7 +1092,9 @@ func (ss *SwarmingServer) newSwarmingTaskWithGolangbuild(ctx context.Context, na Value: properties.BuilderId, }, }, - ExecutionTimeoutSecs: 86400, + // The swarming limits state it must be between 30s and 601140s. This information is returned + // as part of an error message when you attempt a request with a value outside of these boundaries. + ExecutionTimeoutSecs: 601140, }, ExpirationSecs: 86400, WaitForCapacity: false,