cmd/coordinator: remove unused functions
Change-Id: Ida78c42f56e1de26e8271e3dcec34aebcb137508
Reviewed-on: https://go-review.googlesource.com/c/build/+/375937
Trust: Michael Pratt <mpratt@google.com>
Run-TryBot: Michael Pratt <mpratt@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Alex Rakoczy <alex@golang.org>
diff --git a/cmd/coordinator/buildstatus.go b/cmd/coordinator/buildstatus.go
index 076d70f..5968a1a 100644
--- a/cmd/coordinator/buildstatus.go
+++ b/cmd/coordinator/buildstatus.go
@@ -218,12 +218,6 @@
return pool.ForHost(st.conf.HostConfig())
}
-// parentRev returns the parent of this build's commit (but only if this build comes from a trySet).
-func (st *buildStatus) parentRev() (pbr buildgo.BuilderRev, err error) {
- err = errors.New("TODO: query maintner")
- return
-}
-
func (st *buildStatus) expectedMakeBashDuration() time.Duration {
// TODO: base this on historical measurements, instead of statically configured.
// TODO: move this to dashboard/builders.go? But once we based on on historical
diff --git a/cmd/coordinator/coordinator.go b/cmd/coordinator/coordinator.go
index fa669bd..f4b9c5e 100644
--- a/cmd/coordinator/coordinator.go
+++ b/cmd/coordinator/coordinator.go
@@ -1940,15 +1940,6 @@
}
}
-func (ti *testItem) isDone() bool {
- select {
- case <-ti.done:
- return true
- default:
- return false
- }
-}
-
// retry reschedules the test to run again, if a machine died before
// or during execution, so its results aren't yet known.
// The caller must own the 'take' semaphore.
diff --git a/cmd/coordinator/coordinator_test.go b/cmd/coordinator/coordinator_test.go
index e0592d5..22087f6 100644
--- a/cmd/coordinator/coordinator_test.go
+++ b/cmd/coordinator/coordinator_test.go
@@ -20,7 +20,6 @@
"time"
"golang.org/x/build/buildenv"
- "golang.org/x/build/dashboard"
"golang.org/x/build/gerrit"
"golang.org/x/build/internal/buildgo"
"golang.org/x/build/internal/coordinator/pool"
@@ -322,14 +321,6 @@
}
}
-func mustConf(t *testing.T, name string) *dashboard.BuildConfig {
- conf, ok := dashboard.Builders[name]
- if !ok {
- t.Fatalf("unknown builder %q", name)
- }
- return conf
-}
-
func TestSlowBotsFromComments(t *testing.T) {
work := &apipb.GerritTryWorkItem{
Version: 2,
