Apply the gerrit REST timeout to only 'GetChanges' calls
https://crrev.com/c/4420526 put a 10s timeout on all calls made via
this script.
But it appears there are a variety of endpoints that take more than
10s, the latest being "SubmitChange", eg:
https://ci.chromium.org/ui/p/chromeos/builders/release/release-R113-15393.B-orchestrator/25/overview
So instead of applying the timeout to everything, and then opting
certain long-running calls out of it, this just opts-in the
"GetChanges" call to the timeout. This should cover the failure mode
that was originally solved for chrome's bots. And also bump the
timeout to 30s since we don't trust all get-changes queries to
reliably finish in under 10s.
This also bumps the step level timeout for the query in tryserver
recipe_mod to 8 min since there could be ~5min worth of sleeping +
150s worth of waiting+timing-out. So worst case the step will now
take 8min to fully exhaust all timeouts/sleeps.
Recipe-Nontrivial-Roll: build
Recipe-Nontrivial-Roll: build_limited
Recipe-Nontrivial-Roll: chromiumos
Recipe-Nontrivial-Roll: infra
Bug: b/278083716
Change-Id: Ib366e004e0bb07297ba732590d488cae779e38ba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4426524
Reviewed-by: Joanna Wang <jojwang@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
diff --git a/gerrit_util.py b/gerrit_util.py
index 3b2e0a0..a8b04ad 100644
--- a/gerrit_util.py
+++ b/gerrit_util.py
@@ -380,7 +380,7 @@
reqtype='GET',
headers=None,
body=None,
- timeout=10.0):
+ timeout=None):
"""Opens an HTTPS connection to a Gerrit service, and sends a request."""
headers = headers or {}
bare_host = host.partition(':')[0]
@@ -556,7 +556,7 @@
path = '%s&n=%d' % (path, limit)
if o_params:
path = '%s&%s' % (path, '&'.join(['o=%s' % p for p in o_params]))
- return ReadHttpJsonResponse(CreateHttpConn(host, path))
+ return ReadHttpJsonResponse(CreateHttpConn(host, path, timeout=30.0))
def GenerateAllChanges(host, params, first_param=None, limit=500,
@@ -1062,7 +1062,7 @@
if not body[key]:
raise GerritError(200, '%s is required' % key.title())
- conn = CreateHttpConn(host, path, reqtype='POST', body=body, timeout=None)
+ conn = CreateHttpConn(host, path, reqtype='POST', body=body)
return ReadHttpJsonResponse(conn, accept_statuses=[201])