Skip to content

docs: unify firewall port tables across snippets#138

Merged
noctarius merged 8 commits into
mainfrom
update-firewall-port-tables
Jun 8, 2026
Merged

docs: unify firewall port tables across snippets#138
noctarius merged 8 commits into
mainfrom
update-firewall-port-tables

Conversation

@schmidt-scaled
Copy link
Copy Markdown
Contributor

Summary

Replaces the contents of the four firewall port-table snippets with a single consolidated table:

  • snippets/storage-plane-network-port-table.md
  • snippets/storage-plane-network-port-table-k8s.md
  • snippets/control-plane-network-port-table.md
  • snippets/control-plane-network-port-table-k8s.md

New table covers: ICMP, spdk-http-proxy (5000), spdk-firewall-proxy (50001-50065, deprecating with 26.2.3), nvmf internal/client-target (4420-4499), FoundationDB (4500), control plane API (80), control plane RPC (8080-9044), and the monitoring stack (12202, 13301, 13302, 9200, 9090).

Notes

  • These four snippets are {% include %}'d by docs/deployments/.../install-sp.md, install-cp.md, and k8s-storage-plane.md, so the rendered firewall docs update automatically.
  • ⚠️ Not touched (flag for follow-up): the inline iptables scripts in snippets/iptables-setup-docker-swarm.md and docs/deployments/install-on-linux/install-cp.md still hard-code the old port ranges (e.g. 8080:8890, 9090-9900, 4420), and the 25-6-ga.md release note documents older ranges. These are now inconsistent with the updated tables.

🤖 Generated with Claude Code

schmidt-scaled and others added 2 commits June 8, 2026 14:49
Replace the four network-port-table snippets (storage/control plane,
docker + k8s variants) with a single consolidated firewall table:
- spdk-http-proxy 5000, spdk-firewall-proxy 50001-50065 (deprecating 26.2.3)
- nvmf 4420-4499, FoundationDB 4500
- control plane API 80, control plane RPC 8080-9044
- monitoring stack 12202, 13301, 13302, 9200, 9090

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Updated the spdk-firewall-proxy entry to include a superscript note about deprecation.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 8, 2026

🤖: Deployment available as https://docstest.simplyblock.io/update-firewall-port-tables

noctarius added 4 commits June 8, 2026 13:53
Updated the spdk-firewall-proxy entry to include a superscript note about deprecation.
Updated the note about spdk-firewall-proxy deprecation with a superscript.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 8, 2026

🤖: Deployment available as https://docstest.simplyblock.io/update-firewall-port-tables

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 8, 2026

🤖: Deployment available as https://docstest.simplyblock.io/update-firewall-port-tables

…cols)

- Split control+storage services into per-direction rows: egress for
  control, ingress for storage (spdk-http-proxy, spdk-firewall-proxy,
  Control plane RPC)
- Split nvmf into client (egress) and storage (ingress, egress)
- Rename "Source / Target Network" column to "Hosts"
- Add "Network" column: Control everywhere except nvmf (Storage)
- Monitoring Stack hosts: monitoring -> control
- FoundationDB: ingress / control

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 8, 2026

🤖: Deployment available as https://docstest.simplyblock.io/update-firewall-port-tables

@noctarius
Copy link
Copy Markdown
Collaborator

I removed the snippet duplicates and moved the includes to the unified snippet version.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 8, 2026

🤖: Deployment available as https://docstest.simplyblock.io/update-firewall-port-tables

@noctarius noctarius self-requested a review June 8, 2026 12:16
@noctarius noctarius merged commit d9d4027 into main Jun 8, 2026
3 checks passed
@noctarius noctarius deleted the update-firewall-port-tables branch June 8, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants