jschoubben
50c55a773e
mesh/merge-gate head 50c55a773eb8 merged into trunk c0c379af2b3c — pass: builds build-agent, mesh-controller → ace, g14, novox, shanks; no bus step; every …
mesh/repo-check head 50c55a773eb8 merged into trunk c0c379af2b3c — pass: its merge-check.sh passed
mesh/delivery delivered
mesh/delivery-group group fix/544-the-build-agent-says-it-is-ready delivered: every member is delivered
The agent listens on nothing, so the node-engine could judge it only alive and held it starting for the whole grace on every machine a walk moved it to. It now writes a mark once it is bound to its worker, after its seat verbs are served, and 'mesh-builder ready' passes only while the mark is there.
33 lines
870 B
Go
33 lines
870 B
Go
package main
|
|
|
|
import (
|
|
"os"
|
|
"testing"
|
|
)
|
|
|
|
// The health check passes only between the agent saying it takes work and its next start (novox/hq
|
|
// issue 544).
|
|
func TestTheAgentIsReadyOnlyOnceItTakesWork(t *testing.T) {
|
|
t.Setenv("TMPDIR", t.TempDir())
|
|
if err := ready(); err == nil {
|
|
t.Fatal("an agent that never took work is ready")
|
|
}
|
|
taking("mesh-build-machine")
|
|
if err := ready(); err != nil {
|
|
t.Fatalf("an agent taking work is not ready: %v", err)
|
|
}
|
|
// Started again, it is not ready until it binds again.
|
|
if err := notReady(); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if err := ready(); err == nil {
|
|
t.Fatal("an agent starting again is ready before it takes work")
|
|
}
|
|
if err := notReady(); err != nil {
|
|
t.Fatalf("removing a mark that is not there failed: %v", err)
|
|
}
|
|
if _, err := os.Stat(readyMark()); err == nil {
|
|
t.Fatal("the mark is still there")
|
|
}
|
|
}
|