Cumulus_NVUE: Add gateway support #1782
Draft
+177
−4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jbemmel
commented
Jan 9, 2025
| wait: 30 | ||
| wait_msg: Waiting for RA message to generate the default route | ||
| nodes: [ h1, h2 ] | ||
| plugin: default6() |
There was a problem hiding this comment.
Issue here is multiple default routes
There was a problem hiding this comment.
Yeah, that's always the issue with multiple IPv6 routers attached to the same subnet. I wrote about that 11 years ago: https://blog.ipspace.net/2014/02/first-hop-load-balancing-in-ipv6/
jbemmel
force-pushed
the
cumulus_nvue_gw_support
branch
from
5655cfa to
a55cbf8
Compare
* Anycast ipv4 (ipv6 also configured if provided, module officially only supports ipv4) * VRRPv3 ipv4 and ipv6 * Comment out ipv6 addresses in VRRP test case (requires OSPFv3 which Cumulus NVUE does not support) * Add VRRP ipv6 test case (issue with multiple ipv6 default routes popping up)
… reach hosts via R2 (ECMP)
jbemmel
force-pushed
the
cumulus_nvue_gw_support
branch
from
583d740 to
5c385f5
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Anycast ipv4 (ipv6 also configured if provided, module officially only supports ipv4)
VRRPv3 ipv4 and ipv6, with preempt and priority
Comment out ipv6 addresses in anycast test case (requires OSPFv3 which Cumulus NVUE does not support, and anycast is only supposed to support ipv4 according to the documentation)
Add VRRP ipv6 test case (issue with multiple ipv6 default routes popping up)
Update documentation
Validation:
Ping test fails on occasion, I suspect due to having multiple ipv6 default routes with different next hops. Also tested EOS;
same issue
I would suggest to consider this after the new release; the changes are incremental and shouldn't break any existing functionality, but the VRRP v6 validation test needs more work