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

Closed
jschoubben wants to merge 0 commits from issue/091-machine-ports-in-manifests into main
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
mesh-admin closed this pull request 2026-10-07 23:24:30 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/hq#78