novox/mesh-controller
3
0
Fork 0
Code Issues Pull Requests 1 Releases Wiki Activity

A repeat assignment says nothing changed (ADR 0115) #72

Merged
jschoubben merged 1 commits from feat/a-second-assignment-says-so into main 2026-09-26 17:02:49 +00:00
Conversation 0 Commits 1 Files Changed 9
+51 -25
jschoubben commented 2026-09-26 17:02:43 +00:00
Owner
Copy Link
Copy Source

One assignment of a module per node is now the rule, not a limitation — the operator dropped the multi-assignment requirement (hq ADR 0115, #133), and the schema's (node, module) key has been the decision since migration 0005. What changes here: Assign reports whether the assignment was new, and the command says "already runs — one node runs one of each (ADR 0115); nothing changed" instead of printing "is assigned" for a no-op, which read as an action that happened. Idempotence stays: a repeat is exit 0, because a script stating what is already true is not wrong. The existing twice-is-not-an-error test now also pins the audibility.

One assignment of a module per node is now the rule, not a limitation — the operator dropped the multi-assignment requirement (hq ADR 0115, #133), and the schema's `(node, module)` key has been the decision since migration 0005. What changes here: `Assign` reports whether the assignment was new, and the command says *"already runs — one node runs one of each (ADR 0115); nothing changed"* instead of printing "is assigned" for a no-op, which read as an action that happened. Idempotence stays: a repeat is exit 0, because a script stating what is already true is not wrong. The existing twice-is-not-an-error test now also pins the audibility.
jschoubben added 1 commit 2026-09-26 17:02:44 +00:00
A repeat assignment says nothing changed (ADR 0115) 50734095b8 
One assignment of a module per node is now the rule, not a limitation —
the operator dropped the multi-assignment requirement, and the schema's
(node, module) key has been the decision since migration 0005. What
changed: Assign reports whether the assignment was new, and the command
says 'already runs — one node runs one of each (ADR 0115); nothing
changed' instead of printing 'is assigned' for a no-op, which read as
an action that happened. Idempotence stays: a repeat is exit 0, because
a script stating what is already true is not wrong.
jschoubben merged commit ed08cc1adc into main 2026-09-26 17:02:49 +00:00
jschoubben deleted branch feat/a-second-assignment-says-so 2026-09-26 17:02:49 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
jschoubben miel.truyen
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/mesh-controller#72