Skip to content

doc: core peering#4926

Open
tzaeschke wants to merge 8 commits into
masterfrom
tzaeschke-core-peering-doc
Open

doc: core peering#4926
tzaeschke wants to merge 8 commits into
masterfrom
tzaeschke-core-peering-doc

Conversation

@tzaeschke

Copy link
Copy Markdown
Contributor

This PR proposes documentation that describes how peering shortcuts with core ASes work. See also: #4909

This PR does not add any diagrams. New/additional diagrams may be useful, in that case I would ask for access to the originals of the existing diagrams. It could generally be useful if diagram sources would be added to the repository.

@tzaeschke tzaeschke requested review from katyatitkova and oncilla May 15, 2026 14:01
Comment thread doc/control-plane.rst Outdated
Comment thread doc/control-plane.rst Outdated
Comment thread doc/control-plane.rst Outdated
shortcuts are only allowed (and announced) between ASes that are on UP segment or DOWN segments.
This trivially allows peering shortcuts between non-core ASes and ASes.

Peering shortcuts However, for segment combination,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am unsure if this explanation will not further confuse the uninitiated reader.

It might be easier to understand if we just describe what the role of the core segment is in such scenarios. In the end, it is simply a mechanism to discover the source ASes for the candidate down segments.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@oncilla I'm not sure what you mean by "role of the core segment"?

Anyway, I rephrased the whole section, maybe it is better now?

Comment thread doc/control-plane.rst Outdated
Comment thread doc/control-plane.rst Outdated
Co-authored-by: Dominik Roos <roos@anapaya.net>
Comment thread doc/control-plane.rst Outdated
Comment thread doc/control-plane.rst Outdated
Comment thread doc/control-plane.rst Outdated
Comment thread doc/control-plane.rst Outdated
Comment thread doc/control-plane.rst Outdated
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