mirror of
https://github.com/golang/go
synced 2024-11-12 01:00:22 -07:00
builder: send commit time to dashboard
R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/5489084
This commit is contained in:
parent
0b28de9a05
commit
9b3799aa89
@ -8,10 +8,12 @@ import (
|
||||
"bytes"
|
||||
"encoding/json"
|
||||
"errors"
|
||||
"fmt"
|
||||
"io"
|
||||
"log"
|
||||
"net/http"
|
||||
"net/url"
|
||||
"time"
|
||||
)
|
||||
|
||||
type obj map[string]interface{}
|
||||
@ -147,20 +149,19 @@ func (b *Builder) updatePackage(pkg string, ok bool, buildLog, info string) erro
|
||||
*/
|
||||
}
|
||||
|
||||
func postCommit(key, pkg string, l *HgLog) bool {
|
||||
err := dash("POST", "commit", url.Values{"key": {key}}, obj{
|
||||
func postCommit(key, pkg string, l *HgLog) error {
|
||||
t, err := time.Parse(time.RFC3339, l.Date)
|
||||
if err != nil {
|
||||
return fmt.Errorf("parsing %q: %v", l.Date, t)
|
||||
}
|
||||
return dash("POST", "commit", url.Values{"key": {key}}, obj{
|
||||
"PackagePath": pkg,
|
||||
"Hash": l.Hash,
|
||||
"ParentHash": l.Parent,
|
||||
// TODO(adg): l.Date as int64 unix epoch secs in Time field
|
||||
"Time": t.Unix() * 1e6, // in microseconds, yuck!
|
||||
"User": l.Author,
|
||||
"Desc": l.Desc,
|
||||
}, nil)
|
||||
if err != nil {
|
||||
log.Printf("failed to add %s to dashboard: %v", key, err)
|
||||
return false
|
||||
}
|
||||
return true
|
||||
}
|
||||
|
||||
func dashboardCommit(pkg, hash string) bool {
|
||||
|
@ -536,7 +536,7 @@ const xmlLogTemplate = `
|
||||
<hash>{node|escape}</hash>
|
||||
<parent>{parent|escape}</parent>
|
||||
<author>{author|escape}</author>
|
||||
<date>{date}</date>
|
||||
<date>{date|rfc3339date}</date>
|
||||
<desc>{desc|escape}</desc>
|
||||
</log>
|
||||
`
|
||||
@ -652,7 +652,11 @@ func addCommit(pkg, hash, key string) bool {
|
||||
}
|
||||
|
||||
// Create commit.
|
||||
return postCommit(key, pkg, l)
|
||||
if err := postCommit(key, pkg, l); err != nil {
|
||||
log.Printf("failed to add %s to dashboard: %v", key, err)
|
||||
return false
|
||||
}
|
||||
return true
|
||||
}
|
||||
|
||||
// fullHash returns the full hash for the given Mercurial revision.
|
||||
|
Loading…
Reference in New Issue
Block a user