-
Notifications
You must be signed in to change notification settings - Fork 220
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
Refactor fabric documentation #655
Refactor fabric documentation #655
Conversation
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.
Looks good to me!
(Tested with an additional 3-stage clos fabric)
Hold on a second. Found a few issues I will fix |
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.
If we list mlag_peer in the topology, should not we also provide L3 interfaces for MLAG as well since these links are now listed in the documentation?
I did not want to delay this by adding too many new areas, but we should add (in another PR): |
Ok sounds like a good plan. |
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.
Looks good to me!
Change Summary
Complete rewrite of the fabric documentation for
eos_l3ls_evpn
Types of changes
Related Issue(s)
Fixes issue #552
Fixes issue #475
Component(s) name
arista.avd.eos_l3ls_evpn
arista.avd.build_output_folders
Proposed changes
How to test
Tested with my repo as well as all the molecule test scenarios.
Works for both l3ls and multipod
Checklist:
pre-commit
,make linting
andmake sanity-lint
).