-
Notifications
You must be signed in to change notification settings - Fork 214
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Topology-aware routing docs #1920
Conversation
Forced DCO because Alen was a Buoyant employee when he wrote this... |
This is super nitpicky, but when we did the site rebuild, we switched all frontmatter to yaml. Prior to that, it was a mix of toml and yaml. I'll let you decide if we want to keep this consistent. |
Nice catch! I didn't notice that at all. |
ed8ab05
to
b6a147b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some minor comments on the 2.17 docs, which would also apply to all other versions.
b6a147b
to
83fec68
Compare
ebaa891
to
0683261
Compare
… hints Signed-off-by: Alen Haric <[email protected]>
…mended by Flynn. Signed-off-by: Alen Haric <[email protected]>
Signed-off-by: Alen Haric <[email protected]>
Signed-off-by: Alen Haric <[email protected]>
Signed-off-by: Flynn <[email protected]>
Signed-off-by: Flynn <[email protected]>
Signed-off-by: Flynn <[email protected]>
Signed-off-by: Flynn <[email protected]>
…new trafficDistribution feature Signed-off-by: Flynn <[email protected]>
Signed-off-by: Flynn <[email protected]>
Signed-off-by: Flynn <[email protected]>
1d39fc0
to
e477620
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Originally from @deusxanima in #1579, thanks! 🙂