New issue
Advanced search Search tips

Issue 655296 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

Link to Milo instead of buildbot

Project Member Reported by seanmccullough@google.com, Oct 12 2016

Issue description

The SoM UI currently links to builds on buildbot (uberchromegw, rather) but we should switch over to linking to builds on Milo soon. It provides most of the same information at this point.

 
Project Member

Comment 1 by sheriffbot@chromium.org, Oct 13 2016

Labels: Hotlist-Google
"Most of the same information"

Is there any known losage?
hinoka@: Are internal builders visible on Milo yet?

Comment 4 by hinoka@chromium.org, Nov 15 2016

Yep.  All of them are except a couple of lingering Official bots (official android i think.  dnj pointed them out to me yesterday).  I'll work on getting those in today.

go to luci-milo.appspot.com, click the login button, and you can see all of the available masters
Cc: -seanmccullough@chromium.org
Owner: seanmccullough@chromium.org
Status: Started (was: Available)
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/275248daeb808f6a1ff7da313fe31d4108836069

commit 275248daeb808f6a1ff7da313fe31d4108836069
Author: Sean McCullough <seanmccullough@chromium.org>
Date: Tue Nov 15 22:53:23 2016

[som] Add an option to render links pointing to Milo for builds

This switches the uberify check box to radio buttons, three states:
- uberchromegw
- milo
- buildbot

BUG= 655296 

Change-Id: I39b10e489d88de136d0936ebe64a904e01ac558c
Reviewed-on: https://chromium-review.googlesource.com/411920
Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org>
Commit-Queue: Sean McCullough <seanmccullough@chromium.org>

[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/bower.json
[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/elements/som-alert-item.html
[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/elements/som-app.html
[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/elements/som-drawer.html
[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/elements/som-examine.html
[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure.html
[add] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/elements/som-linkify-behavior.html
[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/elements/som-settings.html
[delete] https://crrev.com/28dc8501fdcc702172d98514a56b8f9f94701459/go/src/infra/appengine/sheriff-o-matic/elements/som-uberify-behavior.html
[modify] https://crrev.com/275248daeb808f6a1ff7da313fe31d4108836069/go/src/infra/appengine/sheriff-o-matic/test/som-alert-item-test.html

Status: Fixed (was: Started)
This change is live on prod. You can use milo links on SoM now.

Sign in to add a comment