builder page does not have links to Gerrit CLs |
||||
Issue descriptionbuilder page does not have links to Gerrit CLs. For example, https://luci-milo.appspot.com/buildbucket/luci.infra.try/Infra%20Win%20Tester displays links to Rietveld CLs, but not Gerrit
,
Mar 20 2017
,
Mar 20 2017
,
Mar 27 2017
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/9c353a31df60535ce05509c32a6671ba84c3f2a5 commit 9c353a31df60535ce05509c32a6671ba84c3f2a5 Author: hinoka <hinoka@google.com> Date: Mon Mar 27 18:30:23 2017 Milo: Fix Gerrit CL reporting in buildbucket view Turns out the properties were named wrong BUG= 687329 Review-Url: https://codereview.chromium.org/2768203002 [modify] https://crrev.com/9c353a31df60535ce05509c32a6671ba84c3f2a5/milo/appengine/buildbucket/common.go [modify] https://crrev.com/9c353a31df60535ce05509c32a6671ba84c3f2a5/milo/appengine/buildbucket/properties.go
,
Apr 22 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by no...@chromium.org
, Mar 10 2017