Use Milo links to PreCQ builds when annotating CLs |
|||||||||
Issue descriptionWe currently use links to buildbot when linking to PreCQ buildslaves as we annotate CLs. We should use Milo links instead. Here is a somewhat related CL: https://chromium-review.googlesource.com/#/c/411513/6 Example annotation today: The Pre-Commit Queue has picked up your change. You can follow along at (mixed-b-pre-cq) : https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/compile_only_pre_cq/builds/12980 (binhost-pre-cq) : https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/pre_cq/builds/9384 (mixed-c-pre-cq) : https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/compile_only_pre_cq/builds/12979 (mixed-a-pre-cq) : https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/compile_only_pre_cq/builds/12981 (rambi-pre-cq) : https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/pre_cq/builds/9385 . Commit queue documentation: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium-os/commit-queue-overview
,
Dec 13 2016
is there work remaining on this? or is this done now?
,
Jan 3 2017
@dgarrett - is there work remaining on this? or is this done now?
,
Jan 4 2017
PreCQLuancherStage: Correctly link to milo for PreCQ builds. We were using the dashboard URL, not the waterfall name when constructing URLs for PreCQ tryjobs, which was no longer correct. BUG= chromium:671861 TEST=None. Sadly, there are no unittests. Change-Id: I48be754f25b7de35f7a8d90663dd634e56e9c021 Reviewed-on: https://chromium-review.googlesource.com/417406 Tested-by: Don Garrett <dgarrett@chromium.org> Tested-by: Prathmesh Prabhu <pprabhu@chromium.org> Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org> Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org> Commit-Queue: Don Garrett <dgarrett@chromium.org>
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by dgarr...@chromium.org
, Dec 6 2016