Skip to content
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

Added ip6tnlEncap to insert ip6tnl encap route #758

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

byteocean
Copy link
Contributor

@byteocean byteocean commented Apr 22, 2022

This commit added support to allow the operation of ip6tnl encapsulation.

It is equivalent to the iproute2 command, e.g., ip route add 192.168.99.0/24 encap ip6 dst 2001:db8:: dev ip6-tunnel

The limitation include that the options field defined in encap nl route attribute is not implemented yet.

Testcase is included.

@MalteJ
Copy link

MalteJ commented Jun 2, 2022

Nice one! Would like to see this merged! 🙂

@byteocean byteocean force-pushed the ip6tnl-pr branch 2 times, most recently from 0d300b8 to d6cf822 Compare June 8, 2022 14:31
@byteocean
Copy link
Contributor Author

byteocean commented Jun 24, 2022

@vishvananda @aboch @trevrosen it would be really cool & nice, this can be merged.

This commit added support to allow the operation of ip6tnl encapsulation.
It is equivalent to the iproute2 command, e.g., ip route add 192.168.99.0/24 encap ip6 dst 2001:db8:: dev ip6-tunnel
The limitation include that the options field defined in encap nl route attribute is not implenmented yet.
Testcase is included.
@byteocean
Copy link
Contributor Author

byteocean commented Jan 23, 2024

@vishvananda @aboch @trevrosen this branch has been rebased to the latest main of netlink. it would be cool to be merged.

@aboch
Copy link
Collaborator

aboch commented Jan 23, 2024

Thanks for adding the UT too.

LGTM

@aboch aboch merged commit 8d48f50 into vishvananda:main Jan 23, 2024
2 checks passed
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.

3 participants