[errors] de-specialize Transient in favor of Tags.

milo/build_source/buildbucket/builder.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 buildbucket
 
 import (
         "encoding/json"
         "fmt"
         "net/url"
@@ skipping 25 matching lines @@
         *buildbucket.ApiSearchResponseMessage, error) {
 
         var res *buildbucket.ApiSearchResponseMessage
         err := retry.Retry(
                 c,
                 retry.TransientOnly(retry.Default),
                 func() error {
                         var err error
                         res, err = req.Do()
                         if apiErr, ok := err.(*googleapi.Error); ok && apiErr.Code >= 500 {
-                                err = errors.WrapTransient(apiErr)
+                                err = retry.Tag.Apply(apiErr)
                         }
                         return err
                 },
                 func(err error, wait time.Duration) {
                         logging.WithError(err).Warningf(c, "buildbucket search request failed transiently, will retry in %s", wait)
                 })
         return res, err
 }
 
 // fetchBuilds fetches builds given a criteria.
@@ skipping 230 matching lines @@
         return time.Unix(microseconds/1e6, microseconds%1e6*1000).UTC()
 }
 
 type newBuildsFirst []*resp.BuildSummary
 
 func (a newBuildsFirst) Len() int      { return len(a) }
 func (a newBuildsFirst) Swap(i, j int) { a[i], a[j] = a[j], a[i] }
 func (a newBuildsFirst) Less(i, j int) bool {
         return a[i].PendingTime.Started.After(a[j].PendingTime.Started)
 }