Every caller composes through mesh-sdk's go/compose since issues 1386 to 1388; the controller's second copy of the composition is dead code. The parent of this commit isfb556d6b, whose try (build-1792493691274712799, 17m28s, 2026-10-20 11:12 UTC) ran the equality test green on the live facts: every machine was sent and compared both ways (ADR 0453, issue 1384). Deleted, 40 functions: in plan.go planFor, declarationWith, renderingFor, grantsFor, theRestOfTheMesh and their helpers (boundOfGrant, busCredentialIssued, carriedBuilds, carriedOn, composeBusUsers, engineJudgesRoot, engineReadsHealth, engineTakesHanded, takesHanded, givenUnderAnOldKey, issuedToHolder, keyFor, licencesFor, managerPublicKeyFor, movedProvider, providerMoved, providerModuleOf, sayLeftOut, seatsOn, wireBuilds, zonesInTheMesh); in network.go generators, whoResolves, namesInTheMesh, onThePrivateNetwork; composedAndValidated, adoptionFor, adoptionOf, engineOpensVaultBox, namedModules, localNetworksOf, keepRecorded, passSeatsOn, theMeshPassesSeatsOn, assignmentsOf. Choosing (Reading, Allocating, Foreseeing) is gone: coreBoxesFor and coreBoxesOf take a plain allocating flag. composeForPush no longer takes the private network it ignored. The seat pass of a send is performSeatPasses, over the passes the composition names. Measured: cmd/mesh-controller non-test code 67,643 -> 65,821 lines (-1,822), its tests 57,691 -> 57,286, with the integration part of the equality test removed here because it calls the deleted functions. Not the issue's 3,000: that counts all of plan.go (2,192 lines), of which the callers, the verbs, the perform step and the sends remain (sendable, recorded builds, the held-back and ready-node logic, the plan verb, the gate's trial); and the 37 functions a name walk lists as unreachable are called through interfaces and method sets and are kept. Tests dropped, each with the mesh-sdk go/compose test that holds its cases: - seat_pass_test.go TestACompositionWithTwoClaimantsRecordsNoneAndSaysSo, TestACompositionChangesNothingWhileTheHolderIsAssignedOrNobodyClaims, TestAHolderAssignedButNotOnTheShelfIsNotTakenForGone: rest_test.go TestNoSeatIsPassedForTwoClaimantsAnAssignedHolderOrNoClaimant, TestAHolderAssignedButNotOnTheShelfIsNotTakenForGone, TestASeatWhoseHolderIsAssignedNowhereGoesToItsOnlyClaimant. TestACompositionPassesTheSeatAndWritesOneHandActByTheController and TestALostPassWritesNoActAndAFailedOneLeavesTheRecord stay as tests of the send's step (performSeatPasses); TestAThrowawayMeshPassesTheSeatInMemoryAndWritesNothing stays as TestAThrowawayMeshWritesNoHandActForAPass. - seat_pass_test.go TestNoSeatIsPassedWhenAMachinesAssignmentsCouldNotBeRead: no counterpart, and none is needed: the adapter reads every machine's assignments for the Input and a machine whose read fails fails the Input, so no seat is passed on a partial read. - network_test.go TestOnlyAMachineOnThePrivateNetworkIsNamed: network_test.go TestANodeNotOnThePrivateNetworkGetsNoAddressAndNoNetwork. - held_back_test.go TestASendCarriesTheCurrentBuildsAndALeftOutModuleKeepsItsOwn: node_test.go TestAModuleLeftOutKeepsTheBuildItWasLastSent (and compose_sdk_test.go's table, now literal). - modules_wire_test.go TestTheBuildOfACarriedCommitIsTheAssignmentsOrTheOneBuiltFromIt: node_test.go TestTheBodyCarriesTheOrderTheInputGivesAndTheBuildsOnlyToAnEngineThatReadsThem. - compose_equal_integration_test.go: the old path it compares against is deleted; its last run is the parent's try. Rewritten against mesh-sdk, same cases: issued_test.go (needs now come from compose.Resolve: Needed.IssuedToHolder and ProviderMoved), module_health_test.go and merge_gate_handing tests (the contract in the Input), provisioner_account_integration_test.go (the composition's refusal), one_resolution_integration_test.go (sayFound). Every test of the send, perform, the adapter and the verbs is kept; the tests that used planFor, declarationWith, generators, renderingFor or grantsFor as setup use resolveNode, composeSend, composeQuestion, renderingOf and grantsOf. compose_callers_test.go now fails when any deleted name is declared or called again. Committed by copying the files of a staging branch (stage/r4b-1389 at198be8217c, built fromfb556d6bby patch commits and compared with the author's worktree), so that this commit's parent isfb556d6b.
116 lines
4.4 KiB
Go
116 lines
4.4 KiB
Go
//go:build integration
|
|
|
|
package main
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"git.novox.be/novox/mesh-sdk/go/catalogue"
|
|
"git.novox.be/novox/mesh-sdk/go/compose"
|
|
)
|
|
|
|
// An own secret issued outside the mesh that nobody gave is never made (novox/hq issue 401, ADR 0228). Seen on
|
|
// 2026-10-10: the send made a random value for the mounts module's smb-credentials and placed it, and the module read
|
|
// it as a credential given and wrong. The send now makes none, places none, and places a value only once a person
|
|
// gave one — and a value the mesh made for it before is not placed either.
|
|
|
|
// anAssistant is a module with a secret the mesh makes and one only an outside party issues.
|
|
func anAssistant() catalogue.Manifest {
|
|
return catalogue.Manifest{Module: "assistant", Version: "1",
|
|
OwnSecrets: catalogue.OwnSecrets{
|
|
"password": {Path: "/var/lib/mesh/assistant/password", Taken: catalogue.TakenAtStart},
|
|
"vendor-key": {Path: "/var/lib/mesh/assistant/vendor-key", Taken: catalogue.TakenAtStart, IssuedBy: catalogue.IssuedOutside},
|
|
},
|
|
Resources: []map[string]any{
|
|
{"id": "state", "type": "directory", "path": "/var/lib/mesh/assistant", "mode": "0700"},
|
|
}}
|
|
}
|
|
|
|
// sentTo composes what a push sends to a machine, on the send path.
|
|
func sentTo(t *testing.T, open *stores, node string) []map[string]any {
|
|
t.Helper()
|
|
plan, settings, err := resolveNode(t.Context(), open, node)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
declared, err := declarationFor(t.Context(), open, node, plan, settings)
|
|
if err != nil {
|
|
t.Fatalf("the send to %s did not compose: %v", node, err)
|
|
}
|
|
return declared.Resources
|
|
}
|
|
|
|
func placedAt(resources []map[string]any, path string) map[string]any {
|
|
for _, r := range resources {
|
|
if r["path"] == path && r["sealed"] != nil {
|
|
return r
|
|
}
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func TestTheSendMakesNoValueForAnOutsideSecretNobodyGave(t *testing.T) {
|
|
open := aMesh(t)
|
|
ctx := t.Context()
|
|
register(t, open, anAssistant())
|
|
if _, err := assign(ctx, open, "laptop", "assistant"); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
sent := sentTo(t, open, "laptop")
|
|
if placedAt(sent, "/var/lib/mesh/assistant/password") == nil {
|
|
t.Fatal("the secret the mesh makes was not placed")
|
|
}
|
|
if got := placedAt(sent, "/var/lib/mesh/assistant/vendor-key"); got != nil {
|
|
t.Fatalf("a secret nobody gave was placed: %v", got)
|
|
}
|
|
if _, held, err := open.inventory.ModuleSecretIfIssued(ctx, "laptop", "assistant", "vendor-key"); err != nil || held {
|
|
t.Fatalf("the send made a value for a secret issued outside the mesh (held %v, %v)", held, err)
|
|
}
|
|
|
|
// A value the mesh made before this rule is not one a person gave: it is not placed.
|
|
if _, err := open.inventory.SecretForModule(ctx, "laptop", "assistant", "vendor-key"); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if got := placedAt(sentTo(t, open, "laptop"), "/var/lib/mesh/assistant/vendor-key"); got != nil {
|
|
t.Fatalf("a value the mesh made for a secret issued outside was placed: %v", got)
|
|
}
|
|
|
|
// Given, it is placed.
|
|
if err := open.inventory.AcceptSecretForModule(ctx, "laptop", "assistant", "vendor-key", "the vendor's key"); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
given, ok, err := open.inventory.GivenOwnSecret(ctx, "laptop", "assistant", "vendor-key")
|
|
if err != nil || !ok || !given.Current {
|
|
t.Fatalf("the value given is not read as given: %+v %v %v", given, ok, err)
|
|
}
|
|
placed := placedAt(sentTo(t, open, "laptop"), "/var/lib/mesh/assistant/vendor-key")
|
|
if placed == nil || placed["sealed"] != given.Sealed {
|
|
t.Fatalf("the value given is placed as %v", placed)
|
|
}
|
|
}
|
|
|
|
// The read paths answer as the send does: a question about the machine composes without the secret nobody gave, and
|
|
// asking writes nothing.
|
|
func TestAQuestionComposesWithoutAnOutsideSecretNobodyGave(t *testing.T) {
|
|
open := aMesh(t)
|
|
ctx := t.Context()
|
|
register(t, open, anAssistant())
|
|
if _, err := assign(ctx, open, "laptop", "assistant"); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
pushedBy(t, open, "laptop") // the mesh's own secret is made by the send
|
|
for _, choosing := range []compose.Mode{compose.Reading, compose.Foreseeing} {
|
|
got, _, err := composeQuestion(ctx, open, "laptop", choosing, false)
|
|
if err != nil {
|
|
t.Fatalf("%v: %v", choosing, err)
|
|
}
|
|
if placedAt(got.Resources, "/var/lib/mesh/assistant/vendor-key") != nil || len(got.foreseen) != 0 {
|
|
t.Fatalf("%v: a secret nobody gave is composed (foreseen %v)", choosing, got.foreseen)
|
|
}
|
|
}
|
|
if _, held, err := open.inventory.ModuleSecretIfIssued(ctx, "laptop", "assistant", "vendor-key"); err != nil || held {
|
|
t.Fatalf("a question made a value (held %v, %v)", held, err)
|
|
}
|
|
}
|