зеркало из https://github.com/golang/build.git
cmd/coordinator: remove redundant startTime global variable
We already had processStartTime. Change-Id: I3b1522d9b365ad06af19518354d2bde9af766f4c Reviewed-on: https://go-review.googlesource.com/22703 Reviewed-by: Andrew Gerrand <adg@golang.org>
This commit is contained in:
Родитель
51b13f2f04
Коммит
176f1071ee
|
@ -87,8 +87,6 @@ var (
|
|||
// not be used along with other locks)
|
||||
|
||||
var (
|
||||
startTime = time.Now()
|
||||
|
||||
statusMu sync.Mutex // guards the following four structures; see LOCK ORDER comment above
|
||||
status = map[builderRev]*buildStatus{}
|
||||
statusDone []*buildStatus // finished recently, capped to maxStatusDone
|
||||
|
@ -346,7 +344,7 @@ func main() {
|
|||
st.start()
|
||||
}
|
||||
case <-ticker.C:
|
||||
if numCurrentBuilds() == 0 && time.Now().After(startTime.Add(10*time.Minute)) {
|
||||
if numCurrentBuilds() == 0 && time.Now().After(processStartTime.Add(10*time.Minute)) {
|
||||
// TODO: halt the whole machine to kill the VM or something
|
||||
}
|
||||
}
|
||||
|
|
Загрузка…
Ссылка в новой задаче