Skip to content

vxlan: Update VXLAN documentation to include EVPN #444

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

Merged
merged 7 commits into from
Oct 16, 2024
Merged

Conversation

wido
Copy link
Contributor

@wido wido commented Oct 8, 2024

This is a first attempt on getting proper documentation on how to use VXLAN with BGP and EVPN underneath CloudStack

For PR apache/cloudstack#9778


📚 Documentation preview 📚: https://cloudstack-documentation--444.org.readthedocs.build/en/444/

This is a first attempt on getting proper documentation on how to use VXLAN
with BGP and EVPN underneath CloudStack
@rohityadavcloud
Copy link
Member

cc @rajujith @NuxRo @andrijapanicsb @alexandremattioli - can you review the content & technicalities, otherwise LGTM. Thanks for the PR @wido

@rohityadavcloud rohityadavcloud added this to the 4.20 milestone Oct 9, 2024
Copy link
Contributor

@NuxRo NuxRo left a comment

Choose a reason for hiding this comment

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

Corrected a few very minor typos, hashtag polish, otherwise LGTM.


Increase needed parameter to sane value (i.e. 100 or 200) as required.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Increase needed parameter to sane value (i.e. 100 or 200) as required.
Increase the needed parameter to an appropriate value (i.e. 100 or 200) as required.

::
- BGP Routing Daemon on the hypervisor
- No LACP/Bonding will be used
- Modified script is required to use EVPN instead of Multicast
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Modified script is required to use EVPN instead of Multicast
- The modified script is required

Choose a reason for hiding this comment

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

@NuxRo Did you truncate the phrase by accident?


.. note:: Please use rebuild kernel and tools at your own risk.
Under Linux you can NOT provision (start) more than 20 VXLAN interfaces and error message "No buffer space available" can be observed in Cloudstack Agent logs after provisioning required bridges and VXLAN interfaces.

Choose a reason for hiding this comment

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

Suggested change
Under Linux you can NOT provision (start) more than 20 VXLAN interfaces and error message "No buffer space available" can be observed in Cloudstack Agent logs after provisioning required bridges and VXLAN interfaces.
Under Linux you can NOT by default provision (start) more than 20 VXLAN interfaces and the error message "No buffer space available" will appear in the Cloudstack Agent logs after provisioning the required bridges and VXLAN interfaces.

Copy link

@alexandremattioli alexandremattioli left a comment

Choose a reason for hiding this comment

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

Left some polish suggestions.

Also, the zone creation isolation method screenshot shows VLAN, probably a good idea to replace it with one showing VXLAN to avoid confusion.

Otherwise LGTM. Am actually setting up an env following the doc together with the guys from https://www.netris.io/ @wido

::
- BGP Routing Daemon on the hypervisor
- No LACP/Bonding will be used
- Modified script is required to use EVPN instead of Multicast

Choose a reason for hiding this comment

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

@NuxRo Did you truncate the phrase by accident?

@wido
Copy link
Contributor Author

wido commented Oct 10, 2024

Left some polish suggestions.

Also, the zone creation isolation method screenshot shows VLAN, probably a good idea to replace it with one showing VXLAN to avoid confusion.

Otherwise LGTM. Am actually setting up an env following the doc together with the guys from https://www.netris.io/ @wido

Great! Good to see more people are going to use it. I do recommend to use Frr-10 btw, has some nice bug fixes. But this is frr-stable nowadays.

@alexandremattioli
Copy link

Left some polish suggestions.
Also, the zone creation isolation method screenshot shows VLAN, probably a good idea to replace it with one showing VXLAN to avoid confusion.
Otherwise LGTM. Am actually setting up an env following the doc together with the guys from https://www.netris.io/ @wido

Great! Good to see more people are going to use it. I do recommend to use Frr-10 btw, has some nice bug fixes. But this is frr-stable nowadays.

Indeed they are using FRR10. They have a very cool product, lots of interesting networking automation/orchestration, and soon with ACS integration.

@wido
Copy link
Contributor Author

wido commented Oct 16, 2024

All changes approved, all checks passed. I see no objections, merging this one soon.

@wido wido merged commit 4c789e1 into apache:main Oct 16, 2024
1 check passed
@wido wido deleted the vxlan-evpn branch October 16, 2024 10:13
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.

5 participants