Update architecture document for network topology #15
Labels
No labels
bug
duplicate
effort
high
effort
low
effort
medium
effort
minimal
enhancement
help wanted
invalid
pr
feedback
pr
reviewing
question
status
backlog
status
document
status
done
status
implement
status
pending
status
planning
status
reviewing
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
tepichord/milner#15
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Naina:
Overview
The
ARCHITECTURE.mdfile covers worker sizing, pod scaling, and pod topology but has no network topology section. This plan adds a Mermaid diagram showing the full cluster network (physical hosts, VMs, NAT bridge, Tailscale overlay) and an IP/port table documenting every application's address, port, and accessibility scope.Technical Approach
Add new section header
## 7. Network Topologyafter the existing## 6. Supporting Infrastructuresection.Insert a
flowchart TDMermaid diagram using nestedsubgraphblocks for boundaries.Add a brief prose paragraph (2 sentences max) above the diagram explaining dual-stack NAT and Tailscale TLS termination.
Add a footnote below the table clarifying NodePort auto-assignment and that planned services are target-architecture items.
Implementation Details
File to modify:
ARCHITECTURE.md## 7. Network Topologysection after section 6Source files referenced (read-only):
control-plane/main.tf— network namecontrol-plane-net, bridgecp-bridgecontrol-plane/control-plane-net.xsl— dual-stack ranges:10.5.0.0/24,2001:db8:b84b:5::/64hc-vault/example.yaml— NodePort values for Vault (32706, 30266, 32536)worker/vm-setup/create-worker-vm.sh— QEMU user-mode networking, MAC generationDependencies: None (documentation-only change).
Constraints:
flowchart TD(notgraph TD) — modern Mermaid keyword, supports subgraph features<br>separatorAGENTS.md)Out-of-scope notes:
architecture-betaMermaid type is still beta — usingflowchart TD+ subgraphs insteadAcceptance Criteria
ARCHITECTURE.mdcontains a## 7. Network Topologysection with a Mermaidflowchart TDdiagramTesting Plan
ARCHITECTURE.mdin a Mermaid-capable viewer (GitHub, VS Code with Mermaid extension) and verify the diagram renders without syntax errorsyamllintand Markdown linting checks against the file to ensure no formatting regressionsRelated Issues/PRs