Wrong trybot is triggered on retry |
|||||
Issue descriptionOn this CL (https://codereview.chromium.org/1878253003#ps1), tryserver.chromium.win:win_clang failed. I clicked the "Retry failed try jobs" link, and it tried tryserver.libyuv:win_clang instead. That obviously failed, and wasn't what I expected.
,
Aug 11 2016
,
Aug 11 2016
this is a bug in the rietveld UI that nobody maintains https://cs.chromium.org/chromium/infra/appengine/chromium_rietveld/new_static/components/cr-issue-patchset.html?q=retry-failed-try-jobs&sq=package:chromium&dr=C&l=131 saves builder name without master name into "failedBuilders" property and that uses this property to retry builders https://cs.chromium.org/chromium/infra/appengine/chromium_rietveld/new_static/components/cr-issue-patchset.html?q=retry-failed-try-jobs&sq=package:chromium&dr=C&l=160 this code assumes that builder names are unique which is not true
,
Oct 20 2016
,
Oct 23 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 23 2017
Rietveld is deprecated, so wontfix |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by aga...@chromium.org
, Apr 27 2016Labels: -Infra-Codereview