Don't put CQ_EXTRA_TRYBOTS for disable CL's. |
|||
Issue descriptionWhat do you think about expanding the logic a bit here to not put the extra trybots in the case of simple disables? I'm not sure there's much point in sending these off to the perf trybots in cases where sheriffs are just disabling broken tests.
,
Jun 9 2016
+1 to what Prasad says. Doing static analysis on the code to know whether the CL is just a disable CL seems hard..
,
Jun 9 2016
FWIW I actually managed to typo the disabled annotation a few times and these bots caught my mistake.
,
Jun 9 2016
Ah ok, sounds like this is catching things :) Nevermind then!
,
Feb 3 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by pras...@chromium.org
, Jun 9 2016Components: -Tests Tests>AutoBisect