Compare commits
24
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
7273ca2f0f | ||
|
|
ad797d8742 | ||
|
|
5b39e95361 | ||
|
|
7e4a0ecf9a | ||
|
|
7661f57833 | ||
|
|
076e0ae259 | ||
|
|
a96e2f0d78 | ||
|
|
17f7cb0d9c | ||
|
|
f6685ed22d | ||
|
|
52c18f7a45 | ||
|
|
fa7415fcd0 | ||
|
|
f8947a806d | ||
|
|
b98e503a61 | ||
|
|
5b832918df | ||
|
|
17bbcc1596 | ||
|
|
29be985c23 | ||
|
|
05d977666a | ||
|
|
e871991495 | ||
|
|
f9e19814eb | ||
|
|
af31315a5f | ||
|
|
474f68b34c | ||
|
|
1a724f20fe | ||
|
|
0da0bb2157 | ||
|
|
118e333ff8 |
+23
-11
@@ -148,20 +148,34 @@ func answer(ctx context.Context, publisher builder.Publisher, on, workspace stri
|
|||||||
// it either finishes or fails is indistinguishable from one that never arrived — which cost a long
|
// it either finishes or fails is indistinguishable from one that never arrived — which cost a long
|
||||||
// diagnosis against a running mesh, chasing "the handler never fired" when the truth was only that
|
// diagnosis against a running mesh, chasing "the handler never fired" when the truth was only that
|
||||||
// the handler said nothing until the end.
|
// the handler said nothing until the end.
|
||||||
fmt.Fprintf(os.Stderr, "a build request arrived for %s\n", request.Repository)
|
fmt.Fprintf(os.Stderr, "a build request arrived for %s (%s)\n", request.Repository, request.ID)
|
||||||
|
|
||||||
|
// **Everything a build says goes two ways**: to stderr, as always, and onto the bus as the
|
||||||
|
// role's own events under the build's id (novox/hq ADR 0157) — so whoever asked, and anybody
|
||||||
|
// watching, reads the same lines this container's log holds, live, and after the fact from the
|
||||||
|
// stream. Said first, before anything runs, so a build that hangs is one that visibly started.
|
||||||
|
say := func(step, message string) {
|
||||||
|
fmt.Fprintf(os.Stderr, " [%s] %s\n", step, message)
|
||||||
|
work.Say(step, message)
|
||||||
|
}
|
||||||
|
builder.Said = say
|
||||||
|
defer func() { builder.Said = nil }()
|
||||||
|
if err := work.Began(ctx); err != nil {
|
||||||
|
fmt.Fprintf(os.Stderr, "cannot say a build started: %v\n", err)
|
||||||
|
}
|
||||||
|
|
||||||
result := link.BuildResult{
|
result := link.BuildResult{
|
||||||
ID: request.ID, Repository: request.Repository, Path: request.Path,
|
ID: request.ID, Repository: request.Repository, Path: request.Path,
|
||||||
Ref: request.Ref, On: on,
|
Ref: request.Ref, On: on, Source: request.Source,
|
||||||
}
|
}
|
||||||
fmt.Fprintf(os.Stderr, "building %s", request.Repository)
|
what := "building " + request.Repository
|
||||||
if request.Path != "" {
|
if request.Path != "" {
|
||||||
fmt.Fprintf(os.Stderr, " at %s", request.Path)
|
what += " at " + request.Path
|
||||||
}
|
}
|
||||||
if request.Ref != "" {
|
if request.Ref != "" {
|
||||||
fmt.Fprintf(os.Stderr, " at %s", request.Ref)
|
what += " on " + request.Ref
|
||||||
}
|
}
|
||||||
fmt.Fprintln(os.Stderr)
|
say("build", what)
|
||||||
|
|
||||||
npmrc, err := packagesFrom()
|
npmrc, err := packagesFrom()
|
||||||
var built builder.Result
|
var built builder.Result
|
||||||
@@ -171,15 +185,13 @@ func answer(ctx context.Context, publisher builder.Publisher, on, workspace stri
|
|||||||
// after a clone that then fails at npm ci.
|
// after a clone that then fails at npm ci.
|
||||||
built, err = builder.Build(ctx, builder.Command, publisher,
|
built, err = builder.Build(ctx, builder.Command, publisher,
|
||||||
request.Repository, request.Path, request.Ref, workspace, request.Held, npmrc,
|
request.Repository, request.Path, request.Ref, workspace, request.Held, npmrc,
|
||||||
forgeFrom(), func(step, message string) {
|
forgeFrom(), say, request.Seats)
|
||||||
fmt.Fprintf(os.Stderr, " [%s] %s\n", step, message)
|
|
||||||
}, request.Seats)
|
|
||||||
}
|
}
|
||||||
if err != nil {
|
if err != nil {
|
||||||
// A failure is a result. A build that fails and says nothing is indistinguishable from a
|
// A failure is a result. A build that fails and says nothing is indistinguishable from a
|
||||||
// builder that is not running, and those want completely different responses.
|
// builder that is not running, and those want completely different responses.
|
||||||
result.Failed = err.Error()
|
result.Failed = err.Error()
|
||||||
fmt.Fprintf(os.Stderr, " failed: %v\n", err)
|
say("failed", err.Error())
|
||||||
} else {
|
} else {
|
||||||
manifest, marshalErr := json.Marshal(built.Manifest)
|
manifest, marshalErr := json.Marshal(built.Manifest)
|
||||||
if marshalErr != nil {
|
if marshalErr != nil {
|
||||||
@@ -196,7 +208,7 @@ func answer(ctx context.Context, publisher builder.Publisher, on, workspace stri
|
|||||||
for _, r := range built.Read {
|
for _, r := range built.Read {
|
||||||
result.Read = append(result.Read, link.ReadRepository{Repository: r.Repository, Ref: r.Ref})
|
result.Read = append(result.Read, link.ReadRepository{Repository: r.Repository, Ref: r.Ref})
|
||||||
}
|
}
|
||||||
fmt.Fprintf(os.Stderr, " built %s from %s\n", built.Manifest.Module, short(built.Commit))
|
say("built", built.Manifest.Module+" from "+short(built.Commit))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -91,6 +91,10 @@ func reportsReaching(t *testing.T, open *stores, reachable []link.Reach, held ..
|
|||||||
if _, err := (link.Enrolment{Inventory: open.inventory}).Heard(ctx, link.Report{
|
if _, err := (link.Enrolment{Inventory: open.inventory}).Heard(ctx, link.Report{
|
||||||
Node: "anchor", Applied: []string{"hello-web.x"}, Declared: digestOf(body),
|
Node: "anchor", Applied: []string{"hello-web.x"}, Declared: digestOf(body),
|
||||||
Firewall: "ufw", Held: held, Reachable: reachable,
|
Firewall: "ufw", Held: held, Reachable: reachable,
|
||||||
|
// A machine says which of its links face outside on every apply (novox/hq ADR 0140), and a
|
||||||
|
// filter is not sent to one that has not. The anchor reports one, as a real host does; this
|
||||||
|
// fixture lacked it from 2026-09-28 and nothing ran the test (issue 177).
|
||||||
|
Outward: []string{"eth0"},
|
||||||
}); err != nil {
|
}); err != nil {
|
||||||
t.Fatal(err)
|
t.Fatal(err)
|
||||||
}
|
}
|
||||||
|
|||||||
+127
-40
@@ -10,6 +10,8 @@ import (
|
|||||||
"strings"
|
"strings"
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
|
"github.com/nats-io/nats.go"
|
||||||
|
|
||||||
"github.com/novox/mesh-controller/internal/broker"
|
"github.com/novox/mesh-controller/internal/broker"
|
||||||
"github.com/novox/mesh-controller/internal/catalogue"
|
"github.com/novox/mesh-controller/internal/catalogue"
|
||||||
"github.com/novox/mesh-controller/internal/inventory"
|
"github.com/novox/mesh-controller/internal/inventory"
|
||||||
@@ -175,10 +177,14 @@ func buildFrom(result link.BuildResult) inventory.Build {
|
|||||||
func buildsCommand(ctx context.Context, args []string) error {
|
func buildsCommand(ctx context.Context, args []string) error {
|
||||||
set := flag.NewFlagSet("builds", flag.ContinueOnError)
|
set := flag.NewFlagSet("builds", flag.ContinueOnError)
|
||||||
limit := set.Int("n", 20, "how many to show")
|
limit := set.Int("n", 20, "how many to show")
|
||||||
|
logOf := set.String("log", "", "a build's id: print what the build machine said, line by line")
|
||||||
positionals, err := parseAround(set, args)
|
positionals, err := parseAround(set, args)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
if *logOf != "" {
|
||||||
|
return buildLog(ctx, *logOf)
|
||||||
|
}
|
||||||
module := ""
|
module := ""
|
||||||
if len(positionals) == 1 {
|
if len(positionals) == 1 {
|
||||||
module = positionals[0]
|
module = positionals[0]
|
||||||
@@ -219,8 +225,8 @@ func buildsCommand(ctx context.Context, args []string) error {
|
|||||||
if !b.Worked() {
|
if !b.Worked() {
|
||||||
outcome = "failed"
|
outcome = "failed"
|
||||||
}
|
}
|
||||||
fmt.Printf("%-18s %-14s %-10s %s\n",
|
fmt.Printf("%-18s %-14s %-10s %s %s\n",
|
||||||
what, outcome, b.On, b.At.Local().Format("2006-01-02 15:04"))
|
what, outcome, b.On, b.At.Local().Format("2006-01-02 15:04"), b.ID)
|
||||||
fmt.Printf(" %s", b.Repository)
|
fmt.Printf(" %s", b.Repository)
|
||||||
if b.Ref != "" {
|
if b.Ref != "" {
|
||||||
fmt.Printf(" at %s", b.Ref)
|
fmt.Printf(" at %s", b.Ref)
|
||||||
@@ -414,6 +420,8 @@ func buildOne(ctx context.Context, source buildSource, path, ref string, wait ti
|
|||||||
if source.Seat != "" {
|
if source.Seat != "" {
|
||||||
fmt.Printf(" (%s)", repository)
|
fmt.Printf(" (%s)", repository)
|
||||||
}
|
}
|
||||||
|
// The id is how a person follows this build while it runs: `builds --log <id>`.
|
||||||
|
fmt.Printf(" as %s", request.ID)
|
||||||
if path != "" {
|
if path != "" {
|
||||||
fmt.Printf(" at %s", path)
|
fmt.Printf(" at %s", path)
|
||||||
}
|
}
|
||||||
@@ -428,66 +436,91 @@ func buildOne(ctx context.Context, source buildSource, path, ref string, wait ti
|
|||||||
}
|
}
|
||||||
defer ask.Close()
|
defer ask.Close()
|
||||||
|
|
||||||
|
if wait == 0 {
|
||||||
|
// Asked and not waited for (novox/hq issue 176): the outcome is the role's event, and the
|
||||||
|
// controller takes it in — records the build, registers the module — whether or not anybody
|
||||||
|
// is still here. A tool call cannot hold a connection for the minutes a build takes; it
|
||||||
|
// follows the build by its id instead.
|
||||||
|
if err := ask.Ask(ctx, request); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
fmt.Printf("asked, not waited for: `builds --log %s` follows it as it runs, and `builds` "+
|
||||||
|
"shows what came of it; the module is registered when the outcome comes\n", request.ID)
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
result, err := ask.Submit(ctx, request, wait)
|
result, err := ask.Submit(ctx, request, wait)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
|
||||||
// Kept before it is judged. A failed build that leaves no trace is indistinguishable from one
|
|
||||||
// nobody asked for, and the difference is the whole of whether somebody should be looking at
|
|
||||||
// something.
|
|
||||||
open, err := openStores(ctx)
|
open, err := openStores(ctx)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
defer open.Close()
|
defer open.Close()
|
||||||
inv := open.inventory
|
manifest, kept, err := takeIn(ctx, open.inventory, result)
|
||||||
kept := buildFrom(result)
|
if err != nil {
|
||||||
if err := inv.RecordBuild(ctx, kept); err != nil {
|
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
|
||||||
if result.Failed != "" {
|
|
||||||
// The builder's own words. Wrapping them in something about the control plane would put
|
|
||||||
// two explanations between a person and a build log.
|
|
||||||
return fmt.Errorf("%s could not build %s:\n%s", result.On, result.Repository, result.Failed)
|
|
||||||
}
|
|
||||||
|
|
||||||
// Said as recorded: what each artifact is, not where this builder happened to push it.
|
// Said as recorded: what each artifact is, not where this builder happened to push it.
|
||||||
for _, made := range kept.Made {
|
for _, made := range kept.Made {
|
||||||
fmt.Printf(" %-12s %s %s\n", made.Name, made.Kind, made.Reference)
|
fmt.Printf(" %-12s %s %s\n", made.Name, made.Kind, made.Reference)
|
||||||
}
|
}
|
||||||
|
|
||||||
// Parsed with the same parser a hand-written manifest goes through. A second path would be a
|
|
||||||
// second thing to disagree about what a manifest is. The manifest as recorded, so the catalogue
|
|
||||||
// holds references by digest and path and every declaration composes the store's address in.
|
|
||||||
manifest, err := catalogue.ParseManifest(kept.Manifest)
|
|
||||||
if err != nil {
|
|
||||||
return fmt.Errorf("%s built %s and what came back is not a manifest: %w",
|
|
||||||
result.On, result.Repository, err)
|
|
||||||
}
|
|
||||||
|
|
||||||
// Recorded with where it came from, so "is this current?" is answerable without building it
|
|
||||||
// again (novox/hq ADR 0009). **For a source on a seat, as the path and the seat, never the URL
|
|
||||||
// just cloned** (ADR 0111): the URL is where the forge runs today, and recording it would put
|
|
||||||
// the forge's address back into every module built from it. The build log above keeps the URL,
|
|
||||||
// because that is what was cloned.
|
|
||||||
recorded := inventory.Source{
|
|
||||||
Repository: result.Repository, Path: result.Path, Ref: result.Ref,
|
|
||||||
BuiltFrom: result.Commit, Head: result.Commit,
|
|
||||||
}
|
|
||||||
if source.Seat != "" {
|
|
||||||
recorded.Repository, recorded.Seat = source.Repository, source.Seat
|
|
||||||
}
|
|
||||||
if err := inv.RegisterModule(ctx, manifest, recorded); err != nil {
|
|
||||||
return err
|
|
||||||
}
|
|
||||||
fmt.Printf("\n%s %s, built on %s from %s\n",
|
fmt.Printf("\n%s %s, built on %s from %s\n",
|
||||||
manifest.Module, manifest.Version, result.On, short(result.Commit))
|
manifest.Module, manifest.Version, result.On, short(result.Commit))
|
||||||
fmt.Printf(" run `assign <node> %s` to put it somewhere\n", manifest.Module)
|
fmt.Printf(" run `assign <node> %s` to put it somewhere\n", manifest.Module)
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// takeIn is what the mesh does with a build's outcome, whoever hears it: the waiting command and
|
||||||
|
// the daemon that follows the role's events both come here (novox/hq issue 176), so a build's
|
||||||
|
// result reaches the catalogue whether or not the asker was still listening.
|
||||||
|
//
|
||||||
|
// Kept before it is judged. A failed build that leaves no trace is indistinguishable from one
|
||||||
|
// nobody asked for, and the difference is the whole of whether somebody should be looking at
|
||||||
|
// something. Then parsed with the same parser a hand-written manifest goes through — a second path
|
||||||
|
// would be a second thing to disagree about what a manifest is — and registered with where it came
|
||||||
|
// from: **for a source on a seat, as the path and the seat, never the URL just cloned** (ADR 0111),
|
||||||
|
// which the request carried and the outcome echoes. A definition naming an installation is refused
|
||||||
|
// here, where it would enter the catalogue; the build stays recorded and the refusal says which.
|
||||||
|
//
|
||||||
|
// Idempotent: the same outcome taken in twice registers the same module twice, which is one row
|
||||||
|
// written with the same values.
|
||||||
|
func takeIn(ctx context.Context, inv *inventory.Inventory, result link.BuildResult) (
|
||||||
|
catalogue.Manifest, inventory.Build, error) {
|
||||||
|
kept := buildFrom(result)
|
||||||
|
if err := inv.RecordBuild(ctx, kept); err != nil {
|
||||||
|
return catalogue.Manifest{}, kept, err
|
||||||
|
}
|
||||||
|
if result.Failed != "" {
|
||||||
|
// The builder's own words. Wrapping them in something about the control plane would put
|
||||||
|
// two explanations between a person and a build log.
|
||||||
|
return catalogue.Manifest{}, kept, fmt.Errorf("%s could not build %s:\n%s",
|
||||||
|
result.On, result.Repository, result.Failed)
|
||||||
|
}
|
||||||
|
manifest, err := catalogue.ParseManifest(kept.Manifest)
|
||||||
|
if err != nil {
|
||||||
|
return catalogue.Manifest{}, kept, fmt.Errorf("%s built %s and what came back is not a manifest: %w",
|
||||||
|
result.On, result.Repository, err)
|
||||||
|
}
|
||||||
|
recorded := inventory.Source{
|
||||||
|
Repository: result.Repository, Path: result.Path, Ref: result.Ref,
|
||||||
|
BuiltFrom: result.Commit, Head: result.Commit,
|
||||||
|
}
|
||||||
|
if result.Source != nil && result.Source.Seat != "" {
|
||||||
|
recorded.Repository, recorded.Seat = result.Source.Repository, result.Source.Seat
|
||||||
|
}
|
||||||
|
if err := namesNoInstallation(manifest); err != nil {
|
||||||
|
return manifest, kept, fmt.Errorf("%s built %s (%s), and the mesh does not register it: %w",
|
||||||
|
result.On, result.Repository, short(result.Commit), err)
|
||||||
|
}
|
||||||
|
if err := inv.RegisterModule(ctx, manifest, recorded); err != nil {
|
||||||
|
return manifest, kept, err
|
||||||
|
}
|
||||||
|
return manifest, kept, nil
|
||||||
|
}
|
||||||
|
|
||||||
// buildAndShow builds and prints the manifest without recording anything.
|
// buildAndShow builds and prints the manifest without recording anything.
|
||||||
func buildAndShow(ctx context.Context, source buildSource, path, ref string, wait time.Duration) error {
|
func buildAndShow(ctx context.Context, source buildSource, path, ref string, wait time.Duration) error {
|
||||||
repository, err := cloneFrom(ctx, source)
|
repository, err := cloneFrom(ctx, source)
|
||||||
@@ -619,3 +652,57 @@ func askOver(_ *link.Server) (link.Builders, error) {
|
|||||||
}
|
}
|
||||||
return link.BuildsOverNATS(address)
|
return link.BuildsOverNATS(address)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// buildLog prints everything a build machine said about one build, read back from the bus.
|
||||||
|
//
|
||||||
|
// **From the stream, not from a record** (novox/hq ADR 0157). A build's lines are the role's own
|
||||||
|
// events under the build's id, retained with every other event; the mesh keeps no second copy. Read
|
||||||
|
// with a consumer of its own that is gone when this returns, so nothing accumulates in the server
|
||||||
|
// for the reading, and filtered by subject, so one build's lines are all that travel.
|
||||||
|
func buildLog(ctx context.Context, id string) error {
|
||||||
|
address, err := broker.BusAddress()
|
||||||
|
if err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
js, err := broker.Dial(address)
|
||||||
|
if err != nil {
|
||||||
|
return fmt.Errorf("cannot reach the bus to read a build's log: %w", err)
|
||||||
|
}
|
||||||
|
defer js.Close()
|
||||||
|
|
||||||
|
sub, err := js.Context().PullSubscribe(link.BuildLog(id), "",
|
||||||
|
nats.BindStream(broker.EventsStream), nats.DeliverAll(), nats.AckNone())
|
||||||
|
if err != nil {
|
||||||
|
return fmt.Errorf("cannot read %s from the bus: %w", link.BuildLog(id), err)
|
||||||
|
}
|
||||||
|
defer func() { _ = sub.Unsubscribe() }()
|
||||||
|
|
||||||
|
printed := 0
|
||||||
|
for {
|
||||||
|
batch, err := sub.Fetch(200, nats.MaxWait(2*time.Second))
|
||||||
|
if err != nil && !errors.Is(err, nats.ErrTimeout) && !errors.Is(err, context.DeadlineExceeded) {
|
||||||
|
return fmt.Errorf("reading a build's log: %w", err)
|
||||||
|
}
|
||||||
|
for _, msg := range batch {
|
||||||
|
var line link.BuildLine
|
||||||
|
if err := json.Unmarshal(msg.Data, &line); err != nil {
|
||||||
|
fmt.Printf(" ? %s\n", string(msg.Data))
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
at := line.At
|
||||||
|
if t, err := time.Parse(time.RFC3339Nano, line.At); err == nil {
|
||||||
|
at = t.Local().Format("15:04:05")
|
||||||
|
}
|
||||||
|
fmt.Printf("%s %4d [%s] %s\n", at, line.Seq, line.Step, line.Message)
|
||||||
|
printed++
|
||||||
|
}
|
||||||
|
if len(batch) < 200 {
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if printed == 0 {
|
||||||
|
fmt.Printf("nothing on the bus for build %s: no build by that id in the last week, or a build "+
|
||||||
|
"machine older than this that said nothing while building\n", id)
|
||||||
|
}
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|||||||
@@ -0,0 +1,65 @@
|
|||||||
|
package main
|
||||||
|
|
||||||
|
import (
|
||||||
|
"encoding/json"
|
||||||
|
"strings"
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"github.com/novox/mesh-controller/internal/link"
|
||||||
|
)
|
||||||
|
|
||||||
|
// A build's outcome is taken in the same way whoever hears it (novox/hq issue 176): recorded, and
|
||||||
|
// the module registered with its source as the seat and path when the request said so — never the
|
||||||
|
// URL. A definition naming an installation is recorded and not registered; a failure is recorded
|
||||||
|
// and said.
|
||||||
|
func TestABuildHeardIsRecordedAndRegistered(t *testing.T) {
|
||||||
|
open := aMesh(t)
|
||||||
|
ctx := t.Context()
|
||||||
|
manifest, _ := json.Marshal(map[string]any{"module": "shop", "version": "3"})
|
||||||
|
m, _, err := takeIn(ctx, open.inventory, link.BuildResult{
|
||||||
|
ID: "b-1", Repository: "http://forge.internal:20000/novox/shop.git", Path: "modules/shop",
|
||||||
|
Ref: "main", On: "anchor", Commit: "abcdef0123", Manifest: manifest,
|
||||||
|
Source: &link.SourceOnSeat{Seat: "git", Repository: "novox/shop"},
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if m.Module != "shop" {
|
||||||
|
t.Fatalf("registered %q", m.Module)
|
||||||
|
}
|
||||||
|
shelf, err := open.inventory.Catalogue(ctx)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if _, held := shelf["shop"]; !held {
|
||||||
|
t.Fatal("the module a heard build produced is not in the catalogue")
|
||||||
|
}
|
||||||
|
src, err := open.inventory.SourceOf(ctx, "shop")
|
||||||
|
if err != nil || src.Seat != "git" || src.Repository != "novox/shop" || src.BuiltFrom != "abcdef0123" {
|
||||||
|
t.Fatalf("the source is the seat and the path, never the URL: %+v %v", src, err)
|
||||||
|
}
|
||||||
|
builds, err := open.inventory.Builds(ctx, "shop", 5)
|
||||||
|
if err != nil || len(builds) != 1 || builds[0].ID != "b-1" {
|
||||||
|
t.Fatalf("the build is not recorded once: %v %v", builds, err)
|
||||||
|
}
|
||||||
|
|
||||||
|
named, _ := json.Marshal(map[string]any{"module": "idp", "version": "1", "resources": []any{
|
||||||
|
map[string]any{"id": "server", "type": "container", "image": "x@sha256:aa",
|
||||||
|
"env": map[string]any{"KC_HOSTNAME": "https://login.mesh-one.be"}}}})
|
||||||
|
_, _, err = takeIn(ctx, open.inventory, link.BuildResult{
|
||||||
|
ID: "b-2", Repository: "/r", On: "anchor", Commit: "0123456789", Manifest: named})
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "does not register it") {
|
||||||
|
t.Fatalf("a definition naming an installation was taken in: %v", err)
|
||||||
|
}
|
||||||
|
if shelf, _ := open.inventory.Catalogue(ctx); shelf["idp"].Module != "" {
|
||||||
|
t.Fatal("the refused module was registered anyway")
|
||||||
|
}
|
||||||
|
if builds, _ := open.inventory.Builds(ctx, "idp", 5); len(builds) != 1 {
|
||||||
|
t.Fatalf("the refused build was not recorded: %v", builds)
|
||||||
|
}
|
||||||
|
|
||||||
|
_, _, err = takeIn(ctx, open.inventory, link.BuildResult{ID: "b-3", Repository: "/r", On: "anchor", Failed: "no compiler"})
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "no compiler") {
|
||||||
|
t.Fatalf("a failure is said in the builder's words: %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -51,8 +51,8 @@ func moduleCheck(paths []string, out io.Writer) error {
|
|||||||
failed++
|
failed++
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
// A definition names no installation (novox/hq ADR 0112, ADR 0155): judged here and in the
|
// A definition names no installation (novox/hq ADR 0112, ADR 0155): judged here, in the
|
||||||
// catalogue-wide test, not yet at registration, while the declared exceptions shrink.
|
// catalogue-wide test, and at registration, which refuses in the same words.
|
||||||
if named := catalogue.InstallationProblems(m); len(named) > 0 {
|
if named := catalogue.InstallationProblems(m); len(named) > 0 {
|
||||||
for _, p := range named {
|
for _, p := range named {
|
||||||
fmt.Fprintf(out, "%s: %s\n", path, p)
|
fmt.Fprintf(out, "%s: %s\n", path, p)
|
||||||
|
|||||||
@@ -5,6 +5,8 @@ import (
|
|||||||
"os"
|
"os"
|
||||||
"path/filepath"
|
"path/filepath"
|
||||||
"strings"
|
"strings"
|
||||||
|
|
||||||
|
"github.com/novox/mesh-controller/internal/catalogue"
|
||||||
"testing"
|
"testing"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -67,3 +69,26 @@ func TestModuleCheckPassesTheCatalogue(t *testing.T) {
|
|||||||
t.Fatalf("the catalogue does not pass its own check: %v\n%s", err, out.String())
|
t.Fatalf("the catalogue does not pass its own check: %v\n%s", err, out.String())
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestRegistrationRefusesADefinitionNamingAnInstallation(t *testing.T) {
|
||||||
|
// novox/hq ADR 0155: the check moves to registration once the catalogue passes it. Both
|
||||||
|
// ways in — `module add` and a build's result — go through this, and a name declared on
|
||||||
|
// purpose passes with its reason.
|
||||||
|
named := catalogue.Manifest{Module: "idp", Resources: []map[string]any{
|
||||||
|
{"id": "server", "type": "container", "image": "x@sha256:aa",
|
||||||
|
"env": map[string]any{"KC_HOSTNAME": "https://login.mesh-one.be"}},
|
||||||
|
}}
|
||||||
|
err := namesNoInstallation(named)
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "login.mesh-one.be") ||
|
||||||
|
!strings.Contains(err.Error(), catalogue.NamesOnPurpose) {
|
||||||
|
t.Fatalf("a definition naming an installation is refused with the name and the way out; got %v", err)
|
||||||
|
}
|
||||||
|
meant := catalogue.Manifest{Module: "site", Resources: []map[string]any{
|
||||||
|
{"id": "server", "type": "container", "image": "registry.mesh-one.be/org/site@sha256:cc",
|
||||||
|
catalogue.NamesOnPurpose: map[string]any{
|
||||||
|
"registry.mesh-one.be": "built outside the mesh until its repository is a build source here"}},
|
||||||
|
}}
|
||||||
|
if err := namesNoInstallation(meant); err != nil {
|
||||||
|
t.Fatalf("a name declared on purpose passes; got %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -241,6 +241,21 @@ func parseAround(set *flag.FlagSet, args []string) ([]string, error) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Built is the daemon hearing a build's outcome on the bus — its own asking, an announcement's, or
|
||||||
|
// a tool's that did not wait (novox/hq issue 176) — and taking it in: recorded, and the module
|
||||||
|
// registered, the same as the waiting command does. Said either way, so the daemon's log tells what
|
||||||
|
// became of a build nobody was watching.
|
||||||
func (b builds) Built(ctx context.Context, result link.BuildResult) error {
|
func (b builds) Built(ctx context.Context, result link.BuildResult) error {
|
||||||
return b.inv.RecordBuild(ctx, buildFrom(result))
|
manifest, _, err := takeIn(ctx, b.inv, result)
|
||||||
|
switch {
|
||||||
|
case err != nil && result.Failed != "":
|
||||||
|
fmt.Printf("%s: %v\n", result.ID, err)
|
||||||
|
return nil
|
||||||
|
case err != nil:
|
||||||
|
fmt.Printf("%s: heard and recorded, and not registered: %v\n", result.ID, err)
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
fmt.Printf("%s: %s %s registered, built on %s from %s\n",
|
||||||
|
result.ID, manifest.Module, manifest.Version, result.On, short(result.Commit))
|
||||||
|
return nil
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -113,6 +113,9 @@ func moduleCommand(ctx context.Context, args []string) error {
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
if err := namesNoInstallation(m); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
if err := inv.RegisterModule(ctx, m, from); err != nil {
|
if err := inv.RegisterModule(ctx, m, from); err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
@@ -662,3 +665,18 @@ func whereItComesFrom(repository, ref, commit, path string, self bool) (inventor
|
|||||||
}
|
}
|
||||||
return from, nil
|
return from, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// namesNoInstallation is the mesh refusing a definition that names an installation, at the moment
|
||||||
|
// it would enter the catalogue (novox/hq ADR 0112, ADR 0155). `module check` says the same thing
|
||||||
|
// earlier, where the author is; this is the last moment the mesh can still say no, and a
|
||||||
|
// definition that got past the check — written elsewhere, or checked by nobody — is refused here
|
||||||
|
// in the same words. A name meant on purpose is declared with its reason and passes.
|
||||||
|
func namesNoInstallation(m catalogue.Manifest) error {
|
||||||
|
named := catalogue.InstallationProblems(m)
|
||||||
|
if len(named) == 0 {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
return fmt.Errorf("%s names an installation, and a definition names none — declare a name meant "+
|
||||||
|
"on purpose under %s with its reason, or take it out:\n - %s",
|
||||||
|
m.Module, catalogue.NamesOnPurpose, strings.Join(named, "\n - "))
|
||||||
|
}
|
||||||
|
|||||||
+13
-31
@@ -731,9 +731,13 @@ func routeNamesInTheMesh(ctx context.Context, open *stores) (map[string]string,
|
|||||||
return nil, fmt.Errorf("which machines the mesh has cannot be read: %w", err)
|
return nil, fmt.Errorf("which machines the mesh has cannot be read: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
out := map[string]string{}
|
// Every machine's resolution first, then the names across them at once: which node serves a
|
||||||
|
// name is a question about the graph — the consumer on one machine, the provider on another —
|
||||||
|
// and answered wrongly by looking at one contribution at a time (novox/hq issue 178).
|
||||||
|
plans := map[string]catalogue.Resolution{}
|
||||||
|
settings := map[string]catalogue.SettingsBy{}
|
||||||
for _, n := range nodes {
|
for _, n := range nodes {
|
||||||
plan, settings, err := planFor(ctx, open, n.Name)
|
plan, layers, err := planFor(ctx, open, n.Name)
|
||||||
switch {
|
switch {
|
||||||
case unresolvable(err):
|
case unresolvable(err):
|
||||||
// Their set does not compose, so they serve no names. Passed over, so one machine's
|
// Their set does not compose, so they serve no names. Passed over, so one machine's
|
||||||
@@ -745,38 +749,16 @@ func routeNamesInTheMesh(ctx context.Context, open *stores) (map[string]string,
|
|||||||
// operator having withdrawn them (novox/hq 04-ISSUES/152).
|
// operator having withdrawn them (novox/hq 04-ISSUES/152).
|
||||||
return nil, fmt.Errorf("the names %s serves cannot be read: %w", n.Name, err)
|
return nil, fmt.Errorf("the names %s serves cannot be read: %w", n.Name, err)
|
||||||
}
|
}
|
||||||
for _, m := range plan.Modules {
|
plans[n.Name], settings[n.Name] = plan, layers
|
||||||
for to := range m.Contributes {
|
}
|
||||||
values, asks, err := plan.ContributionsFrom(to, m.Module, settings)
|
served, err := catalogue.NamesServed(plans, settings)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
if !asks {
|
out := map[string]string{}
|
||||||
continue
|
for name, node := range served {
|
||||||
}
|
if at := address[node]; at != "" {
|
||||||
// A routed name, and only that: a contribution the mesh composed a name for from a
|
out[name] = at
|
||||||
// label it was given. A grant that happens to carry a `name` of its own — a database
|
|
||||||
// name — carries no label and is left alone.
|
|
||||||
if _, labelled := values["label"]; !labelled {
|
|
||||||
continue
|
|
||||||
}
|
|
||||||
name, _ := values["name"].(string)
|
|
||||||
if name == "" {
|
|
||||||
continue
|
|
||||||
}
|
|
||||||
// The node that serves it: whoever answers this consumer's route requirement, or
|
|
||||||
// this same node when the proxy is beside the consumer.
|
|
||||||
serving := n.Name
|
|
||||||
for _, need := range plan.Needs {
|
|
||||||
if need.Name == to && need.For == m.Module {
|
|
||||||
serving = need.From
|
|
||||||
break
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if at := address[serving]; at != "" {
|
|
||||||
out[strings.ToLower(name)] = at
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return out, nil
|
return out, nil
|
||||||
|
|||||||
@@ -62,6 +62,9 @@ func argvFor(verb string, args map[string]any) ([]string, error) {
|
|||||||
case "seats":
|
case "seats":
|
||||||
return []string{"seats", "--json"}, nil
|
return []string{"seats", "--json"}, nil
|
||||||
case "builds":
|
case "builds":
|
||||||
|
if id := str("log"); id != "" {
|
||||||
|
return []string{"builds", "--log", id}, nil
|
||||||
|
}
|
||||||
if m := str("module"); m != "" {
|
if m := str("module"); m != "" {
|
||||||
return []string{"builds", m}, nil
|
return []string{"builds", m}, nil
|
||||||
}
|
}
|
||||||
@@ -88,7 +91,13 @@ func argvFor(verb string, args map[string]any) ([]string, error) {
|
|||||||
if err := need("repository"); err != nil {
|
if err := need("repository"); err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
// Not waited for: a tool call cannot hold a connection for the minutes a build takes; the
|
||||||
|
// daemon takes the outcome in when it comes and the id follows the build (issue 176). A
|
||||||
|
// repository given without a scheme is a path on the forge holding the git seat.
|
||||||
argv := []string{"build", str("repository"), "--wait", "0"}
|
argv := []string{"build", str("repository"), "--wait", "0"}
|
||||||
|
if !strings.Contains(str("repository"), "://") && !strings.HasPrefix(str("repository"), "git@") {
|
||||||
|
argv = append(argv, "--self")
|
||||||
|
}
|
||||||
if p := str("path"); p != "" {
|
if p := str("path"); p != "" {
|
||||||
argv = append(argv, "--path", p)
|
argv = append(argv, "--path", p)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -30,6 +30,32 @@ func TestEveryDeclaredVerbHasACommandLine(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// `builds` given a build's id reads that build's log from the bus rather than listing builds
|
||||||
|
// (novox/hq ADR 0157).
|
||||||
|
func TestBuildsWithAnIdReadsThatBuildsLog(t *testing.T) {
|
||||||
|
argv, err := argvFor("builds", map[string]any{"log": "build-17"})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if strings.Join(argv, " ") != "builds --log build-17" {
|
||||||
|
t.Fatalf("builds with a log id became %q", strings.Join(argv, " "))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// The build tool takes a repository as a URL or as its path on the forge holding the git seat, and
|
||||||
|
// says which it was given, so the command reads the path as a seat source rather than handing it to
|
||||||
|
// git as written (novox/hq issue 176). And it never waits: the id follows the build.
|
||||||
|
func TestTheBuildToolTellsAForgePathFromAURL(t *testing.T) {
|
||||||
|
argv, _ := argvFor("build", map[string]any{"repository": "novox/mesh-catalog", "path": "modules/x"})
|
||||||
|
if line := strings.Join(argv, " "); !strings.Contains(line, "--self") || !strings.Contains(line, "--wait 0") {
|
||||||
|
t.Fatalf("a forge path is a seat source, not waited for; got %q", line)
|
||||||
|
}
|
||||||
|
argv, _ = argvFor("build", map[string]any{"repository": "https://example.tld/o/r.git"})
|
||||||
|
if line := strings.Join(argv, " "); strings.Contains(line, "--self") {
|
||||||
|
t.Fatalf("a URL is cloned as given; got %q", line)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// A required argument missing is refused in the verb's own words, before anything runs.
|
// A required argument missing is refused in the verb's own words, before anything runs.
|
||||||
func TestAVerbMissingWhatItNeedsIsRefused(t *testing.T) {
|
func TestAVerbMissingWhatItNeedsIsRefused(t *testing.T) {
|
||||||
if _, err := argvFor("node", map[string]any{}); err == nil || !strings.Contains(err.Error(), `node needs "node"`) {
|
if _, err := argvFor("node", map[string]any{}); err == nil || !strings.Contains(err.Error(), `node needs "node"`) {
|
||||||
@@ -40,14 +66,15 @@ func TestAVerbMissingWhatItNeedsIsRefused(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// A push and a build are sent, not waited for: the asker reads status for what happened.
|
// A push and a build are sent, not waited for: the asker reads status, or the build's log by its
|
||||||
|
// id, for what happened. A repository given as a forge path is said to be one (issue 176).
|
||||||
func TestActsDoNotBlockTheCall(t *testing.T) {
|
func TestActsDoNotBlockTheCall(t *testing.T) {
|
||||||
argv, _ := argvFor("push", map[string]any{"node": "one"})
|
argv, _ := argvFor("push", map[string]any{"node": "one"})
|
||||||
if strings.Join(argv, " ") != "push one --wait 0" {
|
if strings.Join(argv, " ") != "push one --wait 0" {
|
||||||
t.Fatalf("push waits: %v", argv)
|
t.Fatalf("push waits: %v", argv)
|
||||||
}
|
}
|
||||||
argv, _ = argvFor("build", map[string]any{"repository": "novox/x", "path": "modules/x"})
|
argv, _ = argvFor("build", map[string]any{"repository": "novox/x", "path": "modules/x"})
|
||||||
if strings.Join(argv, " ") != "build novox/x --wait 0 --path modules/x" {
|
if strings.Join(argv, " ") != "build novox/x --wait 0 --self --path modules/x" {
|
||||||
t.Fatalf("build: %v", argv)
|
t.Fatalf("build: %v", argv)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -47,7 +47,12 @@ func composed(t *testing.T, open *stores, node string) sendable {
|
|||||||
// aMesh's laptop with the private network taken off it, so nothing in the declaration is random:
|
// aMesh's laptop with the private network taken off it, so nothing in the declaration is random:
|
||||||
// what changes this string is a change to what a converged machine is sent, which is the thing an
|
// what changes this string is a change to what a converged machine is sent, which is the thing an
|
||||||
// older host would refuse.
|
// older host would refuse.
|
||||||
const convergedBefore = `{"declaration":1,"resources":[{"content":"hello","id":"hello-web.page","path":"/var/lib/hello-web/index.html","type":"file"},{"hosts":["anchor.internal:10.77.0.1"],"id":"hello-web.server","image":"registry.example/hello@sha256:aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa","name":"hello-web","type":"container"},{"id":"hello-web.served","path":"/var/lib/hello-web","type":"directory"}]}`
|
//
|
||||||
|
// Re-captured 2026-10-01 (novox/hq issue 177): c978aa7 took `hosts` off every container — a
|
||||||
|
// machine's own resolver knows the mesh's names now — and left this string carrying it, so the
|
||||||
|
// guard failed for a day and nothing ran it. A field an older host never sees is the one change
|
||||||
|
// this guard permits; a field it would refuse is the one it exists to catch.
|
||||||
|
const convergedBefore = `{"declaration":1,"resources":[{"content":"hello","id":"hello-web.page","path":"/var/lib/hello-web/index.html","type":"file"},{"id":"hello-web.server","image":"registry.example/hello@sha256:aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa","name":"hello-web","type":"container"},{"id":"hello-web.served","path":"/var/lib/hello-web","type":"directory"}]}`
|
||||||
|
|
||||||
func TestAConvergedDeclarationIsByteForByteWhatItWas(t *testing.T) {
|
func TestAConvergedDeclarationIsByteForByteWhatItWas(t *testing.T) {
|
||||||
open := aMesh(t)
|
open := aMesh(t)
|
||||||
|
|||||||
@@ -40,6 +40,13 @@ type Consumer struct {
|
|||||||
AckWaitSeconds int
|
AckWaitSeconds int
|
||||||
// MaxDeliver before the message is dead-lettered; zero for the mesh's default.
|
// MaxDeliver before the message is dead-lettered; zero for the mesh's default.
|
||||||
MaxDeliver int
|
MaxDeliver int
|
||||||
|
// MaxAckPending is how many deliveries the server lets stand unacknowledged at once; zero for
|
||||||
|
// the server's default, which is many. **One, for a consumer handled one at a time**
|
||||||
|
// (novox/hq issue 175): a handler that builds for minutes keeps its own message alive with a
|
||||||
|
// heartbeat, but everything handed over behind it times out unacknowledged and comes back —
|
||||||
|
// and a merge that came back rebuilt what it had just built, five times over on 2026-09-30.
|
||||||
|
// With one outstanding, the server holds the rest, and the heartbeat is keeping the message.
|
||||||
|
MaxAckPending int
|
||||||
Why string
|
Why string
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -179,6 +179,7 @@ func (j *JetStream) EnsureConsumer(c Consumer) error {
|
|||||||
AckPolicy: nats.AckExplicitPolicy,
|
AckPolicy: nats.AckExplicitPolicy,
|
||||||
AckWait: time.Duration(c.AckWaitSeconds) * time.Second,
|
AckWait: time.Duration(c.AckWaitSeconds) * time.Second,
|
||||||
MaxDeliver: c.MaxDeliver,
|
MaxDeliver: c.MaxDeliver,
|
||||||
|
MaxAckPending: c.MaxAckPending,
|
||||||
DeliverGroup: c.Queue,
|
DeliverGroup: c.Queue,
|
||||||
DeliverSubject: "",
|
DeliverSubject: "",
|
||||||
Description: c.Why,
|
Description: c.Why,
|
||||||
|
|||||||
@@ -71,6 +71,18 @@ func TestHoldingASeatIsTheMirrorOfUsingIt(t *testing.T) {
|
|||||||
hasNot(t, perms.Publish, "mesh.seat.telegram-sender.accept.send")
|
hasNot(t, perms.Publish, "mesh.seat.telegram-sender.accept.send")
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// A build machine may say everything about a build as it happens (novox/hq ADR 0157): that it
|
||||||
|
// started, and every line under the build's own id — the seat's `log.*` becomes a publish over
|
||||||
|
// one token, so a reader follows one build by subject and the holder can name no other subject.
|
||||||
|
func TestTheBuildMachineMaySayWhatItDoesUnderTheBuildsId(t *testing.T) {
|
||||||
|
seat := Seat{Name: "mesh-build-machine", Accepts: []string{"build"}, Emits: []string{"started", "built", "log.*"}}
|
||||||
|
perms, _ := PermissionsFor(Principal{Kind: KindModule, Node: "anchor", Module: "builder",
|
||||||
|
Holds: []Seat{seat}, PasswordHash: "x"})
|
||||||
|
has(t, perms.Publish, "mesh.seat.mesh-build-machine.event.started")
|
||||||
|
has(t, perms.Publish, "mesh.seat.mesh-build-machine.event.log.*")
|
||||||
|
hasNot(t, perms.Publish, "mesh.seat.mesh-build-machine.event.>")
|
||||||
|
}
|
||||||
|
|
||||||
// Without an ack permission a durable consumer never really consumes: every message it receives is
|
// Without an ack permission a durable consumer never really consumes: every message it receives is
|
||||||
// redelivered forever, refused by the permission list it already has (design 25 §4).
|
// redelivered forever, refused by the permission list it already has (design 25 §4).
|
||||||
func TestAModuleMayAckItsOwnDeliveriesAndNoOthers(t *testing.T) {
|
func TestAModuleMayAckItsOwnDeliveriesAndNoOthers(t *testing.T) {
|
||||||
|
|||||||
@@ -79,7 +79,7 @@ func MeshStreams() []Stream {
|
|||||||
"n-1 by construction (issue 107)",
|
"n-1 by construction (issue 107)",
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
Name: "EVENTS",
|
Name: EventsStream,
|
||||||
// A seat's own events ride here too: they are 1:many like any event, and the
|
// A seat's own events ride here too: they are 1:many like any event, and the
|
||||||
// `event` token keeps them clear of both the seat's work queue (`accept`) and its
|
// `event` token keeps them clear of both the seat's work queue (`accept`) and its
|
||||||
// tools (`tool`), which must not be persisted.
|
// tools (`tool`), which must not be persisted.
|
||||||
@@ -220,6 +220,9 @@ func seatEventSubject(seat, verb string) string {
|
|||||||
return "mesh.seat." + seat + ".event." + verb
|
return "mesh.seat." + seat + ".event." + verb
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// EventsStream holds every module's and every role's events, a build's log among them.
|
||||||
|
const EventsStream = "EVENTS"
|
||||||
|
|
||||||
// MeshConsumers is what the controller consumes, in the order a person reads it.
|
// MeshConsumers is what the controller consumes, in the order a person reads it.
|
||||||
//
|
//
|
||||||
// **Unlimited redelivery on CONTROL, deliberately.** The store window's bound is the controller's,
|
// **Unlimited redelivery on CONTROL, deliberately.** The store window's bound is the controller's,
|
||||||
@@ -244,8 +247,13 @@ func MeshConsumers() []Consumer {
|
|||||||
Push: true,
|
Push: true,
|
||||||
AckWaitSeconds: 30,
|
AckWaitSeconds: 30,
|
||||||
MaxDeliver: 5,
|
MaxDeliver: 5,
|
||||||
Why: "the two events the mesh's own controller reacts to; after max-deliver it " +
|
// One at a time (novox/hq issue 175): acting on a merge builds for minutes, and an
|
||||||
"dead-letters, because an announcement it cannot act on will not become actionable",
|
// announcement handed over behind it must wait on the server, not time out on the
|
||||||
|
// client and come back to be acted on again.
|
||||||
|
MaxAckPending: 1,
|
||||||
|
Why: "the two events the mesh's own controller reacts to, one at a time; after " +
|
||||||
|
"max-deliver it dead-letters, because an announcement it cannot act on will not " +
|
||||||
|
"become actionable",
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -260,3 +260,14 @@ func TestNoTwoConsumersDeliverOntoTheSameSubject(t *testing.T) {
|
|||||||
seen[subject] = c.Name + " on " + c.Stream
|
seen[subject] = c.Name + " on " + c.Stream
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// The controller's events consumer is handed one announcement at a time (novox/hq issue 175): a
|
||||||
|
// merge's handler builds for minutes, and what is queued behind it must wait on the server rather
|
||||||
|
// than time out on the client and be acted on twice.
|
||||||
|
func TestTheControllerTakesOneAnnouncementAtATime(t *testing.T) {
|
||||||
|
for _, c := range MeshConsumers() {
|
||||||
|
if c.Stream == "EVENTS" && c.Name == ControllerName && c.MaxAckPending != 1 {
|
||||||
|
t.Fatalf("the events consumer may have %d outstanding; one announcement at a time", c.MaxAckPending)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -719,6 +719,21 @@ func short(commit string) string {
|
|||||||
return commit
|
return commit
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Said is where the lines Command speaks go, beside the build's own Log: what runs, how long it
|
||||||
|
// took, and that it failed. Nil prints them to stderr, as a build machine with nobody listening
|
||||||
|
// should. The machine sets it per build so every line reaches the bus too (novox/hq ADR 0157) —
|
||||||
|
// the step is "run", and the message is the line as it has always been printed.
|
||||||
|
var Said Log
|
||||||
|
|
||||||
|
func tell(step, format string, args ...any) {
|
||||||
|
message := fmt.Sprintf(format, args...)
|
||||||
|
if Said == nil {
|
||||||
|
fmt.Fprintf(os.Stderr, " %s\n", message)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
Said(step, message)
|
||||||
|
}
|
||||||
|
|
||||||
// Command is a Runner that actually runs things.
|
// Command is a Runner that actually runs things.
|
||||||
func Command(ctx context.Context, dir, name string, args ...string) (string, error) {
|
func Command(ctx context.Context, dir, name string, args ...string) (string, error) {
|
||||||
// **Every command is echoed before it runs**, with where. On a build that hangs, the last line
|
// **Every command is echoed before it runs**, with where. On a build that hangs, the last line
|
||||||
@@ -726,16 +741,23 @@ func Command(ctx context.Context, dir, name string, args ...string) (string, err
|
|||||||
// nothing" and "git clone is waiting on a network that will not answer". Silent on success is
|
// nothing" and "git clone is waiting on a network that will not answer". Silent on success is
|
||||||
// what made an empty workspace unreadable.
|
// what made an empty workspace unreadable.
|
||||||
started := timeNow()
|
started := timeNow()
|
||||||
fmt.Fprintf(os.Stderr, " $ (%s) %s %s\n", short(filepath.Base(dir)), name, strings.Join(args, " "))
|
tell("run", "$ (%s) %s %s", short(filepath.Base(dir)), name, strings.Join(args, " "))
|
||||||
cmd := exec.CommandContext(ctx, name, args...)
|
cmd := exec.CommandContext(ctx, name, args...)
|
||||||
cmd.Dir = dir
|
cmd.Dir = dir
|
||||||
out, err := cmd.CombinedOutput()
|
out, err := cmd.CombinedOutput()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
fmt.Fprintf(os.Stderr, " ! %s %s failed after %s\n", name, args[0], since(started))
|
tell("run", "! %s %s failed after %s", name, args[0], since(started))
|
||||||
|
// The command's own output is part of what a reader needs — the compiler's error, the
|
||||||
|
// clone's refusal — and a line per output line keeps it readable on the bus.
|
||||||
|
for _, line := range strings.Split(strings.TrimSpace(string(out)), "\n") {
|
||||||
|
if line != "" {
|
||||||
|
tell("output", "%s", line)
|
||||||
|
}
|
||||||
|
}
|
||||||
return string(out), fmt.Errorf("%s %s: %w\n%s",
|
return string(out), fmt.Errorf("%s %s: %w\n%s",
|
||||||
name, strings.Join(args, " "), err, strings.TrimSpace(string(out)))
|
name, strings.Join(args, " "), err, strings.TrimSpace(string(out)))
|
||||||
}
|
}
|
||||||
fmt.Fprintf(os.Stderr, " ✓ %s %s (%s)\n", name, firstArg(args), since(started))
|
tell("run", "✓ %s %s (%s)", name, firstArg(args), since(started))
|
||||||
return string(out), nil
|
return string(out), nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -11,11 +11,24 @@ import (
|
|||||||
//
|
//
|
||||||
// Not a fixture: the point is whether the manifests as written are accepted by the control plane that
|
// Not a fixture: the point is whether the manifests as written are accepted by the control plane that
|
||||||
// will read them, and a copy of one manifest proves nothing about the other seventy-one.
|
// will read them, and a copy of one manifest proves nothing about the other seventy-one.
|
||||||
func TestEveryCatalogueManifestParses(t *testing.T) {
|
// catalogueRoot is the catalogue these checks run over: MESH_CATALOGUE when set, else the checkout
|
||||||
root := os.Getenv("MESH_CATALOGUE")
|
// beside this one, the way the main layout has it. A check that only ran when somebody remembered a
|
||||||
if root == "" {
|
// variable was a check nobody ran (novox/hq issue 134, 2026-09-30); it skips only when there is no
|
||||||
t.Skip("set MESH_CATALOGUE to a catalogue checkout to run this")
|
// catalogue to be found at all.
|
||||||
|
func catalogueRoot(t *testing.T) string {
|
||||||
|
t.Helper()
|
||||||
|
if root := os.Getenv("MESH_CATALOGUE"); root != "" {
|
||||||
|
return root
|
||||||
}
|
}
|
||||||
|
sibling := filepath.Join("..", "..", "..", "mesh-catalog")
|
||||||
|
if _, err := os.Stat(filepath.Join(sibling, "modules")); err != nil {
|
||||||
|
t.Skip("no catalogue beside this checkout and MESH_CATALOGUE unset")
|
||||||
|
}
|
||||||
|
return sibling
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestEveryCatalogueManifestParses(t *testing.T) {
|
||||||
|
root := catalogueRoot(t)
|
||||||
found, err := filepath.Glob(filepath.Join(root, "modules", "*", "module.json"))
|
found, err := filepath.Glob(filepath.Join(root, "modules", "*", "module.json"))
|
||||||
if err != nil || len(found) == 0 {
|
if err != nil || len(found) == 0 {
|
||||||
t.Fatalf("no manifests under %s: %v", root, err)
|
t.Fatalf("no manifests under %s: %v", root, err)
|
||||||
@@ -51,10 +64,7 @@ func TestEveryCatalogueManifestParses(t *testing.T) {
|
|||||||
// definition names a domain or a public address the mesh acts on, and every value that must for now
|
// definition names a domain or a public address the mesh acts on, and every value that must for now
|
||||||
// carries its reason (novox/hq ADR 0155, issue 134). The list it prints is the one that shrinks.
|
// carries its reason (novox/hq ADR 0155, issue 134). The list it prints is the one that shrinks.
|
||||||
func TestNoCatalogueManifestNamesAnInstallation(t *testing.T) {
|
func TestNoCatalogueManifestNamesAnInstallation(t *testing.T) {
|
||||||
root := os.Getenv("MESH_CATALOGUE")
|
root := catalogueRoot(t)
|
||||||
if root == "" {
|
|
||||||
t.Skip("set MESH_CATALOGUE to a catalogue checkout to run this")
|
|
||||||
}
|
|
||||||
found, err := filepath.Glob(filepath.Join(root, "modules", "*", "module.json"))
|
found, err := filepath.Glob(filepath.Join(root, "modules", "*", "module.json"))
|
||||||
if err != nil || len(found) == 0 {
|
if err != nil || len(found) == 0 {
|
||||||
t.Fatalf("no manifests under %s: %v", root, err)
|
t.Fatalf("no manifests under %s: %v", root, err)
|
||||||
|
|||||||
@@ -197,6 +197,27 @@ func TestARouteCanBeSetPerMesh(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestASettingReachesAContributionOnlyWhereItDeclaresTheKey(t *testing.T) {
|
||||||
|
// novox/hq 04-ISSUES/173: the mail module's site name, set so its environment file could read
|
||||||
|
// it, arrived in every route it contributed. A setting overrides a key the contribution
|
||||||
|
// declares and adds none — the provider reads the contribution as a contract.
|
||||||
|
got, _ := Resolve(shelf(proxy(), published("board", "board", 8080)), []string{"board"}, workstation(), World{})
|
||||||
|
|
||||||
|
out, err := got.Declaration(Rendering{Settings: SettingsBy{
|
||||||
|
"board": {{From: "the mesh", Values: map[string]any{"host": "dashboard", "sitename": "Board"}}},
|
||||||
|
}})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
given := received(t, out)
|
||||||
|
if given[0].Values["host"] != "dashboard" {
|
||||||
|
t.Fatalf("the setting did not override the route's host: %v", given[0].Values)
|
||||||
|
}
|
||||||
|
if _, leaked := given[0].Values["sitename"]; leaked {
|
||||||
|
t.Fatalf("a setting the route never declared reached the proxy: %v", given[0].Values)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestReceivingWhatYouDoNotProvideIsRefused(t *testing.T) {
|
func TestReceivingWhatYouDoNotProvideIsRefused(t *testing.T) {
|
||||||
// It would create a file nobody ever writes to, on a machine where nothing asked for it.
|
// It would create a file nobody ever writes to, on a machine where nothing asked for it.
|
||||||
_, err := ParseManifest([]byte(`{"module":"traefik","version":"1",
|
_, err := ParseManifest([]byte(`{"module":"traefik","version":"1",
|
||||||
|
|||||||
@@ -473,9 +473,15 @@ func (r Resolution) compose(with Rendering, owner map[string]string) ([]map[stri
|
|||||||
// find each present — refusing clearly if the operator has not provided it — before it
|
// find each present — refusing clearly if the operator has not provided it — before it
|
||||||
// starts anything that depends on it. The mesh creates, chowns and reconciles none of it;
|
// starts anything that depends on it. The mesh creates, chowns and reconciles none of it;
|
||||||
// an `access` resource says only *this path must exist, and this module reaches it*.
|
// an `access` resource says only *this path must exist, and this module reaches it*.
|
||||||
for _, a := range m.Accesses {
|
// Where each is on THIS machine is the assignment's (novox/hq issue 153): placed by id
|
||||||
|
// where the operator said, the definition's default otherwise, refused where neither.
|
||||||
|
accesses, accessPaths, err := accessesFor(m, with.Settings[m.Module])
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
for _, a := range accesses {
|
||||||
first = append(first, map[string]any{
|
first = append(first, map[string]any{
|
||||||
"id": AccessID(a.Path), "type": "access", "path": a.Path, "mode": a.At(),
|
"id": AccessID(a.Path), "type": "access", "path": a.Path, "mode": a.Mode,
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
for _, to := range m.SecretRequirements() {
|
for _, to := range m.SecretRequirements() {
|
||||||
@@ -670,6 +676,12 @@ func (r Resolution) compose(with Rendering, owner map[string]string) ([]map[stri
|
|||||||
}
|
}
|
||||||
// And where this node places the directories the module declared without a path
|
// And where this node places the directories the module declared without a path
|
||||||
// (novox/hq ADR 0112) — resolved once per module, named by ${dir:…} from any resource.
|
// (novox/hq ADR 0112) — resolved once per module, named by ${dir:…} from any resource.
|
||||||
|
// — and, on an adopted machine, where the assignment says they already are, with the
|
||||||
|
// owner the data already has (novox/hq issue 153). Malformed placements are refused here.
|
||||||
|
placed, err := Places(m, with.Settings[m.Module])
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
dirs := dirsFor(m, with)
|
dirs := dirsFor(m, with)
|
||||||
// And the machine underneath, which no binding of its own can tell it.
|
// And the machine underneath, which no binding of its own can tell it.
|
||||||
thisMachine := machineFacts(r, with.Names, with.MeshRange)
|
thisMachine := machineFacts(r, with.Names, with.MeshRange)
|
||||||
@@ -710,6 +722,12 @@ func (r Resolution) compose(with Rendering, owner map[string]string) ([]map[stri
|
|||||||
if err := dirInto(copied, dirs, m.Module); err != nil {
|
if err := dirInto(copied, dirs, m.Module); err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
// The operator's data the same way: ${access:…} becomes where this node keeps it,
|
||||||
|
// and a placed directory takes the owner the assignment said (issue 153).
|
||||||
|
if err := accessInto(copied, accessPaths, m.Module); err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
ownerInto(copied, placed)
|
||||||
// **After settings, and that is the whole reason it is here.** A module's file
|
// **After settings, and that is the whole reason it is here.** A module's file
|
||||||
// content is where a setting lands, so a placeholder may only exist once the setting
|
// content is where a setting lands, so a placeholder may only exist once the setting
|
||||||
// has been put in — filling secrets first would look at content that is not yet what
|
// has been put in — filling secrets first would look at content that is not yet what
|
||||||
@@ -1161,7 +1179,10 @@ func (r Resolution) contributions(settings SettingsBy, grants []Grant,
|
|||||||
// module wrote another into its configuration.
|
// module wrote another into its configuration.
|
||||||
func (r Resolution) composed(m Manifest, to string, raw map[string]any, layers []Layer, what string) (
|
func (r Resolution) composed(m Manifest, to string, raw map[string]any, layers []Layer, what string) (
|
||||||
map[string]any, error) {
|
map[string]any, error) {
|
||||||
values, err := settle(raw, layers, nil, what)
|
// Overridden, not merged: a setting changes a key the contribution declares and adds none.
|
||||||
|
// The provider reads the contribution as a contract, and a setting made for one of this
|
||||||
|
// module's files is no part of it (novox/hq 04-ISSUES/173).
|
||||||
|
values, err := overridden(raw, layers, what)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, fmt.Errorf("%s: %w", what, err)
|
return nil, fmt.Errorf("%s: %w", what, err)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -20,6 +20,12 @@ import (
|
|||||||
// declared with the path as the exception it is, and everything else in the module names it by
|
// declared with the path as the exception it is, and everything else in the module names it by
|
||||||
// id — so moving it later is one line, not a search.
|
// id — so moving it later is one line, not a search.
|
||||||
//
|
//
|
||||||
|
// **The mesh's own files for a module are placed too** (novox/hq issue 174). What the mesh writes
|
||||||
|
// *for* a module — its sealed bus credential, its merged configuration, its bindings — is the
|
||||||
|
// mesh's plumbing, not the module's data, and sits under `<root>/mesh/<module>`. A directory
|
||||||
|
// saying `"place": "mesh"` is that place; the definition names the files beneath it by
|
||||||
|
// `${dir:<id>}` and states no path.
|
||||||
|
//
|
||||||
// **Resolved here, not on the machine.** The host receives concrete paths exactly as it always
|
// **Resolved here, not on the machine.** The host receives concrete paths exactly as it always
|
||||||
// has; nothing new reaches it and it learns no field. Which also means a resolved path changing
|
// has; nothing new reaches it and it learns no field. Which also means a resolved path changing
|
||||||
// is a spec change like any other — and the spec comparison must see it (novox/hq issue 126).
|
// is a spec change like any other — and the spec comparison must see it (novox/hq issue 126).
|
||||||
@@ -27,6 +33,15 @@ import (
|
|||||||
// defaultDataRoot is where module data lands when a node states no root of its own.
|
// defaultDataRoot is where module data lands when a node states no root of its own.
|
||||||
const defaultDataRoot = "/var/lib"
|
const defaultDataRoot = "/var/lib"
|
||||||
|
|
||||||
|
// The two places a pathless directory may name, beside its own id.
|
||||||
|
const (
|
||||||
|
// placeOwn is the assignment's own root, <root>/<module> — to-be 27's one directory per
|
||||||
|
// assignment, which every other placed thing of the module sits beneath.
|
||||||
|
placeOwn = "."
|
||||||
|
// placeMesh is where the mesh keeps what it writes for the module, <root>/mesh/<module>.
|
||||||
|
placeMesh = "mesh"
|
||||||
|
)
|
||||||
|
|
||||||
// dirRef is how a module names one of its placed directories: ${dir:<id>}.
|
// dirRef is how a module names one of its placed directories: ${dir:<id>}.
|
||||||
var dirRef = regexp.MustCompile(`\$\{dir:([a-z0-9][a-z0-9-]*)\}`)
|
var dirRef = regexp.MustCompile(`\$\{dir:([a-z0-9][a-z0-9-]*)\}`)
|
||||||
|
|
||||||
@@ -40,27 +55,54 @@ func dataRoot(with Rendering) string {
|
|||||||
|
|
||||||
// dirsFor is every placed directory of a module, id → the path it resolves to on this node.
|
// dirsFor is every placed directory of a module, id → the path it resolves to on this node.
|
||||||
//
|
//
|
||||||
// A pathless directory saying `"place": "."` is the assignment's own root, <root>/<module> —
|
// A pathless directory saying `"place": "."` is the assignment's own root, <root>/<module>; one
|
||||||
// to-be 27's one directory per assignment, which every other placed thing sits beneath. At most
|
// saying `"place": "mesh"` is the mesh's directory for the module, <root>/mesh/<module>; one
|
||||||
// one makes sense; nothing enforces one, because two ids resolving to one path is a mistake the
|
// saying neither is <root>/<module>/<id>. At most one of each place makes sense; nothing enforces
|
||||||
// module's own files make visible immediately.
|
// one, because two ids resolving to one path is a mistake the module's own files make visible
|
||||||
|
// immediately.
|
||||||
|
//
|
||||||
|
// A stated path may itself begin with a placed reference — `${dir:mesh-state}/state` — and is
|
||||||
|
// filled after the directories it can name are resolved; one level, because a directory beneath
|
||||||
|
// a placed one is the whole of what an adopted layout needs (issue 174's `state` and `out`).
|
||||||
func dirsFor(m Manifest, with Rendering) map[string]string {
|
func dirsFor(m Manifest, with Rendering) map[string]string {
|
||||||
dirs := map[string]string{}
|
dirs := map[string]string{}
|
||||||
|
var beneath []map[string]any
|
||||||
|
// The assignment's placement wins over both (novox/hq issue 153). Refused elsewhere when
|
||||||
|
// malformed; here an invalid setting simply places nothing.
|
||||||
|
placed, _ := Places(m, with.Settings[m.Module])
|
||||||
for _, r := range m.Resources {
|
for _, r := range m.Resources {
|
||||||
if fmt.Sprint(r["type"]) != "directory" {
|
if fmt.Sprint(r["type"]) != "directory" {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
id := fmt.Sprint(r["id"])
|
id := fmt.Sprint(r["id"])
|
||||||
|
if p, said := placed[id]; said {
|
||||||
|
dirs[id] = p.Path
|
||||||
|
continue
|
||||||
|
}
|
||||||
if path, stated := r["path"].(string); stated && path != "" {
|
if path, stated := r["path"].(string); stated && path != "" {
|
||||||
|
if strings.HasPrefix(path, "${dir:") {
|
||||||
|
beneath = append(beneath, r)
|
||||||
|
continue
|
||||||
|
}
|
||||||
dirs[id] = strings.TrimRight(path, "/")
|
dirs[id] = strings.TrimRight(path, "/")
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
if place, said := r["place"].(string); said && place == "." {
|
switch place, _ := r["place"].(string); place {
|
||||||
|
case placeOwn:
|
||||||
dirs[id] = dataRoot(with) + "/" + m.Module
|
dirs[id] = dataRoot(with) + "/" + m.Module
|
||||||
continue
|
case placeMesh:
|
||||||
}
|
dirs[id] = dataRoot(with) + "/mesh/" + m.Module
|
||||||
|
default:
|
||||||
dirs[id] = dataRoot(with) + "/" + m.Module + "/" + id
|
dirs[id] = dataRoot(with) + "/" + m.Module + "/" + id
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
for _, r := range beneath {
|
||||||
|
path := strings.TrimRight(r["path"].(string), "/")
|
||||||
|
// A reference to no directory is left as written and refused where the resource is
|
||||||
|
// placed (dirInto), with the message that names what exists.
|
||||||
|
filled, _ := dirFill(path, dirs, m.Module)
|
||||||
|
dirs[fmt.Sprint(r["id"])] = filled
|
||||||
|
}
|
||||||
return dirs
|
return dirs
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -244,10 +286,11 @@ func (m Manifest) unknownDirRefs() []string {
|
|||||||
"%s states both path and place on %v — a stated path IS the placement",
|
"%s states both path and place on %v — a stated path IS the placement",
|
||||||
m.Module, r["id"]))
|
m.Module, r["id"]))
|
||||||
}
|
}
|
||||||
if place != "." {
|
if place != placeOwn && place != placeMesh {
|
||||||
problems = append(problems, fmt.Sprintf(
|
problems = append(problems, fmt.Sprintf(
|
||||||
"%s says place %q on %v, and the only place is %q — the assignment's own root",
|
"%s says place %q on %v, and the places are %q — the assignment's own root — and "+
|
||||||
m.Module, place, r["id"], "."))
|
"%q — where the mesh keeps what it writes for the module",
|
||||||
|
m.Module, place, r["id"], placeOwn, placeMesh))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
seen := map[string]bool{}
|
seen := map[string]bool{}
|
||||||
|
|||||||
@@ -216,8 +216,48 @@ func TestPlaceIsValidatedAtTheManifest(t *testing.T) {
|
|||||||
wrong := Manifest{Module: "x", Resources: []map[string]any{
|
wrong := Manifest{Module: "x", Resources: []map[string]any{
|
||||||
{"id": "d", "type": "directory", "place": "sub/dir"},
|
{"id": "d", "type": "directory", "place": "sub/dir"},
|
||||||
}}
|
}}
|
||||||
if got := wrong.unknownDirRefs(); len(got) != 1 || !strings.Contains(got[0], `the only place is "."`) {
|
if got := wrong.unknownDirRefs(); len(got) != 1 || !strings.Contains(got[0], `the places are "."`) {
|
||||||
t.Fatalf("a place that is not the root refuses; got %v", got)
|
t.Fatalf("a place that is neither root refuses; got %v", got)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestTheMeshsDirectoryForAModuleIsAPlace(t *testing.T) {
|
||||||
|
// novox/hq issue 174. What the mesh writes for a module — its bus credential, its bindings —
|
||||||
|
// is the mesh's plumbing under <root>/mesh/<module>, and the definition names it by id.
|
||||||
|
m := Manifest{Module: "umami",
|
||||||
|
Resources: []map[string]any{
|
||||||
|
{"id": "mesh-state", "type": "directory", "place": "mesh"},
|
||||||
|
{"id": "state", "type": "directory", "place": "."},
|
||||||
|
{"id": "server", "type": "container", "image": "x@sha256:aa",
|
||||||
|
"volumes": []any{"${dir:mesh-state}/broker:/run/secrets/broker:ro"}},
|
||||||
|
},
|
||||||
|
OwnSecrets: map[string]string{"broker": "${dir:mesh-state}/broker"},
|
||||||
|
Binds: map[string]string{"route": "${dir:state}/route.json"},
|
||||||
|
}
|
||||||
|
if got := m.unknownDirRefs(); len(got) != 0 {
|
||||||
|
t.Fatalf("place %q is a place; got %v", "mesh", got)
|
||||||
|
}
|
||||||
|
dirs := dirsFor(m, Rendering{})
|
||||||
|
if dirs["mesh-state"] != "/var/lib/mesh/umami" || dirs["state"] != "/var/lib/umami" {
|
||||||
|
t.Fatalf("the mesh's directory sits beside the module's, not in it; got %v", dirs)
|
||||||
|
}
|
||||||
|
dirs = dirsFor(m, Rendering{DataRoot: "/srv"})
|
||||||
|
if dirs["mesh-state"] != "/srv/mesh/umami" {
|
||||||
|
t.Fatalf("a node's root moves the mesh's files with the module's; got %v", dirs)
|
||||||
|
}
|
||||||
|
placed, err := placedManifest(m, Rendering{})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if placed.OwnSecrets["broker"] != "/var/lib/mesh/umami/broker" {
|
||||||
|
t.Fatalf("own-secrets are placed under the mesh's directory; got %v", placed.OwnSecrets)
|
||||||
|
}
|
||||||
|
container := shallowCopy(m.Resources[2])
|
||||||
|
if err := dirInto(container, dirsFor(m, Rendering{}), m.Module); err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if container["volumes"].([]any)[0] != "/var/lib/mesh/umami/broker:/run/secrets/broker:ro" {
|
||||||
|
t.Fatalf("the mount's host side is placed; got %v", container["volumes"])
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -250,3 +290,36 @@ func shallowCopy(resource map[string]any) map[string]any {
|
|||||||
}
|
}
|
||||||
return copied
|
return copied
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestADirectoryBeneathAPlacedOneIsPlacedWithIt(t *testing.T) {
|
||||||
|
// An adopted layout keeps a subdirectory the predecessor made under the mesh's directory
|
||||||
|
// (issue 174: a forge's runtime state, a manager's output). Stated as beneath the placed one,
|
||||||
|
// it moves with it — a node's root moves both, and the definition names no host path.
|
||||||
|
m := Manifest{Module: "gitea", Resources: []map[string]any{
|
||||||
|
{"id": "mesh-state", "type": "directory", "place": "mesh"},
|
||||||
|
{"id": "runtime-state", "type": "directory", "path": "${dir:mesh-state}/state"},
|
||||||
|
{"id": "server", "type": "container", "image": "x@sha256:aa",
|
||||||
|
"volumes": []any{"${dir:runtime-state}:/data"}},
|
||||||
|
}}
|
||||||
|
if got := m.unknownDirRefs(); len(got) != 0 {
|
||||||
|
t.Fatalf("a path beneath a placed directory is well formed; got %v", got)
|
||||||
|
}
|
||||||
|
dirs := dirsFor(m, Rendering{DataRoot: "/srv"})
|
||||||
|
if dirs["runtime-state"] != "/srv/mesh/gitea/state" {
|
||||||
|
t.Fatalf("the subdirectory follows the placed one; got %v", dirs)
|
||||||
|
}
|
||||||
|
sub := shallowCopy(m.Resources[1])
|
||||||
|
if err := dirInto(sub, dirs, m.Module); err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if sub["path"] != "/srv/mesh/gitea/state" {
|
||||||
|
t.Fatalf("the directory resource itself is resolved; got %v", sub["path"])
|
||||||
|
}
|
||||||
|
container := shallowCopy(m.Resources[2])
|
||||||
|
if err := dirInto(container, dirs, m.Module); err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if container["volumes"].([]any)[0] != "/srv/mesh/gitea/state:/data" {
|
||||||
|
t.Fatalf("a reference to the subdirectory resolves whole; got %v", container["volumes"])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -91,8 +91,13 @@ const (
|
|||||||
// If the path is absent when a machine applies, the host refuses clearly rather than creating it:
|
// If the path is absent when a machine applies, the host refuses clearly rather than creating it:
|
||||||
// the mesh does not own it, so conjuring it would be a lie the host then acts on.
|
// the mesh does not own it, so conjuring it would be a lie the host then acts on.
|
||||||
type Access struct {
|
type Access struct {
|
||||||
// Path is the absolute path on the machine, as the operator provides it.
|
// ID is the name the module gives this access, which the assignment places
|
||||||
Path string `json:"path"`
|
// (`accesses: {<id>: <path>}`, novox/hq ADR 0112, issue 153) and the module's mounts name as
|
||||||
|
// ${access:<id>}. The shape a definition should use: it names no path of any machine.
|
||||||
|
ID string `json:"id,omitempty"`
|
||||||
|
// Path is the absolute path on the machine. A definition carrying one names an installation;
|
||||||
|
// tolerated as the default the assignment may replace, for accesses declared before ids.
|
||||||
|
Path string `json:"path,omitempty"`
|
||||||
// Mode is "read" or "read-write". Absent narrows to read.
|
// Mode is "read" or "read-write". Absent narrows to read.
|
||||||
Mode string `json:"mode,omitempty"`
|
Mode string `json:"mode,omitempty"`
|
||||||
}
|
}
|
||||||
@@ -1525,7 +1530,16 @@ func ParseManifest(raw []byte) (Manifest, error) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
for _, a := range m.Accesses {
|
for _, a := range m.Accesses {
|
||||||
if !strings.HasPrefix(a.Path, "/") {
|
if a.ID == "" && a.Path == "" {
|
||||||
|
problems = append(problems, fmt.Sprintf(
|
||||||
|
"%s declares an access with neither an id nor a path — an id, which the assignment places",
|
||||||
|
m.Module))
|
||||||
|
}
|
||||||
|
if a.ID != "" && !accessRef.MatchString("${access:"+a.ID+"}") {
|
||||||
|
problems = append(problems, fmt.Sprintf(
|
||||||
|
"%s accesses %q; an access id is lowercase letters, digits and dashes", m.Module, a.ID))
|
||||||
|
}
|
||||||
|
if a.Path != "" && !strings.HasPrefix(a.Path, "/") {
|
||||||
problems = append(problems, fmt.Sprintf(
|
problems = append(problems, fmt.Sprintf(
|
||||||
"%s accesses %q, which is not an absolute path", m.Module, a.Path))
|
"%s accesses %q, which is not an absolute path", m.Module, a.Path))
|
||||||
}
|
}
|
||||||
@@ -1557,6 +1571,7 @@ func ParseManifest(raw []byte) (Manifest, error) {
|
|||||||
// the time it sees the mount it is being asked to create the directory, which it can do.
|
// the time it sees the mount it is being asked to create the directory, which it can do.
|
||||||
problems = append(problems, m.undeclaredMounts()...)
|
problems = append(problems, m.undeclaredMounts()...)
|
||||||
problems = append(problems, m.unknownDirRefs()...)
|
problems = append(problems, m.unknownDirRefs()...)
|
||||||
|
problems = append(problems, m.unknownAccessRefs()...)
|
||||||
|
|
||||||
for i, r := range m.Resources {
|
for i, r := range m.Resources {
|
||||||
id, _ := r["id"].(string)
|
id, _ := r["id"].(string)
|
||||||
|
|||||||
@@ -0,0 +1,124 @@
|
|||||||
|
package catalogue
|
||||||
|
|
||||||
|
import (
|
||||||
|
"sort"
|
||||||
|
"strings"
|
||||||
|
)
|
||||||
|
|
||||||
|
// Which machine serves each routed name (novox/hq ADR 0066, issue 178).
|
||||||
|
//
|
||||||
|
// A routed name is a label the mesh composed for a consumer's endpoint, and it is *served* by the
|
||||||
|
// provider that answers requests for it — the proxy the consumer's route reaches. The same name is
|
||||||
|
// composed into every labelled contribution the consumer makes, because a provider that must know
|
||||||
|
// the consumer's public name (an identity provider composing a redirect) is told it the same way
|
||||||
|
// (04-ISSUES/122). Attributing the name to whichever of those providers a map happened to yield
|
||||||
|
// last sent a public name to the identity provider's machine on one plan and to the proxy's on the
|
||||||
|
// next (forge issue 227), and the whole names region flipped with it.
|
||||||
|
//
|
||||||
|
// **The terminus serves the name.** Among the providers a name reaches, the one that serves it is
|
||||||
|
// the one that is not itself routed: a provider that contributes a labelled name of its own to some
|
||||||
|
// requirement is published through another provider, and is a consumer of names, not their end.
|
||||||
|
// Name-agnostic — nothing here knows what "route" means — and structural: it reads the graph the
|
||||||
|
// modules declared. Deterministic: names, requirements and nodes are walked in order, so two
|
||||||
|
// plans of one mesh yield one region.
|
||||||
|
|
||||||
|
// NamesServed is every routed name across the mesh and the node that serves it, from every node's
|
||||||
|
// resolution and settings. A name several termini claim goes to the first node in name order, so
|
||||||
|
// the answer is stable; a name nothing terminal claims is left out.
|
||||||
|
func NamesServed(plans map[string]Resolution, settings map[string]SettingsBy) (map[string]string, error) {
|
||||||
|
nodes := make([]string, 0, len(plans))
|
||||||
|
for n := range plans {
|
||||||
|
nodes = append(nodes, n)
|
||||||
|
}
|
||||||
|
sort.Strings(nodes)
|
||||||
|
|
||||||
|
out := map[string]string{}
|
||||||
|
for _, node := range nodes {
|
||||||
|
plan := plans[node]
|
||||||
|
all, err := plan.contributions(settings[node], nil, nil)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
requirements := make([]string, 0, len(all))
|
||||||
|
for to := range all {
|
||||||
|
requirements = append(requirements, to)
|
||||||
|
}
|
||||||
|
sort.Strings(requirements)
|
||||||
|
for _, to := range requirements {
|
||||||
|
for _, given := range all[to] {
|
||||||
|
if given.Node != "" {
|
||||||
|
// Said from another machine; that machine's own resolution carries it.
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
// A routed name, and only that: a contribution the mesh composed a name for from a
|
||||||
|
// label it was given. A grant that happens to carry a `name` of its own — a database
|
||||||
|
// name — carries no label and is left alone.
|
||||||
|
if _, labelled := given.Values["label"]; !labelled {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
name, _ := given.Values["name"].(string)
|
||||||
|
if name == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
serving := servingNodeOf(plan, to, given.From, node)
|
||||||
|
if !servesNames(plans[serving], to) {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
name = strings.ToLower(name)
|
||||||
|
if held, taken := out[name]; !taken || serving < held {
|
||||||
|
out[name] = serving
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return out, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
// servingNodeOf is the node answering one consumer's requirement: whoever the plan needs it from,
|
||||||
|
// or this same node when the provider is beside the consumer.
|
||||||
|
func servingNodeOf(plan Resolution, requirement, consumer, self string) string {
|
||||||
|
for _, need := range plan.Needs {
|
||||||
|
if need.Name == requirement && need.For == consumer && need.From != "" {
|
||||||
|
return need.From
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return self
|
||||||
|
}
|
||||||
|
|
||||||
|
// servesNames says whether the module providing a requirement on a node is a terminus: it is not
|
||||||
|
// itself published under a labelled name through some other provider. A node whose plan is not
|
||||||
|
// known (it did not resolve) serves nothing.
|
||||||
|
func servesNames(plan Resolution, requirement string) bool {
|
||||||
|
for _, m := range plan.Modules {
|
||||||
|
if !offers(m, requirement) {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
return !contributesALabel(m)
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
func offers(m Manifest, requirement string) bool {
|
||||||
|
for _, o := range m.Offers() {
|
||||||
|
if o == requirement {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
func contributesALabel(m Manifest) bool {
|
||||||
|
for _, values := range m.Contributes {
|
||||||
|
if _, labelled := values["label"]; labelled {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, locals := range m.ContributesMany {
|
||||||
|
for _, values := range locals {
|
||||||
|
if _, labelled := values["label"]; labelled {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
@@ -0,0 +1,99 @@
|
|||||||
|
package catalogue
|
||||||
|
|
||||||
|
import (
|
||||||
|
"testing"
|
||||||
|
)
|
||||||
|
|
||||||
|
// The mesh of forge issue 227 (novox/hq issue 178): a dashboard on the home server contributes its
|
||||||
|
// label to the route its proxy serves AND to the identity provider on the control node, which must
|
||||||
|
// know the dashboard's public name to compose a redirect. Both contributions carry the composed
|
||||||
|
// name; only the proxy serves it.
|
||||||
|
func twoNodesOneName(t *testing.T) (map[string]Resolution, map[string]SettingsBy) {
|
||||||
|
t.Helper()
|
||||||
|
catalogue := shelf(
|
||||||
|
Manifest{Module: "route-adapter", Version: "1", Provides: Offers("route"),
|
||||||
|
Serves: map[string]map[string]any{"route": {}}, Receives: map[string]string{"route": "/etc/adapter/mesh.json"}},
|
||||||
|
Manifest{Module: "route-proxy", Version: "1", Provides: Offers("route"),
|
||||||
|
Serves: map[string]map[string]any{"route": {}}, Receives: map[string]string{"route": "/etc/proxy/mesh.json"}},
|
||||||
|
Manifest{Module: "keycloak", Version: "1", Provides: FromAnywhere("oidc-client"),
|
||||||
|
Serves: map[string]map[string]any{"oidc-client": {"token-path": "/token"}},
|
||||||
|
Receives: map[string]string{"oidc-client": "/etc/keycloak/clients.json"},
|
||||||
|
Listens: []Listening{{Port: 8080, From: FromMesh, Why: "the login page"}},
|
||||||
|
// Published through the proxy itself: the identity provider is routed, not a router.
|
||||||
|
Contributes: map[string]map[string]any{"route": {"label": "login", "endpoint": "web", "port": 8080}}},
|
||||||
|
Manifest{Module: "grafana", Version: "1",
|
||||||
|
Listens: []Listening{{Port: 3000, From: FromMesh, Why: "dashboards"}},
|
||||||
|
Contributes: map[string]map[string]any{
|
||||||
|
"route": {"label": "grafana", "endpoint": "web", "port": 3000},
|
||||||
|
"oidc-client": {"label": "grafana", "endpoint": "web", "port": 3000, "callback": "/login"},
|
||||||
|
}},
|
||||||
|
)
|
||||||
|
home := withDomain("home.example")
|
||||||
|
home.Name, home.At = "home-server", "home-server.internal"
|
||||||
|
control := withDomain("control.example")
|
||||||
|
control.Name, control.At = "anchor", "anchor.internal"
|
||||||
|
|
||||||
|
onHome, err := Resolve(catalogue, []string{"grafana", "route-adapter"}, home, World{
|
||||||
|
Offered: map[string][]Provider{"oidc-client": {{Node: "anchor", At: "anchor.internal", Module: "keycloak"}}},
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
onControl, err := Resolve(catalogue, []string{"keycloak", "route-proxy"}, control, World{})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
return map[string]Resolution{"home-server": onHome, "anchor": onControl},
|
||||||
|
map[string]SettingsBy{"home-server": {}, "anchor": {}}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestANameResolvesToTheNodeWhoseProxyServesIt(t *testing.T) {
|
||||||
|
plans, settings := twoNodesOneName(t)
|
||||||
|
// Many times, because the fault was map order: one plan said one node, the next the other.
|
||||||
|
for i := 0; i < 25; i++ {
|
||||||
|
served, err := NamesServed(plans, settings)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if served["grafana.home.example"] != "home-server" {
|
||||||
|
t.Fatalf("run %d: the dashboard's name is served by %q, and its proxy is on the home server: %v",
|
||||||
|
i, served["grafana.home.example"], served)
|
||||||
|
}
|
||||||
|
if served["login.control.example"] != "anchor" {
|
||||||
|
t.Fatalf("run %d: the identity provider's own name is served by its proxy on the control node: %v", i, served)
|
||||||
|
}
|
||||||
|
if _, leaked := served["grafana.control.example"]; leaked {
|
||||||
|
t.Fatalf("a name composed for the identity provider's benefit is not one it serves: %v", served)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// A module that is routed several times names each route (ADR 0094's sibling for contributes);
|
||||||
|
// every one of them is a name the mesh must resolve, and none reached the names region before.
|
||||||
|
func TestEveryRouteOfAModuleWithSeveralIsANameServed(t *testing.T) {
|
||||||
|
catalogue := shelf(
|
||||||
|
Manifest{Module: "route-proxy", Version: "1", Provides: Offers("route"),
|
||||||
|
Serves: map[string]map[string]any{"route": {}}, Receives: map[string]string{"route": "/etc/proxy/mesh.json"}},
|
||||||
|
Manifest{Module: "photos", Version: "1",
|
||||||
|
Listens: []Listening{{Port: 8102, From: FromMesh, Why: "web"}, {Port: 9102, From: FromMesh, Why: "api"}},
|
||||||
|
ContributesMany: map[string]map[string]map[string]any{"route": {
|
||||||
|
"site": {"label": "photos", "endpoint": "web", "port": 8102},
|
||||||
|
"api": {"label": "photos-api", "endpoint": "api", "port": 9102},
|
||||||
|
}}},
|
||||||
|
)
|
||||||
|
node := withDomain("control.example")
|
||||||
|
node.Name, node.At = "anchor", "anchor.internal"
|
||||||
|
plan, err := Resolve(catalogue, []string{"photos", "route-proxy"}, node, World{})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
served, err := NamesServed(map[string]Resolution{"anchor": plan}, map[string]SettingsBy{"anchor": {}})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
for _, name := range []string{"photos.control.example", "photos-api.control.example"} {
|
||||||
|
if served[name] != "anchor" {
|
||||||
|
t.Fatalf("%s is not served by its proxy: %v", name, served)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -16,7 +16,9 @@ import (
|
|||||||
//
|
//
|
||||||
// Two checkouts: MESH_CATALOGUE_BEFORE, the catalogue as it was, and MESH_CATALOGUE, as it is now.
|
// Two checkouts: MESH_CATALOGUE_BEFORE, the catalogue as it was, and MESH_CATALOGUE, as it is now.
|
||||||
// Every module in both is resolved with the controller's own rule (dirsFor, dirFill) and compared
|
// Every module in both is resolved with the controller's own rule (dirsFor, dirFill) and compared
|
||||||
// whole — not only the directories, but every string a directory's id was written into.
|
// whole — not only the directories, but every string a directory's id was written into. Both
|
||||||
|
// sides are resolved, so a manifest converted in two steps (issue 119, then issue 174) is judged
|
||||||
|
// against the paths it named, not the text it used to name them with.
|
||||||
func TestPlacedDirectoriesKeepTheirPaths(t *testing.T) {
|
func TestPlacedDirectoriesKeepTheirPaths(t *testing.T) {
|
||||||
before, after := os.Getenv("MESH_CATALOGUE_BEFORE"), os.Getenv("MESH_CATALOGUE")
|
before, after := os.Getenv("MESH_CATALOGUE_BEFORE"), os.Getenv("MESH_CATALOGUE")
|
||||||
if before == "" || after == "" {
|
if before == "" || after == "" {
|
||||||
@@ -42,7 +44,11 @@ func TestPlacedDirectoriesKeepTheirPaths(t *testing.T) {
|
|||||||
t.Errorf("%s: %v", module, err)
|
t.Errorf("%s: %v", module, err)
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
dirs := dirsFor(m, Rendering{})
|
earlier, err := ParseManifest(old)
|
||||||
|
if err != nil {
|
||||||
|
t.Errorf("%s before: %v", module, err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
var was, is any
|
var was, is any
|
||||||
if err := json.Unmarshal(old, &was); err != nil {
|
if err := json.Unmarshal(old, &was); err != nil {
|
||||||
t.Fatal(err)
|
t.Fatal(err)
|
||||||
@@ -50,7 +56,15 @@ func TestPlacedDirectoriesKeepTheirPaths(t *testing.T) {
|
|||||||
if err := json.Unmarshal(now, &is); err != nil {
|
if err := json.Unmarshal(now, &is); err != nil {
|
||||||
t.Fatal(err)
|
t.Fatal(err)
|
||||||
}
|
}
|
||||||
resolved := resolvedTree(is, dirs, module, t)
|
// Both sides resolved: the manifest before may itself already place some directories
|
||||||
|
// (issue 119's conversion), and what must not move is the path a machine sees.
|
||||||
|
was = resolvedTree(was, dirsFor(earlier, Rendering{}), module, t)
|
||||||
|
resolved := resolvedTree(is, dirsFor(m, Rendering{}), module, t)
|
||||||
|
// An access named by id resolves to the path the definition still carries as its default
|
||||||
|
// (issue 153) — the same rule as a placed directory: an assignment that says nothing moves
|
||||||
|
// nothing.
|
||||||
|
was = accessesResolved(was, earlier)
|
||||||
|
resolved = accessesResolved(resolved, m)
|
||||||
if !reflect.DeepEqual(was, resolved) {
|
if !reflect.DeepEqual(was, resolved) {
|
||||||
wasJSON, _ := json.MarshalIndent(was, "", " ")
|
wasJSON, _ := json.MarshalIndent(was, "", " ")
|
||||||
isJSON, _ := json.MarshalIndent(resolved, "", " ")
|
isJSON, _ := json.MarshalIndent(resolved, "", " ")
|
||||||
@@ -119,3 +133,48 @@ func firstDifference(a, b string) string {
|
|||||||
}
|
}
|
||||||
return "(the difference is beyond the shorter document)"
|
return "(the difference is beyond the shorter document)"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// accessesResolved fills every ${access:<id>} with the default path the definition carries for that
|
||||||
|
// access, and drops the id, so a manifest that names its accesses is compared by the paths a machine
|
||||||
|
// with no placement receives.
|
||||||
|
func accessesResolved(node any, m Manifest) any {
|
||||||
|
defaults := map[string]string{}
|
||||||
|
for _, a := range m.Accesses {
|
||||||
|
if a.ID != "" && a.Path != "" {
|
||||||
|
defaults[a.ID] = a.Path
|
||||||
|
}
|
||||||
|
}
|
||||||
|
var walk func(any) any
|
||||||
|
walk = func(n any) any {
|
||||||
|
switch v := n.(type) {
|
||||||
|
case map[string]any:
|
||||||
|
out := map[string]any{}
|
||||||
|
for k, child := range v {
|
||||||
|
if k == "id" {
|
||||||
|
if _, isAccess := v["mode"]; isAccess && v["type"] == nil {
|
||||||
|
if _, hasPath := v["path"]; hasPath {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
out[k] = walk(child)
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
case []any:
|
||||||
|
out := make([]any, len(v))
|
||||||
|
for i, child := range v {
|
||||||
|
out[i] = walk(child)
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
case string:
|
||||||
|
return accessRef.ReplaceAllStringFunc(v, func(ref string) string {
|
||||||
|
if p, ok := defaults[accessRef.FindStringSubmatch(ref)[1]]; ok {
|
||||||
|
return p
|
||||||
|
}
|
||||||
|
return ref
|
||||||
|
})
|
||||||
|
}
|
||||||
|
return n
|
||||||
|
}
|
||||||
|
return walk(node)
|
||||||
|
}
|
||||||
|
|||||||
@@ -0,0 +1,382 @@
|
|||||||
|
package catalogue
|
||||||
|
|
||||||
|
import (
|
||||||
|
"fmt"
|
||||||
|
"regexp"
|
||||||
|
"sort"
|
||||||
|
"strings"
|
||||||
|
)
|
||||||
|
|
||||||
|
// Where a module's data is on THIS machine is the assignment's (novox/hq ADR 0112, issue 153).
|
||||||
|
//
|
||||||
|
// A definition names no host path. It declares the directories it owns by id, and the operator's
|
||||||
|
// shared data it needs by id too (an `access`, ADR 0051). A node has a default layout for the
|
||||||
|
// former — <root>/<module>/<id> — and nothing at all for the latter, because shared data is
|
||||||
|
// wherever the operator keeps it. An adopted machine keeps its data where the predecessor put it:
|
||||||
|
// a 40 TB library on its own pool, a configuration on a second disk. Both halves are said on the
|
||||||
|
// assignment, validated the way `endpoints` is — an id the module does not declare is refused,
|
||||||
|
// because a setting that reaches nothing is a mistake — and resolved here, so the host receives
|
||||||
|
// concrete paths exactly as it always has and learns no field.
|
||||||
|
//
|
||||||
|
// {"places": {"config": "/services/sonarr/config",
|
||||||
|
// "data": {"path": "/mnt/plex/data", "owner": "1000:1000"}},
|
||||||
|
// "accesses": {"series": "/storage/media/series",
|
||||||
|
// "downloads": "/storage/downloads"}}
|
||||||
|
//
|
||||||
|
// A placed directory is still the mesh's: created, chowned to the owner the assignment says (or
|
||||||
|
// the manifest's), removed when empty and no longer declared. A placed access is still the
|
||||||
|
// operator's: mounted, never created, chowned or removed.
|
||||||
|
|
||||||
|
// PlacesSetting is the settings key that places a module's declared directories, by id.
|
||||||
|
const PlacesSetting = "places"
|
||||||
|
|
||||||
|
// AccessesSetting is the settings key that says where a module's accesses are on this node, by id.
|
||||||
|
const AccessesSetting = "accesses"
|
||||||
|
|
||||||
|
// Placement is what an assignment says about one of a module's directories.
|
||||||
|
type Placement struct {
|
||||||
|
// Path is where the directory is on this machine. Absolute.
|
||||||
|
Path string
|
||||||
|
// Owner is "uid:gid" when the assignment overrides the manifest's — the predecessor's data is
|
||||||
|
// owned by whoever it ran as, and that is one machine's fact.
|
||||||
|
Owner string
|
||||||
|
}
|
||||||
|
|
||||||
|
var ownerShape = regexp.MustCompile(`^[0-9]+:[0-9]+$`)
|
||||||
|
|
||||||
|
// accessRef is how a module names one of its accesses: ${access:<id>}.
|
||||||
|
var accessRef = regexp.MustCompile(`\$\{access:([a-z0-9][a-z0-9-]*)\}`)
|
||||||
|
|
||||||
|
// Places reads where this node places the module's directories, by directory id.
|
||||||
|
//
|
||||||
|
// It refuses an id the module declares no directory for, a path that is not absolute, and an
|
||||||
|
// owner that is not uid:gid. A directory the assignment does not mention keeps the manifest's
|
||||||
|
// stated path or the node's default layout.
|
||||||
|
func Places(m Manifest, layers []Layer) (map[string]Placement, error) {
|
||||||
|
declared := map[string]bool{}
|
||||||
|
for _, r := range m.Resources {
|
||||||
|
if fmt.Sprint(r["type"]) == "directory" {
|
||||||
|
declared[fmt.Sprint(r["id"])] = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
out := map[string]Placement{}
|
||||||
|
for _, layer := range layers {
|
||||||
|
raw, ok := layer.Values[PlacesSetting]
|
||||||
|
if !ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
blocks, ok := raw.(map[string]any)
|
||||||
|
if !ok {
|
||||||
|
return nil, fmt.Errorf("%s: %s is a { directory: path | { path, owner } } map, and %q set it to something else",
|
||||||
|
m.Module, PlacesSetting, layer.From)
|
||||||
|
}
|
||||||
|
for id, body := range blocks {
|
||||||
|
if !declared[id] {
|
||||||
|
return nil, fmt.Errorf(
|
||||||
|
"%s places the directory %q, which it does not declare — the setting reaches "+
|
||||||
|
"nothing. It declares %s", m.Module, id, orNothing(namesOfDirs(directoriesOf(m))))
|
||||||
|
}
|
||||||
|
p := out[id]
|
||||||
|
switch v := body.(type) {
|
||||||
|
case string:
|
||||||
|
p.Path = strings.TrimSpace(v)
|
||||||
|
case map[string]any:
|
||||||
|
if path, said := v["path"]; said {
|
||||||
|
text, _ := path.(string)
|
||||||
|
p.Path = strings.TrimSpace(text)
|
||||||
|
}
|
||||||
|
if owner, said := v["owner"]; said {
|
||||||
|
text, _ := owner.(string)
|
||||||
|
if !ownerShape.MatchString(strings.TrimSpace(text)) {
|
||||||
|
return nil, fmt.Errorf("%s places %q with owner %v; an owner is uid:gid, numeric",
|
||||||
|
m.Module, id, owner)
|
||||||
|
}
|
||||||
|
p.Owner = strings.TrimSpace(text)
|
||||||
|
}
|
||||||
|
default:
|
||||||
|
return nil, fmt.Errorf("%s places %q with %v; a placement is a path, or { path, owner }",
|
||||||
|
m.Module, id, body)
|
||||||
|
}
|
||||||
|
if p.Path == "" {
|
||||||
|
return nil, fmt.Errorf("%s places %q without a path", m.Module, id)
|
||||||
|
}
|
||||||
|
if !strings.HasPrefix(p.Path, "/") {
|
||||||
|
return nil, fmt.Errorf("%s places %q at %q, which is not an absolute path", m.Module, id, p.Path)
|
||||||
|
}
|
||||||
|
p.Path = strings.TrimRight(p.Path, "/")
|
||||||
|
out[id] = p
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if len(out) == 0 {
|
||||||
|
return nil, nil
|
||||||
|
}
|
||||||
|
return out, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
// AccessPlaces reads where this node keeps the operator's data the module accesses, by access id.
|
||||||
|
//
|
||||||
|
// It refuses an id the module declares no access under, and a path that is not absolute. An
|
||||||
|
// access declared by path alone cannot be placed — it has no name to place it by.
|
||||||
|
func AccessPlaces(m Manifest, layers []Layer) (map[string]string, error) {
|
||||||
|
declared := map[string]bool{}
|
||||||
|
for _, a := range m.Accesses {
|
||||||
|
if a.ID != "" {
|
||||||
|
declared[a.ID] = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
out := map[string]string{}
|
||||||
|
for _, layer := range layers {
|
||||||
|
raw, ok := layer.Values[AccessesSetting]
|
||||||
|
if !ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
blocks, ok := raw.(map[string]any)
|
||||||
|
if !ok {
|
||||||
|
return nil, fmt.Errorf("%s: %s is a { access: path } map, and %q set it to something else",
|
||||||
|
m.Module, AccessesSetting, layer.From)
|
||||||
|
}
|
||||||
|
for id, body := range blocks {
|
||||||
|
if !declared[id] {
|
||||||
|
return nil, fmt.Errorf(
|
||||||
|
"%s places the access %q, which it does not declare — the setting reaches "+
|
||||||
|
"nothing. It declares %s", m.Module, id, orNothing(namesOfAccesses(m)))
|
||||||
|
}
|
||||||
|
path, _ := body.(string)
|
||||||
|
path = strings.TrimSpace(path)
|
||||||
|
if !strings.HasPrefix(path, "/") {
|
||||||
|
return nil, fmt.Errorf("%s places the access %q at %v, which is not an absolute path",
|
||||||
|
m.Module, id, body)
|
||||||
|
}
|
||||||
|
out[id] = strings.TrimRight(path, "/")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if len(out) == 0 {
|
||||||
|
return nil, nil
|
||||||
|
}
|
||||||
|
return out, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
// placedAccess is one access with the path it resolves to on this node.
|
||||||
|
type placedAccess struct {
|
||||||
|
ID string
|
||||||
|
Path string
|
||||||
|
Mode string
|
||||||
|
}
|
||||||
|
|
||||||
|
// accessesFor is every access of a module with its path on this node: the assignment's where it
|
||||||
|
// placed one, the definition's where it carries a default, and refused where neither says — an
|
||||||
|
// access that resolves to nowhere would reach the machine as a mount of nothing.
|
||||||
|
func accessesFor(m Manifest, layers []Layer) ([]placedAccess, map[string]string, error) {
|
||||||
|
placed, err := AccessPlaces(m, layers)
|
||||||
|
if err != nil {
|
||||||
|
return nil, nil, err
|
||||||
|
}
|
||||||
|
var out []placedAccess
|
||||||
|
byID := map[string]string{}
|
||||||
|
for _, a := range m.Accesses {
|
||||||
|
path := a.Path
|
||||||
|
if a.ID != "" {
|
||||||
|
if at, said := placed[a.ID]; said {
|
||||||
|
path = at
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if path == "" {
|
||||||
|
return nil, nil, fmt.Errorf(
|
||||||
|
"%s accesses %q, and nothing says where that is on this node — the definition "+
|
||||||
|
"carries no path (it must not, novox/hq ADR 0112) and the assignment places "+
|
||||||
|
"none. Set %s: {%q: \"/where/it/is\"}",
|
||||||
|
m.Module, a.ID, AccessesSetting, a.ID)
|
||||||
|
}
|
||||||
|
out = append(out, placedAccess{ID: a.ID, Path: path, Mode: a.At()})
|
||||||
|
if a.ID != "" {
|
||||||
|
byID[a.ID] = path
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return out, byID, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
// accessFill resolves every ${access:…} in one string, or refuses a reference naming no access.
|
||||||
|
func accessFill(s string, accesses map[string]string, module string) (string, error) {
|
||||||
|
var missing error
|
||||||
|
out := accessRef.ReplaceAllStringFunc(s, func(ref string) string {
|
||||||
|
id := accessRef.FindStringSubmatch(ref)[1]
|
||||||
|
path, has := accesses[id]
|
||||||
|
if !has {
|
||||||
|
missing = fmt.Errorf(
|
||||||
|
"%s says ${access:%s}, and %s declares no access %q. It declares %s",
|
||||||
|
module, id, module, id, orNothing(namesOfAccessIDs(accesses)))
|
||||||
|
return ref
|
||||||
|
}
|
||||||
|
return path
|
||||||
|
})
|
||||||
|
return out, missing
|
||||||
|
}
|
||||||
|
|
||||||
|
// accessInto fills every ${access:…} a resource carries — in its path, its content, its mounts,
|
||||||
|
// its environment and its env-files — with the path this node resolved for it. The same walk as
|
||||||
|
// dirInto, for the same reason: a literal `${access:x}` reaching the machine would be mounted as
|
||||||
|
// a directory called that.
|
||||||
|
func accessInto(resource map[string]any, accesses map[string]string, module string) error {
|
||||||
|
if !mentionsAccess(resource) {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
fill := func(s string) (string, error) { return accessFill(s, accesses, module) }
|
||||||
|
var err error
|
||||||
|
if path, ok := resource["path"].(string); ok {
|
||||||
|
if resource["path"], err = fill(path); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if content, ok := resource["content"].(string); ok {
|
||||||
|
if resource["content"], err = fill(content); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, field := range []string{"volumes", "env-file"} {
|
||||||
|
list, ok := resource[field].([]any)
|
||||||
|
if !ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
filled := make([]any, len(list))
|
||||||
|
for i, v := range list {
|
||||||
|
filled[i] = v
|
||||||
|
if s, ok := v.(string); ok {
|
||||||
|
if filled[i], err = fill(s); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
resource[field] = filled
|
||||||
|
}
|
||||||
|
if env, ok := resource["env"].(map[string]any); ok {
|
||||||
|
filled := make(map[string]any, len(env))
|
||||||
|
for key, v := range env {
|
||||||
|
filled[key] = v
|
||||||
|
if s, ok := v.(string); ok {
|
||||||
|
if filled[key], err = fill(s); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
resource["env"] = filled
|
||||||
|
}
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
|
func mentionsAccess(resource map[string]any) bool {
|
||||||
|
for _, field := range []string{"path", "content"} {
|
||||||
|
if s, ok := resource[field].(string); ok && accessRef.MatchString(s) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, field := range []string{"volumes", "env-file"} {
|
||||||
|
if list, ok := resource[field].([]any); ok {
|
||||||
|
for _, v := range list {
|
||||||
|
if s, ok := v.(string); ok && accessRef.MatchString(s) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if env, ok := resource["env"].(map[string]any); ok {
|
||||||
|
for _, v := range env {
|
||||||
|
if s, ok := v.(string); ok && accessRef.MatchString(s) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
// ownerInto gives a placed directory the owner the assignment said, where it said one. The
|
||||||
|
// manifest's owner is what the image expects on any machine; the assignment's is what this
|
||||||
|
// machine's data already is.
|
||||||
|
func ownerInto(resource map[string]any, placed map[string]Placement) {
|
||||||
|
if fmt.Sprint(resource["type"]) != "directory" {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if p, ok := placed[fmt.Sprint(resource["id"])]; ok && p.Owner != "" {
|
||||||
|
resource["owner"] = p.Owner
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// unknownAccessRefs is every ${access:…} in the definition that names no access the definition
|
||||||
|
// declares by id — refused where the author is, as unknownDirRefs does for directories.
|
||||||
|
func (m Manifest) unknownAccessRefs() []string {
|
||||||
|
declared := map[string]bool{}
|
||||||
|
for _, a := range m.Accesses {
|
||||||
|
if a.ID != "" {
|
||||||
|
declared[a.ID] = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
seen := map[string]bool{}
|
||||||
|
var problems []string
|
||||||
|
refuse := func(s string, where any) {
|
||||||
|
for _, match := range accessRef.FindAllStringSubmatch(s, -1) {
|
||||||
|
id := match[1]
|
||||||
|
if declared[id] || seen[id] {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
seen[id] = true
|
||||||
|
problems = append(problems, fmt.Sprintf(
|
||||||
|
"%s says ${access:%s} in %v, and declares no access %q — a reference the mesh "+
|
||||||
|
"cannot place would reach the machine as a literal path",
|
||||||
|
m.Module, id, where, id))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, r := range m.Resources {
|
||||||
|
for _, field := range []string{"path", "content"} {
|
||||||
|
if s, ok := r[field].(string); ok {
|
||||||
|
refuse(s, r["id"])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, field := range []string{"volumes", "env-file"} {
|
||||||
|
if list, ok := r[field].([]any); ok {
|
||||||
|
for _, v := range list {
|
||||||
|
if s, ok := v.(string); ok {
|
||||||
|
refuse(s, r["id"])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if env, ok := r["env"].(map[string]any); ok {
|
||||||
|
for _, v := range env {
|
||||||
|
if s, ok := v.(string); ok {
|
||||||
|
refuse(s, r["id"])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
sort.Strings(problems)
|
||||||
|
return problems
|
||||||
|
}
|
||||||
|
|
||||||
|
func directoriesOf(m Manifest) map[string]string {
|
||||||
|
dirs := map[string]string{}
|
||||||
|
for _, r := range m.Resources {
|
||||||
|
if fmt.Sprint(r["type"]) == "directory" {
|
||||||
|
dirs[fmt.Sprint(r["id"])] = ""
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return dirs
|
||||||
|
}
|
||||||
|
|
||||||
|
func namesOfAccesses(m Manifest) []string {
|
||||||
|
var names []string
|
||||||
|
for _, a := range m.Accesses {
|
||||||
|
if a.ID != "" {
|
||||||
|
names = append(names, fmt.Sprintf("%q", a.ID))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
sort.Strings(names)
|
||||||
|
return names
|
||||||
|
}
|
||||||
|
|
||||||
|
func namesOfAccessIDs(accesses map[string]string) []string {
|
||||||
|
var names []string
|
||||||
|
for id := range accesses {
|
||||||
|
names = append(names, fmt.Sprintf("%q", id))
|
||||||
|
}
|
||||||
|
sort.Strings(names)
|
||||||
|
return names
|
||||||
|
}
|
||||||
@@ -0,0 +1,172 @@
|
|||||||
|
package catalogue
|
||||||
|
|
||||||
|
import (
|
||||||
|
"strings"
|
||||||
|
"testing"
|
||||||
|
)
|
||||||
|
|
||||||
|
// The case novox/hq issue 153 records: an adopted machine keeps its data where the predecessor put
|
||||||
|
// it — a library on its own pool that must never move, a configuration on a second disk owned by
|
||||||
|
// whoever the predecessor ran as. A definition may name none of that (ADR 0112); the assignment
|
||||||
|
// says it, by the ids the definition declared, and the machine receives concrete paths as always.
|
||||||
|
func placeable() Manifest {
|
||||||
|
m := mod("arr", nil, nil, nil)
|
||||||
|
m.Resources = []map[string]any{
|
||||||
|
{"id": "state", "type": "directory", "place": ".", "mode": "0700"},
|
||||||
|
{"id": "config", "type": "directory", "mode": "0755", "owner": "1000:1000"},
|
||||||
|
{"id": "server", "type": "container", "name": "arr", "image": "arr:1",
|
||||||
|
"volumes": []any{"${dir:config}:/config", "${access:series}:/series", "${access:spool}:/downloads:ro"},
|
||||||
|
"env": map[string]any{"SPOOL": "${access:spool}"}},
|
||||||
|
}
|
||||||
|
m.Accesses = []Access{{ID: "series", Mode: AccessReadWrite}, {ID: "spool"}}
|
||||||
|
return m
|
||||||
|
}
|
||||||
|
|
||||||
|
func placedBy(values map[string]any) Rendering {
|
||||||
|
return Rendering{Settings: SettingsBy{"arr": {{From: "node anchor", Values: values}}}}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAnAssignmentPlacesDirectoriesAndAccesses(t *testing.T) {
|
||||||
|
got, err := Resolve(shelf(placeable()), []string{"arr"}, workstation(), World{})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
out, err := got.Declaration(placedBy(map[string]any{
|
||||||
|
PlacesSetting: map[string]any{
|
||||||
|
"config": map[string]any{"path": "/services/arr/config/", "owner": "1001:2000"},
|
||||||
|
},
|
||||||
|
AccessesSetting: map[string]any{
|
||||||
|
"series": "/storage/media/series",
|
||||||
|
"spool": "/storage/downloads",
|
||||||
|
},
|
||||||
|
}))
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
seen := map[string]map[string]any{}
|
||||||
|
for _, r := range out {
|
||||||
|
seen[r["id"].(string)] = r
|
||||||
|
}
|
||||||
|
config := seen["arr.config"]
|
||||||
|
if config["path"] != "/services/arr/config" || config["owner"] != "1001:2000" {
|
||||||
|
t.Fatalf("the placed directory is %v %v; want the assignment's path and owner", config["path"], config["owner"])
|
||||||
|
}
|
||||||
|
if seen["arr.state"]["path"] != "/var/lib/arr" {
|
||||||
|
t.Fatalf("an unplaced directory left the default layout: %v", seen["arr.state"]["path"])
|
||||||
|
}
|
||||||
|
var accesses []string
|
||||||
|
for _, r := range out {
|
||||||
|
if r["type"] == "access" {
|
||||||
|
accesses = append(accesses, r["path"].(string)+" "+r["mode"].(string))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if strings.Join(accesses, ",") != "/storage/media/series read-write,/storage/downloads read" {
|
||||||
|
t.Fatalf("the accesses reached the machine as %v", accesses)
|
||||||
|
}
|
||||||
|
server := seen["arr.server"]
|
||||||
|
mounts := server["volumes"].([]any)
|
||||||
|
if mounts[0] != "/services/arr/config:/config" || mounts[1] != "/storage/media/series:/series" ||
|
||||||
|
mounts[2] != "/storage/downloads:/downloads:ro" {
|
||||||
|
t.Fatalf("the mounts were not filled with the placed paths: %v", mounts)
|
||||||
|
}
|
||||||
|
if server["env"].(map[string]any)["SPOOL"] != "/storage/downloads" {
|
||||||
|
t.Fatalf("the environment was not filled: %v", server["env"])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// An access declared by id and placed by nobody resolves to nowhere, and that is refused with the
|
||||||
|
// setting to write — not mounted as the literal, not skipped.
|
||||||
|
func TestAnUnplacedAccessIsRefusedByName(t *testing.T) {
|
||||||
|
got, err := Resolve(shelf(placeable()), []string{"arr"}, workstation(), World{})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
_, err = got.Declaration(placedBy(map[string]any{
|
||||||
|
AccessesSetting: map[string]any{"series": "/storage/media/series"},
|
||||||
|
}))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), `"spool"`) || !strings.Contains(err.Error(), AccessesSetting) {
|
||||||
|
t.Fatalf("an access nobody placed was not refused by name: %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Validated like endpoints: an id the module does not declare reaches nothing, and the refusal
|
||||||
|
// says what it does declare; a relative path and a non-numeric owner are refused too.
|
||||||
|
func TestPlacementsAreValidated(t *testing.T) {
|
||||||
|
m := placeable()
|
||||||
|
layers := func(values map[string]any) []Layer { return placedBy(values).Settings["arr"] }
|
||||||
|
|
||||||
|
_, err := Places(m, layers(map[string]any{PlacesSetting: map[string]any{"data": "/mnt/data"}}))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "does not declare") || !strings.Contains(err.Error(), `"config"`) {
|
||||||
|
t.Fatalf("placing an undeclared directory was accepted: %v", err)
|
||||||
|
}
|
||||||
|
_, err = Places(m, layers(map[string]any{PlacesSetting: map[string]any{"config": "services/arr"}}))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "absolute") {
|
||||||
|
t.Fatalf("a relative placement was accepted: %v", err)
|
||||||
|
}
|
||||||
|
_, err = Places(m, layers(map[string]any{PlacesSetting: map[string]any{
|
||||||
|
"config": map[string]any{"path": "/services/arr", "owner": "media"}}}))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "uid:gid") {
|
||||||
|
t.Fatalf("a non-numeric owner was accepted: %v", err)
|
||||||
|
}
|
||||||
|
_, err = AccessPlaces(m, layers(map[string]any{AccessesSetting: map[string]any{"movies": "/storage/media/movies"}}))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "does not declare") || !strings.Contains(err.Error(), `"series"`) {
|
||||||
|
t.Fatalf("placing an undeclared access was accepted: %v", err)
|
||||||
|
}
|
||||||
|
_, err = AccessPlaces(m, layers(map[string]any{AccessesSetting: map[string]any{"series": "media/series"}}))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "absolute") {
|
||||||
|
t.Fatalf("a relative access was accepted: %v", err)
|
||||||
|
}
|
||||||
|
// And the two keys are never stray: they are validated here, not merged into a file.
|
||||||
|
if stray := UnusedSettings(m, layers(map[string]any{
|
||||||
|
PlacesSetting: map[string]any{"config": "/services/arr/config"},
|
||||||
|
AccessesSetting: map[string]any{"series": "/storage/media/series"},
|
||||||
|
})); len(stray) != 0 {
|
||||||
|
t.Fatalf("the placement keys were reported as unused: %v", stray)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// A definition that carries a path still works, as the default the assignment may replace — and
|
||||||
|
// the assignment's placement wins where both say.
|
||||||
|
func TestADefinitionsPathIsTheDefaultTheAssignmentReplaces(t *testing.T) {
|
||||||
|
m := placeable()
|
||||||
|
m.Accesses = []Access{{ID: "series", Path: "/services/media/series", Mode: AccessReadWrite}, {ID: "spool", Path: "/services/media/downloads"}}
|
||||||
|
got, err := Resolve(shelf(m), []string{"arr"}, workstation(), World{})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
out, err := got.Declaration(placedBy(map[string]any{
|
||||||
|
PlacesSetting: map[string]any{"config": "/services/arr/config"},
|
||||||
|
AccessesSetting: map[string]any{"series": "/storage/media/series"},
|
||||||
|
}))
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
var paths []string
|
||||||
|
for _, r := range out {
|
||||||
|
if r["type"] == "access" {
|
||||||
|
paths = append(paths, r["path"].(string))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if strings.Join(paths, ",") != "/storage/media/series,/services/media/downloads" {
|
||||||
|
t.Fatalf("placed one, defaulted the other: got %v", paths)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// A reference to an access the definition does not declare is refused where the author is.
|
||||||
|
func TestAnUnknownAccessReferenceIsRefusedAtParse(t *testing.T) {
|
||||||
|
_, err := ParseManifest([]byte(`{
|
||||||
|
"module": "arr", "version": "1",
|
||||||
|
"accesses": [{"id": "series", "mode": "read-write"}],
|
||||||
|
"resources": [
|
||||||
|
{"id": "state", "type": "directory", "place": ".", "mode": "0700"},
|
||||||
|
{"id": "server", "type": "container", "name": "arr", "image": "arr:1",
|
||||||
|
"volumes": ["${access:movies}:/movies"]}
|
||||||
|
]}`))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "${access:movies}") {
|
||||||
|
t.Fatalf("a reference to an undeclared access was accepted: %v", err)
|
||||||
|
}
|
||||||
|
_, err = ParseManifest([]byte(`{"module": "arr", "version": "1", "accesses": [{"mode": "read"}]}`))
|
||||||
|
if err == nil || !strings.Contains(err.Error(), "neither an id nor a path") {
|
||||||
|
t.Fatalf("an access with no id and no path was accepted: %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -791,6 +791,9 @@ func checkResources(modules []Manifest) []string {
|
|||||||
// which is what lets the stack in 04-ISSUES/036 co-resolve.
|
// which is what lets the stack in 04-ISSUES/036 co-resolve.
|
||||||
for _, m := range modules {
|
for _, m := range modules {
|
||||||
for _, a := range m.Accesses {
|
for _, a := range m.Accesses {
|
||||||
|
if a.Path == "" {
|
||||||
|
continue // placed by the assignment; nothing to compare at registration
|
||||||
|
}
|
||||||
switch other := ownedPath[a.Path]; other {
|
switch other := ownedPath[a.Path]; other {
|
||||||
case "":
|
case "":
|
||||||
// Nobody owns it — the ordinary, correct case for shared data.
|
// Nobody owns it — the ordinary, correct case for shared data.
|
||||||
|
|||||||
@@ -80,8 +80,11 @@ var defaultSeats = []Seat{
|
|||||||
// A build is work submitted to this role and its outcome is the role's own event (ADR 0129).
|
// A build is work submitted to this role and its outcome is the role's own event (ADR 0129).
|
||||||
// One publish reaches whoever asked, the controller that records it, and the catalogue that
|
// One publish reaches whoever asked, the controller that records it, and the catalogue that
|
||||||
// places it in the graph — what the old bus's shared exchange did for free.
|
// places it in the graph — what the old bus's shared exchange did for free.
|
||||||
|
// A build says what it does as it does it (novox/hq ADR 0157): `started` when work is taken,
|
||||||
|
// `log.<build id>` for every line, `built` for the outcome. The log's tail token is the build's
|
||||||
|
// id, so a reader follows one build by subject alone.
|
||||||
{Name: "mesh-build-machine", Scope: ScopeMesh,
|
{Name: "mesh-build-machine", Scope: ScopeMesh,
|
||||||
Accepts: []string{"build"}, Emits: []string{"built"}, Decision: "novox/hq ADR 0121"},
|
Accepts: []string{"build"}, Emits: []string{"started", "built", "log.*"}, Decision: "novox/hq ADR 0121"},
|
||||||
{Name: "node-dns-resolver", Scope: ScopeNode, Decision: "novox/hq ADR 0121"},
|
{Name: "node-dns-resolver", Scope: ScopeNode, Decision: "novox/hq ADR 0121"},
|
||||||
{Name: "node-intrusion-prevention", Scope: ScopeNode, Decision: "novox/hq ADR 0121"},
|
{Name: "node-intrusion-prevention", Scope: ScopeNode, Decision: "novox/hq ADR 0121"},
|
||||||
{Name: "node-packet-filter", Scope: ScopeNode, Decision: "novox/hq ADR 0121"},
|
{Name: "node-packet-filter", Scope: ScopeNode, Decision: "novox/hq ADR 0121"},
|
||||||
|
|||||||
@@ -91,19 +91,40 @@ func orNoSettings(layers []Layer) string {
|
|||||||
return "; set today: " + strings.Join(keys, ", ")
|
return "; set today: " + strings.Join(keys, ", ")
|
||||||
}
|
}
|
||||||
|
|
||||||
// settingKeysUsedBy is every key a module's files ask for, so a setting that lands in one is not
|
// settingKeysUsedBy is every key a module's files, contributions and served facts ask for, so a
|
||||||
// called stray.
|
// setting that lands in one is not called stray.
|
||||||
func settingKeysUsedBy(m Manifest) map[string]bool {
|
func settingKeysUsedBy(m Manifest) map[string]bool {
|
||||||
used := map[string]bool{}
|
used := map[string]bool{}
|
||||||
|
note := func(s string) {
|
||||||
|
for _, k := range settingsUsed(s) {
|
||||||
|
used[k] = true
|
||||||
|
}
|
||||||
|
}
|
||||||
for _, r := range m.Resources {
|
for _, r := range m.Resources {
|
||||||
if fmt.Sprint(r["type"]) != "file" {
|
if fmt.Sprint(r["type"]) != "file" {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
if content, ok := r["content"].(string); ok {
|
if content, ok := r["content"].(string); ok {
|
||||||
for _, k := range settingsUsed(content) {
|
note(content)
|
||||||
used[k] = true
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
inValues := func(values map[string]any) {
|
||||||
|
for _, v := range values {
|
||||||
|
if s, ok := v.(string); ok {
|
||||||
|
note(s)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, values := range m.Contributes {
|
||||||
|
inValues(values)
|
||||||
|
}
|
||||||
|
for _, locals := range m.ContributesMany {
|
||||||
|
for _, values := range locals {
|
||||||
|
inValues(values)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, values := range m.Serves {
|
||||||
|
inValues(values)
|
||||||
}
|
}
|
||||||
return used
|
return used
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -85,17 +85,67 @@ func ApplySettings(resource map[string]any, layers []Layer) (map[string]any, err
|
|||||||
return out, nil
|
return out, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// Settle lays settings over a module's own values. Exported for what a provider serves, which is
|
// Settle lays settings over what a provider serves. Exported because a served fact is settled where
|
||||||
// settled where the mesh is walked rather than where a node is declared.
|
// the mesh is walked rather than where a node is declared.
|
||||||
|
//
|
||||||
|
// **A setting overrides a served key; it never adds one** (novox/hq 04-ISSUES/173). What a consumer
|
||||||
|
// is told is the provider's contract, and a setting made for one of the provider's files — a site
|
||||||
|
// name, a public address — is not part of it. Before this, every setting of a module reached every
|
||||||
|
// consumer of every provision it served.
|
||||||
func Settle(base map[string]any, layers []Layer) (map[string]any, error) {
|
func Settle(base map[string]any, layers []Layer) (map[string]any, error) {
|
||||||
return settle(base, layers, nil, "what is served")
|
return overridden(base, layers, "what is served")
|
||||||
|
}
|
||||||
|
|
||||||
|
// overridden lays settings over a map whose keys are its contract: a contribution, a served fact.
|
||||||
|
// Only the keys the map already declares are touched; the rest of a layer is somebody else's
|
||||||
|
// business (a file's, another destination's) and is left to reach it there.
|
||||||
|
//
|
||||||
|
// A declared value may itself be the operator's, `${setting:<key>}` (ADR 0155): a mail provider
|
||||||
|
// serves its domain, an identity provider its issuer, and neither is the definition's to state.
|
||||||
|
// Filled from the layers after the overrides, and refused by name when nothing sets it — a literal
|
||||||
|
// placeholder handed to a consumer is a service configured against a string nobody meant.
|
||||||
|
func overridden(base map[string]any, layers []Layer, what string) (map[string]any, error) {
|
||||||
|
kept := make([]Layer, 0, len(layers))
|
||||||
|
for _, layer := range layers {
|
||||||
|
values := map[string]any{}
|
||||||
|
for key, value := range layer.Values {
|
||||||
|
if _, declared := base[key]; declared {
|
||||||
|
values[key] = value
|
||||||
|
}
|
||||||
|
}
|
||||||
|
kept = append(kept, Layer{From: layer.From, Values: values})
|
||||||
|
}
|
||||||
|
merged, err := settle(base, kept, nil, what)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
for key, value := range merged {
|
||||||
|
s, ok := value.(string)
|
||||||
|
if !ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
for _, asked := range settingsUsed(s) {
|
||||||
|
v, set := settingValue(layers, asked)
|
||||||
|
if !set {
|
||||||
|
return nil, fmt.Errorf(
|
||||||
|
"%s says ${setting:%s} for %q, and nothing sets %q — an operator's value is the "+
|
||||||
|
"assignment's, never the definition's (novox/hq ADR 0112)%s",
|
||||||
|
what, asked, key, asked, orNoSettings(layers))
|
||||||
|
}
|
||||||
|
s = strings.ReplaceAll(s, "${setting:"+asked+"}", plainly(v))
|
||||||
|
}
|
||||||
|
merged[key] = s
|
||||||
|
}
|
||||||
|
return merged, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// settle lays the layers over a module's own values, in order.
|
// settle lays the layers over a module's own values, in order.
|
||||||
//
|
//
|
||||||
// Shared by a file's content and a module's contributions, because they are the same act: the
|
// Shared by a file's content and a module's contributions, because they are the same act: the
|
||||||
// module says what it means by default, and somebody says what it means here. A contribution that
|
// module says what it means by default, and somebody says what it means here. A contribution that
|
||||||
// could not be settled would have to be edited to be reused anywhere else.
|
// could not be settled would have to be edited to be reused anywhere else. The two differ in one
|
||||||
|
// respect, and the caller decides it: a file takes keys it did not declare (a setting may add to a
|
||||||
|
// configuration), a contribution or served fact does not (overridden).
|
||||||
func settle(base map[string]any, layers []Layer, protected map[string]bool, what string) (
|
func settle(base map[string]any, layers []Layer, protected map[string]bool, what string) (
|
||||||
map[string]any, error) {
|
map[string]any, error) {
|
||||||
merged := deepCopy(base)
|
merged := deepCopy(base)
|
||||||
@@ -149,23 +199,22 @@ func deepCopy(in map[string]any) map[string]any {
|
|||||||
return out
|
return out
|
||||||
}
|
}
|
||||||
|
|
||||||
// UnusedSettings names settings that reached no file.
|
// UnusedSettings names settings that reach nothing.
|
||||||
//
|
//
|
||||||
// Somebody who sets a key on a module with nothing mergeable, or misspells one, has changed
|
// Somebody who sets a key on a module with nothing mergeable, or misspells one, has changed
|
||||||
// nothing — and would find out by the machine not behaving differently, which is the slowest
|
// nothing — and would find out by the machine not behaving differently, which is the slowest
|
||||||
// way there is. This is what makes that visible at the moment they set it.
|
// way there is. This is what makes that visible at the moment they set it.
|
||||||
|
//
|
||||||
|
// Where a key can land: any mergeable file takes any key; a file asking for `${setting:<key>}`
|
||||||
|
// takes that key (ADR 0155); a contribution or a served fact takes a key it declares, and no other
|
||||||
|
// (novox/hq 04-ISSUES/173); and the mesh's own words — `expose`, `ports`, `reach`, `endpoints` —
|
||||||
|
// are read by the mesh. A key none of those takes is stray, and is said so rather than dropped.
|
||||||
func UnusedSettings(m Manifest, layers []Layer) []string {
|
func UnusedSettings(m Manifest, layers []Layer) []string {
|
||||||
for _, r := range m.Resources {
|
for _, r := range m.Resources {
|
||||||
if how, _ := r["merge"].(string); how != "" {
|
if how, _ := r["merge"].(string); how != "" {
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
// A contribution is a destination too. A route's hostname is exactly the kind of thing that
|
|
||||||
// differs between one mesh and the next, and calling it stray would refuse the one setting
|
|
||||||
// most modules that publish anything will have.
|
|
||||||
if len(m.Contributes) > 0 {
|
|
||||||
return nil
|
|
||||||
}
|
|
||||||
// A computed module has no resources here to look at — they are worked out per node, and
|
// A computed module has no resources here to look at — they are worked out per node, and
|
||||||
// whether a setting lands is not knowable until then. Silence rather than a wrong answer:
|
// whether a setting lands is not knowable until then. Silence rather than a wrong answer:
|
||||||
// claiming every setting on the private network is stray would be worse than saying nothing.
|
// claiming every setting on the private network is stray would be worse than saying nothing.
|
||||||
@@ -173,12 +222,28 @@ func UnusedSettings(m Manifest, layers []Layer) []string {
|
|||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// A key a file's content asks for with ${setting:<key>} is a destination too (ADR 0155).
|
lands := settingKeysUsedBy(m)
|
||||||
asked := settingKeysUsedBy(m)
|
for _, values := range m.Contributes {
|
||||||
|
for key := range values {
|
||||||
|
lands[key] = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, locals := range m.ContributesMany {
|
||||||
|
for _, values := range locals {
|
||||||
|
for key := range values {
|
||||||
|
lands[key] = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, served := range m.Serves {
|
||||||
|
for key := range served {
|
||||||
|
lands[key] = true
|
||||||
|
}
|
||||||
|
}
|
||||||
var unused []string
|
var unused []string
|
||||||
for _, layer := range layers {
|
for _, layer := range layers {
|
||||||
for key := range layer.Values {
|
for key := range layer.Values {
|
||||||
if asked[key] {
|
if lands[key] {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
// `expose` is a real destination for a module that listens: it overrides a port's
|
// `expose` is a real destination for a module that listens: it overrides a port's
|
||||||
@@ -202,9 +267,18 @@ func UnusedSettings(m Manifest, layers []Layer) []string {
|
|||||||
if key == EndpointsSetting && len(m.Listens) > 0 {
|
if key == EndpointsSetting && len(m.Listens) > 0 {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
// `places` puts a declared directory where this machine keeps it, `accesses` says where
|
||||||
|
// the operator's data is (novox/hq issue 153). Validated in Places and AccessPlaces.
|
||||||
|
if key == PlacesSetting && len(directoriesOf(m)) > 0 {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if key == AccessesSetting && len(m.Accesses) > 0 {
|
||||||
|
continue
|
||||||
|
}
|
||||||
unused = append(unused, fmt.Sprintf(
|
unused = append(unused, fmt.Sprintf(
|
||||||
"%s sets %q, and %s has no file or contribution to merge it into",
|
"%s sets %q, and %s has no file that merges it, asks for no ${setting:%s}, and "+
|
||||||
layer.From, key, m.Module))
|
"declares no %q in what it contributes or serves",
|
||||||
|
layer.From, key, m.Module, key, key))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
sort.Strings(unused)
|
sort.Strings(unused)
|
||||||
|
|||||||
@@ -167,11 +167,45 @@ func TestSettingsThatReachNothingAreNamed(t *testing.T) {
|
|||||||
{"id": "conf", "type": "file", "path": "/etc/thing", "content": "plain"},
|
{"id": "conf", "type": "file", "path": "/etc/thing", "content": "plain"},
|
||||||
}}
|
}}
|
||||||
unused := UnusedSettings(m, []Layer{{From: "node", Values: map[string]any{"port": 1}}})
|
unused := UnusedSettings(m, []Layer{{From: "node", Values: map[string]any{"port": 1}}})
|
||||||
if len(unused) != 1 || !strings.Contains(unused[0], "no file or contribution to merge it into") {
|
if len(unused) != 1 || !strings.Contains(unused[0], "no file that merges it") {
|
||||||
t.Errorf("settings that reached nothing were not named: %v", unused)
|
t.Errorf("settings that reached nothing were not named: %v", unused)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestASettingLandsOnlyWhereSomethingDeclaresIt(t *testing.T) {
|
||||||
|
// novox/hq 04-ISSUES/173. A module that contributes a route and serves a provision takes a
|
||||||
|
// setting for a key either declares, and a setting for a key neither declares is stray — it
|
||||||
|
// would not reach the route or the served fact, so it must be said rather than dropped.
|
||||||
|
m := Manifest{Module: "mail",
|
||||||
|
Contributes: map[string]map[string]any{"reverse-proxy": {"host": "mail", "port": 8080}},
|
||||||
|
Serves: map[string]map[string]any{"smtp": {"host": "mail", "port": 25}},
|
||||||
|
Resources: []map[string]any{
|
||||||
|
{"id": "env", "type": "file", "path": "/etc/mail.env", "content": "SITE=${setting:sitename}\n"},
|
||||||
|
}}
|
||||||
|
layers := []Layer{{From: "the mesh", Values: map[string]any{
|
||||||
|
"host": "post", "sitename": "Mail", "website": "https://www.example.tld"}}}
|
||||||
|
unused := UnusedSettings(m, layers)
|
||||||
|
if len(unused) != 1 || !strings.Contains(unused[0], `"website"`) {
|
||||||
|
t.Errorf("only website reaches nothing; named: %v", unused)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestASettingOverridesAServedKeyAndAddsNone(t *testing.T) {
|
||||||
|
// What a consumer is told is the provider's contract. A setting made for one of the
|
||||||
|
// provider's files — its site name, its public address — is not part of it.
|
||||||
|
served, err := Settle(map[string]any{"host": "mail", "port": 25},
|
||||||
|
[]Layer{{From: "the mesh", Values: map[string]any{"host": "post", "sitename": "Mail"}}})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if served["host"] != "post" {
|
||||||
|
t.Errorf("the setting did not override the served host: %v", served)
|
||||||
|
}
|
||||||
|
if _, leaked := served["sitename"]; leaked {
|
||||||
|
t.Errorf("a setting for a file reached the consumers: %v", served)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestContentThatIsNotJSONIsRefusedWhereSomebodyIsLooking(t *testing.T) {
|
func TestContentThatIsNotJSONIsRefusedWhereSomebodyIsLooking(t *testing.T) {
|
||||||
// Rather than on the machine, at apply time, as a file the program cannot read.
|
// Rather than on the machine, at apply time, as a file the program cannot read.
|
||||||
_, err := ApplySettings(file(`this is not json`), nil)
|
_, err := ApplySettings(file(`this is not json`), nil)
|
||||||
@@ -179,3 +213,24 @@ func TestContentThatIsNotJSONIsRefusedWhereSomebodyIsLooking(t *testing.T) {
|
|||||||
t.Fatal("a module claiming to merge as JSON shipped something else and was accepted")
|
t.Fatal("a module claiming to merge as JSON shipped something else and was accepted")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestAServedValueMayBeTheOperators(t *testing.T) {
|
||||||
|
// A mail provider serves its domain and an identity provider its issuer; neither is the
|
||||||
|
// definition's to state (ADR 0155). Filled from the layers, refused by name when unset.
|
||||||
|
served, err := Settle(map[string]any{"port": 587, "domain": "${setting:domain}"},
|
||||||
|
[]Layer{{From: "the mesh", Values: map[string]any{"domain": "example.tld"}}})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if served["domain"] != "example.tld" {
|
||||||
|
t.Errorf("the operator's value did not fill the served key: %v", served)
|
||||||
|
}
|
||||||
|
_, err = Settle(map[string]any{"domain": "${setting:domain}"}, nil)
|
||||||
|
if err == nil || !strings.Contains(err.Error(), `"domain"`) {
|
||||||
|
t.Errorf("a served value nothing sets must be refused by name; got %v", err)
|
||||||
|
}
|
||||||
|
m := Manifest{Module: "mail", Serves: map[string]map[string]any{"smtp": {"domain": "${setting:domain}"}}}
|
||||||
|
if unused := UnusedSettings(m, []Layer{{From: "the mesh", Values: map[string]any{"domain": "x"}}}); len(unused) != 0 {
|
||||||
|
t.Errorf("a setting a served fact asks for is not stray: %v", unused)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -85,8 +85,12 @@ var ControllerVerbs = []Verb{
|
|||||||
Input: schema(nil, nil)},
|
Input: schema(nil, nil)},
|
||||||
{Name: "seats", Description: "Every seat the mesh defines, what it delivers, and who holds it.",
|
{Name: "seats", Description: "Every seat the mesh defines, what it delivers, and who holds it.",
|
||||||
Input: schema(nil, nil)},
|
Input: schema(nil, nil)},
|
||||||
{Name: "builds", Description: "What has been built lately and what came of it, for every module or for one.",
|
{Name: "builds", Description: "What has been built lately and what came of it, for every module or for one; " +
|
||||||
Input: schema(map[string]string{"module": "one module's name; every module when absent"}, nil)},
|
"or, given a build's id, everything the build machine said while building it, line by line, from the bus.",
|
||||||
|
Input: schema(map[string]string{
|
||||||
|
"module": "one module's name; every module when absent",
|
||||||
|
"log": "a build's id (as `builds` lists it): print what the build machine said, line by line",
|
||||||
|
}, nil)},
|
||||||
{Name: "plan", Description: "What one machine would run, and why: the declaration the mesh would send it.",
|
{Name: "plan", Description: "What one machine would run, and why: the declaration the mesh would send it.",
|
||||||
Input: schema(map[string]string{"node": "the machine's name"}, []string{"node"})},
|
Input: schema(map[string]string{"node": "the machine's name"}, []string{"node"})},
|
||||||
{Name: "assign", Description: "Put a module on a machine. Refused with the mesh's own words when it cannot resolve there.",
|
{Name: "assign", Description: "Put a module on a machine. Refused with the mesh's own words when it cannot resolve there.",
|
||||||
@@ -95,9 +99,10 @@ var ControllerVerbs = []Verb{
|
|||||||
Input: schema(map[string]string{"node": "the machine's name", "module": "the module's name"}, []string{"node", "module"})},
|
Input: schema(map[string]string{"node": "the machine's name", "module": "the module's name"}, []string{"node", "module"})},
|
||||||
{Name: "push", Description: "Send a machine everything it should be — or every machine that is behind, when no machine is named.",
|
{Name: "push", Description: "Send a machine everything it should be — or every machine that is behind, when no machine is named.",
|
||||||
Input: schema(map[string]string{"node": "the machine's name; every machine behind when absent"}, nil)},
|
Input: schema(map[string]string{"node": "the machine's name; every machine behind when absent"}, nil)},
|
||||||
{Name: "build", Description: "Have the build machine build a repository and record what came out.",
|
{Name: "build", Description: "Have the build machine build a repository. Answers at once with the build's id: " +
|
||||||
|
"`builds` with that id follows it line by line, and the module is registered when the outcome comes.",
|
||||||
Input: schema(map[string]string{
|
Input: schema(map[string]string{
|
||||||
"repository": "the repository's URL, or its path on the forge holding the git seat",
|
"repository": "the repository's URL, or its path on the forge holding the git seat (owner/name)",
|
||||||
"path": "the module's directory inside it (optional)",
|
"path": "the module's directory inside it (optional)",
|
||||||
"ref": "the branch, tag or commit to build (optional)",
|
"ref": "the branch, tag or commit to build (optional)",
|
||||||
}, []string{"repository"})},
|
}, []string{"repository"})},
|
||||||
|
|||||||
+14
-1
@@ -5,7 +5,20 @@
|
|||||||
-- than for the mesh; ADR 0121 decided the rename and deferred it because a delivering seat that stops
|
-- than for the mesh; ADR 0121 decided the rename and deferred it because a delivering seat that stops
|
||||||
-- resolving mid-flight takes a provision away from every consumer. ADR 0122 removed that risk: a seat's
|
-- resolving mid-flight takes a provision away from every consumer. ADR 0122 removed that risk: a seat's
|
||||||
-- former name is an alias that resolves to it forever, a held record follows the rename by cascade, and
|
-- former name is an alias that resolves to it forever, a held record follows the rename by cascade, and
|
||||||
-- a claim written with the old name still holds. So the rename is one update and one alias.
|
-- a claim written with the old name still holds.
|
||||||
|
--
|
||||||
|
-- **Both rows may exist when this runs.** A controller whose compiled defaults already carry the new
|
||||||
|
-- name seeds it as a new seat the moment it can, and on the mesh this was written for that happened
|
||||||
|
-- before the rename: the first form of this migration renamed into a duplicate key and the control
|
||||||
|
-- node's prepare failed on every attempt (2026-09-30). So: if the new row is already there, the old
|
||||||
|
-- row's holding moves to it and the old row goes; otherwise the old row is renamed. Either way the old
|
||||||
|
-- name becomes an alias.
|
||||||
|
update seat_holding set seat = 'mesh-artifact-store'
|
||||||
|
where seat = 'the-artifact-store'
|
||||||
|
and exists (select 1 from seat where name = 'mesh-artifact-store');
|
||||||
|
delete from seat
|
||||||
|
where name = 'the-artifact-store'
|
||||||
|
and exists (select 1 from seat where name = 'mesh-artifact-store');
|
||||||
update seat set name = 'mesh-artifact-store' where name = 'the-artifact-store';
|
update seat set name = 'mesh-artifact-store' where name = 'the-artifact-store';
|
||||||
insert into seat_alias (alias, seat) values ('the-artifact-store', 'mesh-artifact-store')
|
insert into seat_alias (alias, seat) values ('the-artifact-store', 'mesh-artifact-store')
|
||||||
on conflict (alias) do update set seat = excluded.seat;
|
on conflict (alias) do update set seat = excluded.seat;
|
||||||
|
|||||||
@@ -49,6 +49,17 @@ type BuildRequest struct {
|
|||||||
// which forge holds the seat here. A builder handed no base for a seat a context names refuses
|
// which forge holds the seat here. A builder handed no base for a seat a context names refuses
|
||||||
// the build and says so.
|
// the build and says so.
|
||||||
Seats map[string]string `json:"seats,omitempty"`
|
Seats map[string]string `json:"seats,omitempty"`
|
||||||
|
// Source is the repository as the mesh records it when it lives on a seat's holder — the seat
|
||||||
|
// and the path on it, never the URL just composed (novox/hq ADR 0111). Carried with the
|
||||||
|
// asking and echoed in the outcome, so whoever hears the outcome can register the module with
|
||||||
|
// its true source, whether or not they were the one who asked (novox/hq issue 176).
|
||||||
|
Source *SourceOnSeat `json:"source,omitempty"`
|
||||||
|
}
|
||||||
|
|
||||||
|
// SourceOnSeat names a repository by the seat whose holder serves it and its path there.
|
||||||
|
type SourceOnSeat struct {
|
||||||
|
Seat string `json:"seat"`
|
||||||
|
Repository string `json:"repository"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// BuildResult is what a builder says back.
|
// BuildResult is what a builder says back.
|
||||||
@@ -101,6 +112,9 @@ type BuildResult struct {
|
|||||||
|
|
||||||
// Failed is why, when it did.
|
// Failed is why, when it did.
|
||||||
Failed string `json:"failed,omitempty"`
|
Failed string `json:"failed,omitempty"`
|
||||||
|
|
||||||
|
// Source is the request's, echoed: the seat form of the repository, for whoever registers.
|
||||||
|
Source *SourceOnSeat `json:"source,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// ReadRepository is a repository a build read source from besides the module's own, at the branch,
|
// ReadRepository is a repository a build read source from besides the module's own, at the branch,
|
||||||
|
|||||||
@@ -27,6 +27,40 @@ const TheBuildMachine = "mesh-build-machine"
|
|||||||
func BuildWork() string { return "mesh.seat." + TheBuildMachine + ".accept.build" }
|
func BuildWork() string { return "mesh.seat." + TheBuildMachine + ".accept.build" }
|
||||||
func BuildOutcome() string { return "mesh.seat." + TheBuildMachine + ".event.built" }
|
func BuildOutcome() string { return "mesh.seat." + TheBuildMachine + ".event.built" }
|
||||||
|
|
||||||
|
// BuildStarted is where a build machine says it has taken a build, and BuildLog is where it says
|
||||||
|
// what it is doing, one line per message, under the build's own id (novox/hq ADR 0157).
|
||||||
|
//
|
||||||
|
// **The whole build is on the bus as it happens.** The outcome alone told a person that a build
|
||||||
|
// failed and its first line why; everything between — which command, how long, where it hung —
|
||||||
|
// lived in one container's stderr on one machine. Every line is now an event of the role, retained
|
||||||
|
// with the rest of the mesh's events, so a reader follows a build live by subscribing its subject,
|
||||||
|
// or reads it back afterwards from the stream, and a viewer is a subscriber and nothing more.
|
||||||
|
func BuildStarted() string { return "mesh.seat." + TheBuildMachine + ".event.started" }
|
||||||
|
func BuildLog(id string) string { return "mesh.seat." + TheBuildMachine + ".event.log." + id }
|
||||||
|
|
||||||
|
// BuildStart is what a build machine says the moment it takes a build.
|
||||||
|
type BuildStart struct {
|
||||||
|
ID string `json:"id"`
|
||||||
|
Repository string `json:"repository"`
|
||||||
|
Path string `json:"path,omitempty"`
|
||||||
|
Ref string `json:"ref,omitempty"`
|
||||||
|
On string `json:"on"`
|
||||||
|
At string `json:"at"`
|
||||||
|
}
|
||||||
|
|
||||||
|
// BuildLine is one thing a build said while building.
|
||||||
|
type BuildLine struct {
|
||||||
|
ID string `json:"id"`
|
||||||
|
// Seq counts the lines of one build from 1, so a reader that joined late or read two copies
|
||||||
|
// can order them and see a gap.
|
||||||
|
Seq int `json:"seq"`
|
||||||
|
At string `json:"at"`
|
||||||
|
// Step is which part of the build spoke — clone, context, image, run, failed — and Message is
|
||||||
|
// what it said, as the builder's own log prints it.
|
||||||
|
Step string `json:"step"`
|
||||||
|
Message string `json:"message"`
|
||||||
|
}
|
||||||
|
|
||||||
// KeyRoleBuilt is the build outcome under the role's name, on the bus the mesh runs on today.
|
// KeyRoleBuilt is the build outcome under the role's name, on the bus the mesh runs on today.
|
||||||
//
|
//
|
||||||
// The same event as KeyModuleBuilt and published beside it, because a catalogue installed before this
|
// The same event as KeyModuleBuilt and published beside it, because a catalogue installed before this
|
||||||
@@ -44,6 +78,12 @@ type Builders interface {
|
|||||||
// need different remedies, which is why the message distinguishes them.
|
// need different remedies, which is why the message distinguishes them.
|
||||||
Submit(ctx context.Context, request BuildRequest, wait time.Duration) (BuildResult, error)
|
Submit(ctx context.Context, request BuildRequest, wait time.Duration) (BuildResult, error)
|
||||||
|
|
||||||
|
// Ask submits one build and does not wait: the outcome is the role's event, heard and taken in
|
||||||
|
// by the controller whether or not anybody waited (novox/hq issue 176). For a caller that
|
||||||
|
// cannot hold a connection for the minutes a build takes — a tool call — and follows the build
|
||||||
|
// by its id instead.
|
||||||
|
Ask(ctx context.Context, request BuildRequest) error
|
||||||
|
|
||||||
// Close lets go of whatever was dialled.
|
// Close lets go of whatever was dialled.
|
||||||
Close()
|
Close()
|
||||||
}
|
}
|
||||||
@@ -57,6 +97,13 @@ type BuildMachine interface {
|
|||||||
|
|
||||||
// Build is one request a machine has been handed.
|
// Build is one request a machine has been handed.
|
||||||
type Build interface {
|
type Build interface {
|
||||||
|
// Began says the build has been taken and is under way, before anything runs.
|
||||||
|
Began(ctx context.Context) error
|
||||||
|
|
||||||
|
// Say publishes one line of what the build is doing. Never fails the build: a line the bus
|
||||||
|
// did not take is a line lost, and the outcome still comes.
|
||||||
|
Say(step, message string)
|
||||||
|
|
||||||
// Request is what to build.
|
// Request is what to build.
|
||||||
Request() BuildRequest
|
Request() BuildRequest
|
||||||
|
|
||||||
|
|||||||
@@ -43,6 +43,20 @@ func (b *natsBuilds) Close() {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Ask publishes the work and returns; see Builders.
|
||||||
|
func (b *natsBuilds) Ask(ctx context.Context, request BuildRequest) error {
|
||||||
|
body, err := json.Marshal(request)
|
||||||
|
if err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
publish, cancel := context.WithTimeout(ctx, 30*time.Second)
|
||||||
|
defer cancel()
|
||||||
|
if _, err := b.js.Context().Publish(BuildWork(), body, nats.Context(publish)); err != nil {
|
||||||
|
return fmt.Errorf("cannot submit a build: %w", err)
|
||||||
|
}
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
func (b *natsBuilds) Submit(ctx context.Context, request BuildRequest,
|
func (b *natsBuilds) Submit(ctx context.Context, request BuildRequest,
|
||||||
wait time.Duration) (BuildResult, error) {
|
wait time.Duration) (BuildResult, error) {
|
||||||
|
|
||||||
@@ -169,6 +183,7 @@ type natsBuild struct {
|
|||||||
msg *nats.Msg
|
msg *nats.Msg
|
||||||
on string
|
on string
|
||||||
js *broker.JetStream
|
js *broker.JetStream
|
||||||
|
seq int
|
||||||
}
|
}
|
||||||
|
|
||||||
func (b *natsBuild) Request() BuildRequest { return b.request }
|
func (b *natsBuild) Request() BuildRequest { return b.request }
|
||||||
@@ -203,4 +218,37 @@ func (b *natsBuild) Announce(ctx context.Context, result BuildResult) error {
|
|||||||
|
|
||||||
func (b *natsBuild) Done() error { return b.msg.Ack() }
|
func (b *natsBuild) Done() error { return b.msg.Ack() }
|
||||||
|
|
||||||
|
// Began publishes that this machine has taken the build, into the stream like the outcome, so a
|
||||||
|
// reader that asks afterwards sees when it started as well as how it ended.
|
||||||
|
func (b *natsBuild) Began(ctx context.Context) error {
|
||||||
|
body, err := json.Marshal(BuildStart{
|
||||||
|
ID: b.request.ID, Repository: b.request.Repository, Path: b.request.Path,
|
||||||
|
Ref: b.request.Ref, On: b.on, At: time.Now().UTC().Format(time.RFC3339Nano),
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
publish, cancel := context.WithTimeout(ctx, 30*time.Second)
|
||||||
|
defer cancel()
|
||||||
|
if _, err := b.js.Context().Publish(BuildStarted(), body, nats.Context(publish)); err != nil {
|
||||||
|
return fmt.Errorf("cannot say a build started: %w", err)
|
||||||
|
}
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
|
// Say publishes one line under the build's id. Core publish, unawaited: the stream that holds the
|
||||||
|
// role's events captures it on its way through, and a build must not slow to the pace of an ack
|
||||||
|
// per line. A line the bus did not take is counted anyway, so the gap is visible to a reader.
|
||||||
|
func (b *natsBuild) Say(step, message string) {
|
||||||
|
b.seq++
|
||||||
|
body, err := json.Marshal(BuildLine{
|
||||||
|
ID: b.request.ID, Seq: b.seq, At: time.Now().UTC().Format(time.RFC3339Nano),
|
||||||
|
Step: step, Message: message,
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
_ = b.js.Conn().Publish(BuildLog(b.request.ID), body)
|
||||||
|
}
|
||||||
|
|
||||||
func (b *natsBuild) Hold(after time.Duration) error { return b.msg.NakWithDelay(after) }
|
func (b *natsBuild) Hold(after time.Duration) error { return b.msg.NakWithDelay(after) }
|
||||||
|
|||||||
@@ -79,6 +79,14 @@ func TestNatsABuildIsTakenAndItsOutcomeReachesEverybody(t *testing.T) {
|
|||||||
defer func() { _ = watching.Unsubscribe() }()
|
defer func() { _ = watching.Unsubscribe() }()
|
||||||
_ = js.Conn().Flush()
|
_ = js.Conn().Flush()
|
||||||
|
|
||||||
|
// And a reader following this one build by its subject alone (novox/hq ADR 0157).
|
||||||
|
lines, err := js.Conn().SubscribeSync(BuildLog("b-1"))
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
defer func() { _ = lines.Unsubscribe() }()
|
||||||
|
_ = js.Conn().Flush()
|
||||||
|
|
||||||
// A build machine holding the role.
|
// A build machine holding the role.
|
||||||
machine := MachineOverNATS(js, "anchor")
|
machine := MachineOverNATS(js, "anchor")
|
||||||
defer machine.Close()
|
defer machine.Close()
|
||||||
@@ -86,6 +94,9 @@ func TestNatsABuildIsTakenAndItsOutcomeReachesEverybody(t *testing.T) {
|
|||||||
go func() {
|
go func() {
|
||||||
failed <- machine.Take(ctx, func(ctx context.Context, work Build) {
|
failed <- machine.Take(ctx, func(ctx context.Context, work Build) {
|
||||||
r := work.Request()
|
r := work.Request()
|
||||||
|
_ = work.Began(ctx)
|
||||||
|
work.Say("clone", "cloning /r")
|
||||||
|
work.Say("image", "building shop")
|
||||||
_ = work.Announce(ctx, BuildResult{
|
_ = work.Announce(ctx, BuildResult{
|
||||||
ID: r.ID, Repository: r.Repository, On: "anchor", Commit: "abc1234",
|
ID: r.ID, Repository: r.Repository, On: "anchor", Commit: "abc1234",
|
||||||
Manifest: json.RawMessage(`{"module":"shop"}`),
|
Manifest: json.RawMessage(`{"module":"shop"}`),
|
||||||
@@ -104,6 +115,27 @@ func TestNatsABuildIsTakenAndItsOutcomeReachesEverybody(t *testing.T) {
|
|||||||
}
|
}
|
||||||
t.Fatalf("the asker never got an outcome: %v", err)
|
t.Fatalf("the asker never got an outcome: %v", err)
|
||||||
}
|
}
|
||||||
|
// The reader heard the build as it went, in order, under its id.
|
||||||
|
for want := 1; want <= 2; want++ {
|
||||||
|
msg, err := lines.NextMsg(5 * time.Second)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("line %d of the build never reached its subject: %v", want, err)
|
||||||
|
}
|
||||||
|
var line BuildLine
|
||||||
|
if err := json.Unmarshal(msg.Data, &line); err != nil || line.ID != "b-1" || line.Seq != want {
|
||||||
|
t.Fatalf("line %d came back as %s (%v)", want, msg.Data, err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
// And it is in the stream for a reader who comes later.
|
||||||
|
info, err := js.Context().StreamInfo(broker.EventsStream, &nats.StreamInfoRequest{SubjectsFilter: BuildLog("b-1")})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if info.State.Subjects[BuildLog("b-1")] != 2 {
|
||||||
|
t.Fatalf("the stream holds %v under the build's subject, want 2", info.State.Subjects)
|
||||||
|
}
|
||||||
|
if err == nil {
|
||||||
|
}
|
||||||
if result.ID != "b-1" || result.Commit != "abc1234" {
|
if result.ID != "b-1" || result.Commit != "abc1234" {
|
||||||
t.Fatalf("the asker got %+v", result)
|
t.Fatalf("the asker got %+v", result)
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user