Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(976)

Unified Diff: milo/appengine/buildbot/builder.go

Issue 2929173002: Milo: Use UTC representation for commit time in blamelist (Closed)
Patch Set: Also builder Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: milo/appengine/buildbot/builder.go
diff --git a/milo/appengine/buildbot/builder.go b/milo/appengine/buildbot/builder.go
index ad545d3b19623b818bd4b698f206d40804ca85ee..d8971b1b58786d73accc4616d774eab83f4bb5a6 100644
--- a/milo/appengine/buildbot/builder.go
+++ b/milo/appengine/buildbot/builder.go
@@ -210,11 +210,11 @@ func builderImpl(
result.PendingBuilds = make([]*resp.BuildSummary, len(p.PendingBuildStates))
logging.Debugf(c, "Number of pending builds: %d", len(p.PendingBuildStates))
for i, pb := range p.PendingBuildStates {
- start := time.Unix(int64(pb.SubmittedAt), 0)
+ start := time.Unix(int64(pb.SubmittedAt), 0).UTC()
result.PendingBuilds[i] = &resp.BuildSummary{
PendingTime: resp.Interval{
Started: start,
- Duration: time.Now().Sub(start),
+ Duration: clock.Now(c).UTC().Sub(start),
},
}
result.PendingBuilds[i].Blame = make([]*resp.Commit, len(pb.Source.Changes))
« no previous file with comments | « milo/appengine/buildbot/build_test.go ('k') | milo/appengine/buildbot/expectations/CrWinGoma.30608.build.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698