Issue 091: a module definition carries a machine port #78

Merged
jschoubben merged 1 commits from issue/091-machine-ports-in-manifests into main 2026-09-23 23:38:40 +00:00
Owner

Measured while planning the control-node's migration: 14 of 46 modules with containers fix the machine side of a published port in the definition, 32 leave it to the node.

Two kinds are defensible — the foundation's own ports, which are rewritten at genesis and are now per-node settings, and ports a protocol fixes by number. The rest are a predecessor's host ports copied in during conversion: facts about one machine written into a definition meant for any machine.

Not fatal, since a node's ports setting overrides the mapping's machine side, and for this migration the copied numbers happen to be the right ones. But the default is in the wrong place and nothing says so when a module is written. The conversion backlog is where this gets decided for the modules still to be written, which is why it is worth answering now.

Measured while planning the control-node's migration: 14 of 46 modules with containers fix the machine side of a published port in the definition, 32 leave it to the node. Two kinds are defensible — the foundation's own ports, which are rewritten at genesis and are now per-node settings, and ports a protocol fixes by number. The rest are a predecessor's host ports copied in during conversion: facts about one machine written into a definition meant for any machine. Not fatal, since a node's `ports` setting overrides the mapping's machine side, and for this migration the copied numbers happen to be the right ones. But the default is in the wrong place and nothing says so when a module is written. The conversion backlog is where this gets decided for the modules still to be written, which is why it is worth answering now.
jschoubben added 1 commit 2026-09-22 20:26:22 +00:00
jschoubben merged commit 60e43f9446 into main 2026-09-23 23:38:40 +00:00
jschoubben deleted branch issue/091-machine-ports-in-manifests 2026-09-23 23:38:40 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/hq#78