Skip to content

Conversation

@qmonnet
Copy link
Member

@qmonnet qmonnet commented Jan 21, 2026

  • gateway: Remove unnecessary white space
  • gateway: Fix grammar and style for peering to external connections
  • gateway: Move note on "ext." prefix, clarify requirement
  • gateway: Document overlap rules, "default" destination expose

Fixes: #250

Remove trailing white spaces and unnecessary blank lines in the file.

Signed-off-by: Quentin Monnet <qmo@qmon.net>
In "this will allow hosts [...] to the WAN", we don't specify what is
allowed. Fix it and reword the paragraph slightly.

Signed-off-by: Quentin Monnet <qmo@qmon.net>
Move the note below the YAML fragment, as other notes are coming and we
probably don't want to insert too many explanations between the two
related YAML samples.

Also clarify that using the prefix is a requirement (although we don't
go as far as to explain why - my understanding is that it's for internal
reasons that the user does not really care about).

Signed-off-by: Quentin Monnet <qmo@qmon.net>
@qmonnet qmonnet self-assigned this Jan 21, 2026
@github-actions
Copy link

github-actions bot commented Jan 21, 2026

🚀 Deployed on https://preview-251--hedgehog-docs.netlify.app

@qmonnet qmonnet force-pushed the pr/qmonnet/document-default branch from e4dce2c to 21cf133 Compare January 21, 2026 11:59
Document:

- The "no exposed prefix overlap" rule, with the exception for the
  "default" exposes
- The behaviour for the "default" exposes
- The current restriction on having at most one "default" exposed to a
  VPC - this may change in the future, but dataplane only supports one
  at the moment.

Signed-off-by: Quentin Monnet <qmo@qmon.net>
@qmonnet qmonnet force-pushed the pr/qmonnet/document-default branch from 21cf133 to 99531cb Compare January 21, 2026 12:00
@qmonnet qmonnet marked this pull request as ready for review January 23, 2026 18:05
@qmonnet qmonnet requested a review from a team as a code owner January 23, 2026 18:05
@qmonnet qmonnet requested a review from mrbojangles3 January 23, 2026 18:05
@Frostman Frostman merged commit c593f0c into master Jan 23, 2026
4 checks passed
@Frostman Frostman deleted the pr/qmonnet/document-default branch January 23, 2026 23:56
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.

Update docs with "default" in the gateway peering

3 participants