Commit 760f06cd authored by Roman Sorokin's avatar Roman Sorokin Committed by Commit Bot

bisect-builds.py: Allow retry during range verification

I hit it several times - wanted to retry during range verification but
the script just exits with
'Unexpected result at a range boundary! Your range is not correct.'

Bug: none
Change-Id: Ib2f89c3c8ece170a1527ecc8518f7d7e3dfa4c27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1963794
Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org>
Reviewed-by: default avatarRoberto Carrillo <robertocn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#727358}
parent f7aa7420
...@@ -738,12 +738,16 @@ def VerifyEndpoint(fetch, context, rev, profile, num_runs, command, try_args, ...@@ -738,12 +738,16 @@ def VerifyEndpoint(fetch, context, rev, profile, num_runs, command, try_args,
evaluate, expected_answer): evaluate, expected_answer):
fetch.WaitFor() fetch.WaitFor()
try: try:
(exit_status, stdout, stderr) = RunRevision( answer = 'r'
context, rev, fetch.zip_file, profile, num_runs, command, try_args) # This is intended to allow evaluate() to return 'r' to retry RunRevision.
while answer == 'r':
(exit_status, stdout, stderr) = RunRevision(
context, rev, fetch.zip_file, profile, num_runs, command, try_args)
answer = evaluate(rev, exit_status, stdout, stderr);
except Exception, e: except Exception, e:
print(e, file=sys.stderr) print(e, file=sys.stderr)
raise SystemExit raise SystemExit
if (evaluate(rev, exit_status, stdout, stderr) != expected_answer): if (answer != expected_answer):
print('Unexpected result at a range boundary! Your range is not correct.') print('Unexpected result at a range boundary! Your range is not correct.')
raise SystemExit raise SystemExit
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment