Skip to content

Implement RFC8781 (NAT64 prefix in RA's) #18626

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

donaldsharp
Copy link
Member

See the individual commits for description as well what is going on.

David wrote the receiver tool and I implemented the topotest to use the scapy script.
This requires scapy version >= 2.6.1 in order to work properly.

@frrbot frrbot bot added tests Topotests, make check, etc zebra labels Apr 9, 2025
@donaldsharp
Copy link
Member Author

frrbot is incorrect imo.

Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

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

looks good

@@ -0,0 +1,67 @@
#!/usr/bin/python
Copy link
Member

Choose a reason for hiding this comment

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

Still fails as rx_ipv6_ra_8781.py has #!/usr/bin/python instead of #!/usr/bin/env python line at the beginning

eqvinox and others added 3 commits April 15, 2025 14:20
This tells hosts on the subnet if (and which) NAT64 prefix is in use.
Useful for things like xlat464, or local dns64.

Updated from the previous -03 draft implementation.  (PLC field did not
exist before.)

Signed-off-by: David Lamparter <[email protected]>
Otherwise pretty much impossible to test this within the current
limitations of topotests.

Signed-off-by: David Lamparter <[email protected]>
This is the test to ensure that the ipv6 ra pref64
extension is working properly..

This is a very simple test.  Enables the feature
on r1 to send out the ra's once every 3 seconds
with the feature turned on.  Then on r2 ensure
that we see the ra with the appropriate values.

Signed-off-by: Donald Sharp <[email protected]>
@donaldsharp
Copy link
Member Author

ci:rerun

1 similar comment
@donaldsharp
Copy link
Member Author

ci:rerun

@mwinter-osr mwinter-osr self-requested a review April 16, 2025 22:35
@mwinter-osr
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master size/XL tests Topotests, make check, etc zebra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants