New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 680530 link

Starred by 2 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 680223


Show other hotlists

Hotlists containing this issue:
speed-bisect


Sign in to add a comment

Bisect - Handle failing to expand revisions more gracefully.

Project Member Reported by simonhatch@chromium.org, Jan 12 2017

Issue description

https://bugs.chromium.org/p/chromium/issues/detail?id=680223#c3

Here someone put in the wrong revision range, we should handle bad input like that more gracefully and output a helpful error.
 
Blocking: 680223
Cc: sullivan@chromium.org
We probably have enough information on the dashboard so that given two chrome commit positions A and B, to find, say, the first clank hash A' with src@A and the last clank hash B' with src@B.

Could that be doable? For humans entering manual ranges it's often much easier to think about commit positions rather than, e.g., clank hashes.
Cc: simonhatch@chromium.org jessimb@chromium.org
Simon: for background on how the dashboard deals with clank revisions, see the discussion in https://github.com/catapult-project/catapult/issues/3014

If a_default_rev were set properly, maybe we could have special handling of multiple revision types in the bisect dialog? It would definitely add complesity, so would depend how big a problem this is.
So if I'm understanding correctly, the suggestion is if you're on clank and input chromium revisions, to auto-match them to when clank rolled them and fill in the corresponding clank hashes?
Components: Speed>Bisection

Sign in to add a comment