Conversation
|
🚀 Deployed on https://preview-257--hedgehog-docs.netlify.app |
There was a problem hiding this comment.
Pull request overview
This PR adds comprehensive documentation for L2 External peering to address issue #246. The documentation explains how to configure Layer 2 external connections as an alternative to the existing BGP-speaking (L3) external connections, using the Hedgehog Gateway for connectivity.
Changes:
- Updated introduction to mention L2/L3 connectivity and Hedgehog Gateway option
- Restructured documentation to distinguish between BGP-speaking and L2 externals
- Added detailed sections on L2 External objects, attachments, and VPC peering
- Included YAML configuration examples for L2 external setup
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
edipascale
force-pushed
the
ema/l2-externals
branch
from
74866ed to
093dc64
Compare
pau-hedgehog
left a comment
pau-hedgehog
left a comment
There was a problem hiding this comment.
Overall looks good. I would suggest some clarifications/refinements
Also the hhfab external parameters changed. Should we update in the relevant sections?
edipascale
force-pushed
the
ema/l2-externals
branch
from
093dc64 to
05c88c2
Compare
|
it would be good at some point to add an example of l2-external e.g. using the virtual external in vlab, but I figure that there are higher priority items in the backlog and we can do that later; if that's not correct let me know and I can re-prioritize |
|
converting to draft as we might be simplifying the APIs |
edipascale
force-pushed
the
ema/l2-externals
branch
2 times, most recently
from
23facd4 to
86061a5
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
edipascale
force-pushed
the
ema/l2-externals
branch
2 times, most recently
from
ea86a75 to
91f6c46
Compare
edipascale
force-pushed
the
ema/l2-externals
branch
2 times, most recently
from
27d3ae0 to
99bd925
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Emanuele Di Pascale <emanuele@githedgehog.com>
edipascale
force-pushed
the
ema/l2-externals
branch
from
99bd925 to
3048a03
Compare
2 participants
Fix #246