catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.93k stars 564 forks source link

[📍] Make swarming "expired" a job-level error #4367

Closed dave-2 closed 6 years ago

dave-2 commented 6 years ago

Since "expired" is an execution-level error, Pinpoint treats it like a test failure, meaning it will try to bisect on it. If the swarming task expired because the bots are overloaded, that will just add insult to injury. It's also unclear to the user what's going on. Instead, the job should bail out completely, which will make the error clear and reduce the bot load.

@anniesullie @simonhatch

anniesullie commented 6 years ago

+1 this would be great!