summaryrefslogtreecommitdiff
path: root/misc/dashboard
Commit message (Collapse)AuthorAgeFilesLines
* misc/dashboard/app, misc/dashboard/builder: delete from main repo; part of ↵Chris Manghane2013-08-0120-3106/+0
| | | | | | | | | | | | move to go.tools See also https://codereview.appspot.com/12180043/ R=adg CC=golang-dev https://codereview.appspot.com/12213043 Committer: Andrew Gerrand <adg@golang.org>
* misc/dashboard/builder: increase cmdTimeout to 10 minutesAlex Brainman2013-07-291-1/+1
| | | | | | | | | It takes more then 5 minutes to clone go repo on windows-386 builder now. R=golang-dev, dave CC=golang-dev https://codereview.appspot.com/12007043
* misc/dashboard: don't update tip tag for release branch commits.David Symonds2013-07-291-2/+3
| | | | | | | | | | | | | This will mean that sub-repositories won't get built against the release branch. They are often not compatible because the subrepos often run ahead of the current release (e.g. go.tools is using new additions to go/ast, and go.net is using new things in syscall) so there's little point in checking them against cherrypick commits when they'll be tested against those commits on tip anyway. R=golang-dev, adg CC=golang-dev https://codereview.appspot.com/12001043
* misc/dashboard: fix typo in commentDmitriy Vyukov2013-07-271-1/+1
| | | | | | R=golang-dev, r CC=golang-dev https://codereview.appspot.com/11936045
* misc/dashboard: show all race builders as "race"Dmitriy Vyukov2013-06-261-1/+1
| | | | | | | | | race is more important than arch (moreover race implies x64) don't know how to test it R=golang-dev, dave, r CC=golang-dev https://codereview.appspot.com/10484046
* misc/dashboard/builder: add environment variables for Plan 9Anthony Martin2013-05-311-1/+8
| | | | | | | | | | We require $objtype in make.rc and rc needs $path for finding commands. Also include $cputype which we may use in the future. R=golang-dev, minux.ma, r CC=golang-dev https://codereview.appspot.com/9905043
* codereview: close the head tagBrad Fitzpatrick2013-05-231-1/+1
| | | | | | R=golang-dev, dsymonds CC=golang-dev https://codereview.appspot.com/9695045
* codereview: add khr as a personBrad Fitzpatrick2013-05-211-0/+1
| | | | | | R=golang-dev, gri CC=golang-dev https://codereview.appspot.com/9459046
* misc/dashboard/builder: handle Plan 9 in defaultSuffix()Shenghou Ma2013-05-211-2/+6
| | | | | | R=golang-dev, bradfitz CC=golang-dev https://codereview.appspot.com/9495045
* misc/dashboard: add go.tools to list of subreposDave Cheney2013-05-191-0/+1
| | | | | | R=r, adg, dsymonds CC=golang-dev https://codereview.appspot.com/9535044
* misc/dashboard/codereview: add cshapiro and mpvl to reviewers list.Rob Pike2013-05-151-0/+2
| | | | | | R=golang-dev, gri CC=golang-dev https://codereview.appspot.com/9104052
* misc/dashboard/builder: always do a native buildDave Cheney2013-04-181-2/+4
| | | | | | | | | | | | https://codereview.appspot.com/8134043 disabled cgo when cross compiling, this means builders which compile for both amd64 and 386 will be compiling the latter with cgo disabled. This proposal modifies the builder to mirror the dist tool by always doing a native build. Tested on my darwin/amd64 builder and confirmed the result when building darwin/386 is a native 386 build with cgo enabled. R=bradfitz, dsymonds, r, adg CC=golang-dev https://codereview.appspot.com/8842044
* misc/dashboard/builder: include $USER in build environment.David Symonds2013-04-181-0/+1
| | | | | | R=golang-dev, r CC=golang-dev https://codereview.appspot.com/8821046
* misc/dashboard/builder: add -race builder supportDave Cheney2013-04-091-1/+12
| | | | | | | | If the build key contains -race, the builder will invoke to the race.{bat,bash} build command. This allows {darwin,linux,windows}-amd64 builders to do race and non race builds in sequence. R=adg, dvyukov, fullung CC=golang-dev https://codereview.appspot.com/8266046
* misc/dashboard/codereview: add R?my to the list of assigneesDave Cheney2013-03-301-1/+2
| | | | | | R=dsymonds, mikioh.mikioh CC=golang-dev https://codereview.appspot.com/8094047
* misc/dashboard: show correct dir name in READMERobert Griesemer2013-03-191-2/+2
| | | | | | R=golang-dev, bradfitz CC=golang-dev https://codereview.appspot.com/7909043
* misc/dashboard/codereview: better debugging when some fetches fail.David Symonds2013-03-151-3/+15
| | | | | | R=rsc CC=golang-dev https://codereview.appspot.com/7836045
* misc/dashboard/codereview: add more people.David Symonds2013-03-151-0/+19
| | | | | | | | I picked everyone who has committed a CL on behalf of someone else in recent times. R=r, rsc CC=golang-dev https://codereview.appspot.com/7836044
* misc/dashboard: add go.blog sub-repo to dashboardAndrew Gerrand2013-03-111-1/+2
| | | | | | R=golang-dev, dsymonds CC=golang-dev https://codereview.appspot.com/7690043
* misc/dashboard/builder: make -fail mode fasterRuss Cox2013-03-041-1/+1
| | | | | | | | Don't check out the repository if we're just going to fail everything. R=golang-dev, bradfitz CC=golang-dev https://codereview.appspot.com/7453048
* misc/dashboard/app: make the builders test go.talks and go.expShenghou Ma2013-03-032-1/+3
| | | | | | R=golang-dev, r CC=golang-dev https://codereview.appspot.com/7419048
* misc/dashboard/builder: various cleanupsDave Cheney2013-02-212-190/+211
| | | | | | | | | | * allow commit watcher to be disabled, useful for small slow builders who will never be the first to notice a commit. * builders always update their local master working copy before cloning a specific revision. * refactor hg repo operations into a new type, Repo. R=adg, shanemhansen, luitvd CC=golang-dev https://codereview.appspot.com/7326053
* cmd/godoc: use go/build to determine package and example filesRobert Griesemer2013-02-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Also: - faster code for example extraction - simplify handling of command documentation: all "main" packages are treated as commands - various minor cleanups along the way For commands written in Go, any doc.go file containing documentation must now be part of package main (rather then package documentation), otherwise the documentation won't show up in godoc (it will still build, though). For commands written in C, documentation may still be in doc.go files defining package documentation, but the recommended way is to explicitly ignore those files with a +build ignore constraint to define package main. Fixes issue 4806. R=adg, rsc, dave, bradfitz CC=golang-dev https://codereview.appspot.com/7333046
* misc/dashboard/builder: record build result on dashboardDave Cheney2013-02-152-37/+49
| | | | | | | | | | | | This is part one of two changes intended to make it easier to debug builder failures. runOutput allows us to control the io.Writer passed to a subcommand. The intention is to add additional debugging information before and after the build which will then be capture and sent to the dashboard. In this proposal, the only additional information is the build status. See http://build.golang.org/log/e7b5bf435b4de1913fc61781b3295fb3f03aeb6e R=adg CC=golang-dev https://codereview.appspot.com/7303090
* misc/dashboard: add missing return valueAndrew Gerrand2013-02-111-1/+1
| | | | | | R=golang-dev, bradfitz CC=golang-dev https://codereview.appspot.com/7300084
* misc/dashboard/app: trim old builds from the historyDave Cheney2013-02-051-1/+17
| | | | | | | | The dashboard is currently failing to store results of new builds for some keys, notable the go.codereview sub repository. This is causing the builders to mark the entire triggering commit as failed. With the help of David Symonds we think it is because the results value has breached the 1mb datastore limit on AppEngine. R=dsymonds, adg CC=golang-dev https://codereview.appspot.com/6858094
* misc/dashboard/builder: synchronize accesses to goroot, always -commitAndrew Gerrand2013-01-251-25/+65
| | | | | | | | | | | | | This prevents the occasional issue when Mercurial screws up the locking itself, and by moving the locking into this process we can use the goroot for other things (such as automatically updating the builder binary). It also asks all builders to poll for new commits. R=bradfitz, dave, minux.ma, rsc CC=golang-dev https://codereview.appspot.com/7178046
* misc/dashboard/godashboard: deleteAndrew Gerrand2013-01-1810-502/+0
| | | | | | | | This code is obsolete and unmaintained. R=bradfitz CC=golang-dev https://codereview.appspot.com/7135056
* misc/dashboard/codereview: add campoy to list of gophersAndrew Gerrand2013-01-141-0/+1
| | | | | | R=dsymonds CC=golang-dev https://codereview.appspot.com/7106044
* misc/dashboard/builder: set GOPATH before building subreposDave Cheney2013-01-071-19/+15
| | | | | | | | | | | | | | This proposal updates the dashboard builder to avoid relying on the (soon to be removed) support for using go get to download to $GOROOT. The result is WORKSPACE=$(the value of the -buildRoot flag / $BUILDER_NAME + hg revision) GOROOT=$WORKSPACE/go GOPATH=$WORKSPACE Required for CL 6941058. R=minux.ma, adg CC=golang-dev https://codereview.appspot.com/7034049
* misc/dashboard: don't email about netbsd failuresBrad Fitzpatrick2012-12-121-0/+11
| | | | | | | | Too flaky. R=iant, rsc CC=golang-dev https://codereview.appspot.com/6925056
* dashboard/builder: pass $CC to builder if setDave Cheney2012-11-221-0/+1
| | | | | | R=adg, minux.ma, rsc CC=golang-dev http://codereview.appspot.com/6843068
* gofmt: apply gofmt -w src miscRobert Griesemer2012-10-305-9/+9
| | | | | | | | | Remove trailing whitespace in comments. No other changes. R=r CC=golang-dev http://codereview.appspot.com/6815053
* misc/dashboard/builder: check http status before processing responseDave Cheney2012-10-311-1/+3
| | | | | | | | Occasionally GAE will return a 500 error response, don't treat this as a valid JSON body. R=adg, dsymonds CC=golang-dev http://codereview.appspot.com/6775066
* misc/dashboard/codereview: only accept "NOT LGTM" on the first line of a ↵David Symonds2012-10-291-1/+1
| | | | | | | | | | | message. Too many people quote entire emails and put their reply at the top ("top posting"), so we shouldn't recognise review commands anywhere in the review text. R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/6815048
* builder: label the race builder as "race"Brad Fitzpatrick2012-10-151-0/+3
| | | | | | R=golang-dev, adg, dave, rsc, minux.ma, dvyukov CC=golang-dev http://codereview.appspot.com/6648043
* misc/dashboard: set charset utf-8 on build log downloadsBrad Fitzpatrick2012-10-031-1/+1
| | | | | | | | Fixes issue 4187 R=golang-dev, dave, minux.ma CC=golang-dev http://codereview.appspot.com/6587071
* misc/dashboard/builder: reinstate 'go get -d' error handling hackAndrew Gerrand2012-09-171-1/+7
| | | | | | | | | I thought this was redundant since the behavior of 'go get -d' had changed. I was wrong. Should have tested more thoroughly. R=golang-dev, minux.ma CC=golang-dev http://codereview.appspot.com/6500136
* misc/dashboard/builder: increase default command timeoutAndrew Gerrand2012-09-171-1/+1
| | | | | | R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/6499127
* misc/dashboard/builder: add timeout to all external command invocationsAndrew Gerrand2012-09-172-22/+38
| | | | | | | | Fixes issue 4083. R=golang-dev, r CC=golang-dev http://codereview.appspot.com/6498136
* misc/dashboard/builder: use c:\ as default buildroot on windowsAlex Brainman2012-09-141-1/+14
| | | | | | | | | | We have some tests (misc/cgo/test) that are disabled only because they will fail to run on go builder - see issue 3358 for details. This change will allow us to enable these tests. R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/6493118
* misc/dashboard/codereview: fixes for non-reviewers.David Symonds2012-08-082-11/+24
| | | | | | | | Also rename the testing CGI argument from "email" to "user". R=golang-dev, r CC=golang-dev http://codereview.appspot.com/6454117
* misc/dashboard/codereview: fix oboRuss Cox2012-08-081-3/+3
| | | | | | | | R=golang-dev, dsymonds CC=golang-dev http://codereview.appspot.com/6443091 Committer: David Symonds <dsymonds@golang.org>
* misc/dashboard/codereview: show first line of last message in threadRuss Cox2012-08-052-34/+142
| | | | | | | | | | | | | | | | This line helps me to tell whether the CL is waiting for me or I'm waiting for the author. Also: - vertical-align table cells so buttons are always aligned with CL headers. - add email= to show front page for someone else. Demo at http://rsc.gocodereview.appspot.com/. Until this is deployed for real, some recently changed CLs may be missing the 'first line of last message' part. R=dsymonds CC=golang-dev http://codereview.appspot.com/6446065
* misc/dashboard/codereview: add LastUpdate field to CLRuss Cox2012-07-301-0/+1
| | | | | | | | | | | | | | | | | | | | To be filled in by a later CL. I deployed a test version to App Engine to work on setting the values, so there are a few records that have this field set already. That field is breaking the live version, so I have pushed a new copy with this 1-line change to the live version I assumed that appengine/datastore was like every other marshaling and unmarshaling package we have in Go (for example, encoding/gob, encoding/json, encoding/xml, and protobuf) and that if it loaded an unknown field it would just ignore it. Apparently not. Sorry. R=dsymonds TBR=dsymonds CC=golang-dev http://codereview.appspot.com/6454064
* misc/dashboard/codereview: pass user information to gobot when assigning ↵David Symonds2012-07-301-2/+3
| | | | | | | | reviewer. R=bradfitz CC=gobot, golang-dev, rsc http://codereview.appspot.com/6457052
* misc/dashboard/codereview: switch to using gobot to update CL reviewer info.David Symonds2012-07-301-25/+11
| | | | | | R=rsc CC=golang-dev http://codereview.appspot.com/6453063
* misc/dashboard/codereview: bump user-specific table limits to 100.David Symonds2012-07-301-2/+5
| | | | | | R=rsc CC=golang-dev http://codereview.appspot.com/6457051
* misc/dashboard/codereview: interpret zero reviewers as the CL being closed.David Symonds2012-07-301-6/+14
| | | | | | | | | This is the state when the CL has been submitted by someone other than the CL author, but before the CL author has synched their client. R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/6458044
* misc/dashboard/codereview: make all active CL tables hold up to 50 CLs.David Symonds2012-07-301-2/+2
| | | | | | R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/6449060
View Lorry Controller Status