Milo: Move instance configuration to luci-config

milo/appengine/buildbot/build.go

 // Copyright 2016 The LUCI Authors. All rights reserved.
 // Use of this source code is governed under the Apache License, Version 2.0
 // that can be found in the LICENSE file.
 
 package buildbot
 
 import (
         "encoding/json"
         "errors"
         "fmt"
@@ skipping 96 matching lines @@
         }()
         if osLogo != nil {
                 return &resp.LogoBanner{
                         OS: []resp.Logo{*osLogo},
                 }
         }
         logging.Warningf(c, "No OS info found.")
         return nil
 }
 
-// summary Extract the top level summary from a buildbot build as a
+// summary extracts the top level summary from a buildbot build as a
 // BuildComponent
 func summary(c context.Context, b *buildbotBuild) resp.BuildComponent {
         // TODO(hinoka): use b.toStatus()
         // Status
         var status resp.Status
         if b.Currentstep != nil {
                 status = resp.Running
         } else {
                 status = result2Status(b.Results)
         }
 
         // Timing info
         started, ended, duration := parseTimes(b.Times)
 
         // Link to bot and original build.
         server := "build.chromium.org/p"
         if b.Internal {
                 server = "uberchromegw.corp.google.com/i"
         }
         bot := &resp.Link{
                 Label: b.Slave,
                 URL:   fmt.Sprintf("https://%s/%s/buildslaves/%s", server, b.Master, b.Slave),
         }
         host := "build.chromium.org/p"
         if b.Internal {
                 host = "uberchromegw.corp.google.com/i"
         }
         source := &resp.Link{
                 Label: fmt.Sprintf("%s/%s/%d", b.Master, b.Buildername, b.Number),
-                // TODO(hinoka): Internal builds.
                 URL: fmt.Sprintf(
                         "https://%s/%s/builders/%s/builds/%d",
                         host, b.Master, b.Buildername, b.Number),
         }
 
         // The link to the builder page.
         parent := &resp.Link{
                 Label: b.Buildername,
                 URL:   ".",
         }
@@ skipping 486 matching lines @@
         var newAliases map[string][]*buildbotLinkAlias
         if l := remainingAliases.Len(); l > 0 {
                 newAliases = make(map[string][]*buildbotLinkAlias, l)
                 remainingAliases.Iter(func(v string) bool {
                         newAliases[v] = s.Aliases[v]
                         return true
                 })
         }
         s.Aliases = newAliases
 }