-
Notifications
You must be signed in to change notification settings - Fork 22
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
V3 vrrp #47
Draft
Paul-weqe
wants to merge
9
commits into
holo-routing:master
Choose a base branch
from
Paul-weqe:v3-vrrp
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
V3 vrrp #47
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
- Introduce ipv6 sockets in network sockets - Modify vrrp packet generation to handle both v2 and v3
Introduce the versioning file for holding the following enums: - IpVersion - VrrpVersion Signed-off-by: Paul Wekesa <[email protected]>
- on Interface struct, there was only one instance BTreeMap. Aim is to eventually introduce a separate BTreeMap for v2 and v3 Signed-off-by: Paul Wekesa <[email protected]>
- Initialize YANG calls for vrrpv3. - Deal with mvlan interface creation and deletion in vrrpv3. Signed-off-by: Paul Wekesa <[email protected]>
- change naming of sockets, network methods and InstanceNet fields. - make sure the update() method has mvlan6 in consideration - include both v2 and v3 instances in `iter_instances` - introduce advertisement interval for ipv6 vrrp packets (not yet enabled / called ) - modify packet generators in Instance Signed-off-by: Paul Wekesa <[email protected]>
- config callbacks and verifications added for ipv6 vrrp - remove socket and tasks fields that had been added for InstanceNet - add ipv4 and ipv6 instance groups for VRRP v3 in Instance - adjust startup() and update() procedures for instance to accomodate vrrp v3 - update format Signed-off-by: Paul wekesa <[email protected]>
mvlan6 & mvlan4 were previously used in case the same vrid had a vrrp2 and vrrp3 instance. Since the distinction of whether it is ipv4 or ipv6 instance is made on the `Interface` level, there is no need to have them as separate fields and both have been merged into a single `mvlan` field. Signed-off-by: Paul Wekesa <[email protected]>
- add neighbor advert method in impl Instance - add ipv6 addresses when collecting system ip addresses Signed-off-by: Paul Wekesa <[email protected]>
- Made sure Neighbor Advertisements were working - Decoding of received VRRP V3 packets fixed. todo: FSM correctness still a WIP. Signed-off-by: Paul Wekesa <[email protected]>
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.
Add VRRP V3 functionality and IPV6 configuration based on RFC 9568
Note: This PR only handles IPV6, IPV4 configuration will be handled in a different PR.