New issue
Advanced search Search tips

Issue 602825 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Wrong trybot is triggered on retry

Project Member Reported by jbroman@chromium.org, Apr 12 2016

Issue description

On 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.
 

Comment 1 by aga...@chromium.org, Apr 27 2016

Components: Infra>Codereview
Labels: -Infra-Codereview
Blocking: 618023
Cc: no...@chromium.org
Status: Available (was: Untriaged)

Comment 3 by no...@chromium.org, 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
Blocking: -618023
Project Member

Comment 5 by sheriffbot@chromium.org, Oct 23 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
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

Comment 6 by no...@chromium.org, Oct 23 2017

Status: WontFix (was: Untriaged)
Rietveld is deprecated, so wontfix

Sign in to add a comment