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

Fixes for BGP-MUP SAFI and support Prefix SID attribute in Interwork/Direct Segment Discovery route #2583

Merged
merged 3 commits into from
Oct 16, 2022

Conversation

higebu
Copy link
Contributor

@higebu higebu commented Oct 12, 2022

This PR is depends on #2586.

Commits:

  • cli,mup: add mup option to t2st
  • api,pkg/packet/bgp: fix SRv6 Endpoint Behavior for MUP values
  • cli,mup: support Prefix SID in Interwork/Direct Segment Discovery route

New version of Usage: https://gist.github.com/higebu/90c0504545978e0cf12cc97b89cb5c81
pcap file: gobgp_mup_pcap_20221014.zip
(You can see this pcap with latest wireshark which can be found at https://www.wireshark.org/download/automated/)

@higebu higebu changed the title Fixes for BGP-MUP SAFI and support Prefix SID attribute in Interwork/Direct Segment Discovery route WIP: Fixes for BGP-MUP SAFI and support Prefix SID attribute in Interwork/Direct Segment Discovery route Oct 13, 2022
@higebu higebu force-pushed the fix-mup-2 branch 2 times, most recently from 233b23e to b3d8cf8 Compare October 14, 2022 06:51
@higebu higebu changed the title WIP: Fixes for BGP-MUP SAFI and support Prefix SID attribute in Interwork/Direct Segment Discovery route Fixes for BGP-MUP SAFI and support Prefix SID attribute in Interwork/Direct Segment Discovery route Oct 16, 2022
@higebu
Copy link
Contributor Author

higebu commented Oct 16, 2022

Rebased to master branch.

@fujita fujita merged commit cc75b03 into osrg:master Oct 16, 2022
@fujita
Copy link
Member

fujita commented Oct 16, 2022

Thanks a lot!

Why not add gobgp_mup.md to docs/?

@higebu
Copy link
Contributor Author

higebu commented Oct 16, 2022

@fujita Thank you for merging. I'll create the PR for docs tomorrow 👍

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.

None yet

2 participants