Skip to content

Cross-link the service VIPs guide and note its limits - #392

Open
jasikpark wants to merge 1 commit into
vip-ecmp-guidefrom
vip-ecmp-guide-crosslinks
Open

jasikpark wants to merge 1 commit into
vip-ecmp-guidefrom
vip-ecmp-guide-crosslinks

Conversation

@jasikpark

Copy link
Copy Markdown
Collaborator

Stacked on #389 (service VIPs guide). @JackDoan, merge this into vip-ecmp-guide if it looks right, or take whatever parts you want.

  • Links the guide from the tun.unsafe_routes ECMP section, the firewall local_cidr field, and the unsafe_routes guide's tips.
  • In the guide: Mobile Nebula only accepts one gateway per unsafe route (via must be a string), so mobile clients can't use the VIP. Notes the firewall.default_local_cidr_any exception to "local_cidr is required", and that a client sending from one fixed source port sticks to one backend, since the ECMP hash uses only the ports.
  • Links local_cidr, weight and tunnels.drop_inactive to their reference sections, per the repo's docs conventions.

pnpm build passes with no broken links.

Link the guide from the tun ECMP section, the firewall local_cidr field,
and the unsafe_routes guide's tips. In the guide, note that Mobile Nebula
can't take a multi-gateway route, the default_local_cidr_any exception,
and that fixed-source-port clients stick to one backend, and link the
config options it names.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for docs-nebula ready!

Name Link
🔨 Latest commit fedd704
🔍 Latest deploy log https://github.com/DefinedNet/nebula-docs/actions/runs/40371753470
😎 Deploy Preview Url https://f5ba4ae0.docs-nebula.pages.dev
😎 Deploy Preview Alias https://vip-ecmp-guide-crosslinks.docs-nebula.pages.dev

@jasikpark
jasikpark marked this pull request as ready for review October 26, 2026 17:52
@jasikpark
jasikpark requested a review from JackDoan October 26, 2026 17:52
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.

1 participant