cmd/coordinator: populate commit label on benchmarks
This was lost in one of the many refactorings of CL 38306.
Updates golang/go#19871
Change-Id: I4514d4350feb1420861eae9a9dd021fee5b5ba6f
Reviewed-on: https://go-review.googlesource.com/43052
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
diff --git a/cmd/coordinator/coordinator.go b/cmd/coordinator/coordinator.go
index 05268ed..e647453 100644
--- a/cmd/coordinator/coordinator.go
+++ b/cmd/coordinator/coordinator.go
@@ -2780,6 +2780,7 @@
if !st.shouldBench() {
return nil
}
+ // We know rev and rev.Commit.Parents[0] exist because benchmarkItem.buildParent has checked.
rev := st.trySet.ci.Revisions[st.trySet.ci.CurrentRevision]
ps := rev.PatchSetNumber
benchFiles := []*benchFile{
@@ -2794,6 +2795,8 @@
benchFiles[0].out.WriteString("staging: true\n")
}
benchFiles[1].out.Write(benchFiles[0].out.Bytes())
+ fmt.Fprintf(&benchFiles[0].out, "commit: %s\n", rev.Commit.Parents[0].CommitID)
+ fmt.Fprintf(&benchFiles[1].out, "commit: %s\n", st.builderRev.rev)
return benchFiles
}