Bug 1652572 - [taskgraph] Temporarily double the bugbug timeout on try to work around perf bug, r=marco

Differential Revision: https://phabricator.services.mozilla.com/D84219
This commit is contained in:
Andrew Halberstadt 2020-07-20 20:37:58 +00:00
Родитель 2003a00dfc
Коммит 357240a389
1 изменённых файлов: 11 добавлений и 2 удалений

Просмотреть файл

@ -22,7 +22,7 @@ except ImportError:
BUGBUG_BASE_URL = "https://bugbug.herokuapp.com"
RETRY_TIMEOUT = 8 * 60 # seconds
RETRY_INTERVAL = 10 # seconds
RETRY_INTERVAL = 10 # seconds
# Preset confidence thresholds.
CT_LOW = 0.7
@ -79,7 +79,16 @@ def push_schedules(branch, rev):
url = BUGBUG_BASE_URL + '/push/{branch}/{rev}/schedules'.format(branch=branch, rev=rev)
start = monotonic()
session = get_session()
attempts = RETRY_TIMEOUT / RETRY_INTERVAL
# TODO Sometimes the call can take much longer due to an issue pulling from
# mercurial. Double the timeout on try (where there is no fallback) to
# compensate. Remove once https://github.com/mozilla/bugbug/issues/1673 is
# fixed.
timeout = RETRY_TIMEOUT
if branch == "try":
timeout *= 2
attempts = timeout / RETRY_INTERVAL
i = 0
while i < attempts:
r = session.get(url)