Skip to content

Commit

Permalink
internal/gomote: extend gomote max timeout
Browse files Browse the repository at this point in the history
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 <dmitshur@golang.org>
Reviewed-by: David Chase <drchase@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
  • Loading branch information
cagedmantis committed Sep 27, 2024
1 parent a411a18 commit b1faa3d
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions internal/gomote/swarming.go
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down Expand Up @@ -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,
Expand Down

0 comments on commit b1faa3d

Please sign in to comment.