Milo: Add machine pool info for buildbot builder view.

milo/appengine/buildbot/builder.go

Index: milo/appengine/buildbot/builder.go
diff --git a/milo/appengine/buildbot/builder.go b/milo/appengine/buildbot/builder.go
index d8971b1b58786d73accc4616d774eab83f4bb5a6..4bbb9878c0d5324601715cdda0cb68c9b80af7fe 100644
--- a/milo/appengine/buildbot/builder.go
+++ b/milo/appengine/buildbot/builder.go
@@ -164,6 +164,39 @@ func (e errBuilderNotFound) Error() string {
 		e.builder, e.master, avail)
 }
 
+func summarizeSlavePool(
+	baseURL string, slaves []string, slaveMap map[string]*buildbotSlave) *resp.MachinePool {
+
+	mp := &resp.MachinePool{
+		Total: len(slaves),
+		Bots:  make([]resp.Bot, 0, len(slaves)),
+	}
+	for _, slaveName := range slaves {
+		slave, ok := slaveMap[slaveName]
+		bot := resp.Bot{
+			Name: &resp.Link{
+				Label: slaveName,
+				URL:   fmt.Sprintf("%s/buildslaves/%s", baseURL, slaveName),
+			},
+		}
+		switch {
+		case !ok:
+			// This shouldn't happen

[iannucci, 2017/06/16 01:44:16]

what will this render like? unknown?

[Ryan Tseng, 2017/06/16 02:05:42]

It just wouldn't show up in the bar graph, and idle + busy + disconnect < total.
 But it shouldn't happen :D.

In the entry, it should render "UnknownStatus" Since that's the status string
text, with whatever the default color is.

+		case !slave.Connected:
+			bot.Status = resp.Disconnected
+			mp.Disconnected++
+		case len(slave.RunningbuildsMap) > 0:
+			bot.Status = resp.Busy
+			mp.Busy++
+		default:
+			bot.Status = resp.Idle
+			mp.Idle++
+		}
+		mp.Bots = append(mp.Bots, bot)
+	}
+	return mp
+}
+
 // builderImpl is the implementation for getting a milo builder page from buildbot.
 // This gets:
 // * Current Builds from querying the master json from the datastore.
@@ -184,7 +217,7 @@ func builderImpl(
 	result := &resp.Builder{
 		Name: builderName,
 	}
-	master, t, err := getMasterJSON(c, masterName)
+	master, internal, t, err := getMasterJSON(c, masterName)
 	if err != nil {
 		return nil, err
 	}
@@ -226,6 +259,12 @@ func builderImpl(
 		}
 	}
 
+	baseURL := "https://build.chromium.org/p/"
+	if internal {
+		baseURL = "https://uberchromegw.corp.google.com/i/"
+	}
+	result.MachinePool = summarizeSlavePool(baseURL+master.Name, p.Slaves, master.Slaves)
+
 	// This is CPU bound anyways, so there's no need to do this in parallel.
 	finishedBuilds, nextCursor, err := getBuilds(c, masterName, builderName, true, limit, thisCursor)
 	if err != nil {