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

SRv6 uSID design for srv6orch #1034

Merged
merged 5 commits into from
Aug 21, 2022
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
112 changes: 112 additions & 0 deletions doc/srv6/SRv6_uSID.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,112 @@
# SONiC uSID

## Table of Content
- [Overview](#Overview)
- [Scope](#Scope)
- [Design](#Design)
- [Example](#Example)

## Revision

| Rev | Date | Author | Change Description |
| :--: | :-------: | :------------------------------: | :---------------------: |
| 0.1 | 7/17/2022 | Shitanshu Shah, Reshma Sudarshan | Initial version |


## Overview
SRv6 uSID (micro-segment) is extension of the SRv6 network programming model [SRv6 uSID instructions IETF draft](https://datatracker.ietf.org/doc/draft-filsfils-spring-net-pgm-extension-srv6-usid/). uSID is a compressed SID value which can be for example carried in 16-bits (unlike full IPv6 address to represent a SID). uSID as is designed scales well with much lower MTU overhead required per uSID carrier. uSID carrier is 128-bit IPv6 address that can carry upto 6 uSIDs [Refer to Example for more details]
Copy link
Collaborator

@venkatmahalingam venkatmahalingam Jul 19, 2022

Choose a reason for hiding this comment

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

Looks like https://datatracker.ietf.org/doc/draft-filsfils-spring-net-pgm-extension-srv6-usid is not an IETF adopted draft yet and it is currently an individual draft under IETF spring WG, why would we need this in SONiC until it becomes IETF adopted draft?

Choose a reason for hiding this comment

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

Indeed, the HLD should also point to the IETF draft draft-ietf-spring-srv6-srh-compression-02, which is IETF Working Group draft and standardizes it.

Copy link

@pcamaril pcamaril Jul 20, 2022

Choose a reason for hiding this comment

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

Venkatesesan, also it might be interesting for you to listen to this presentation from MPLS World Congress if you have the time.
Alibaba: Full stack SRv6 towards a 'Predictable Network' - YouTube
Alibaba explains their plans to deploy SRv6, not only with commercial routers, but also with a SONiC based platform. This deployment is happening in early CY23, and hence throughout the next few months the community will work hard on the different missing components (e.g., FRR).
Thanks!!

Copy link
Contributor Author

@svshah-intel svshah-intel Jul 25, 2022

Choose a reason for hiding this comment

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

updated reference to both IETF drafts. Please review/approve accordingly.
Thanks,


## Scope
The scope of this document is to enhance orchagent to support uSID programming instructions in this IETF draft. Current SAI API definitions already support uSID instructions. No SAI API change required in scope of this document. Current version of routing protocols in SONiC does not support SRv6, it is not in the scope of this document to add such a support for FRR routing stack.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think, currently we use FRR 8.1 version on SONiC which supports SRv6, any plan to use FRR stack for SRv6 feature?

Copy link

@reshmaintel reshmaintel Jul 19, 2022

Choose a reason for hiding this comment

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

FRR SRv6 feature set currently has only END functions - and recently DT4 and DT6 were added, but headend functions were missing. Although integration has already been started by Ahmed in Cisco. We will have an update on this soon.


## Design
Current srv6orch is designed, per [SRv6 HLD](https://github.com/sonic-net/SONiC/blob/master/doc/srv6/srv6_hld.md), to support SRv6 programming instructions as described in RFC 8754 and RFC 8986. This design extends SRv6 Network Programming with a new type of SRv6 SID behaviors defined as uSID.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the uSID functionality natively supported in FRR or does it need to be upstreamed? what is the plan or is there any PR to be reviewed? please share if there is any PR for community to review.


SRv6 uSID fully leverages current SRv6 control-plane, without any change, as is implemented by srv6orch.
Copy link
Contributor

Choose a reason for hiding this comment

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

what is the plan for uSID integration with SONiC FRR? please share the roadmap.

Choose a reason for hiding this comment

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

FRR SRv6 integration with SONiC is in progress. Here is the PR shared by Ahmed (https://netdevconf.info/0x16/). We will update soon about which SONiC release is targeted for this.

Choose a reason for hiding this comment

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

https://github.com/cscarpitta/frr Updated the link. This is fork from FRR. in which the university of Rome team added the uSID feature on top.

Following uSID behaviors are added,
uN - uN behavior is implemented with PSP and USD flavor
uA - uA local behavior is implemented with PSP and USD flavor
uDT - uDT local behavior is implemented exactly same as that of End.DT4/End.DT6
uDX - uDX local behavior is implemented exactly same as that of End.DX4/End.DX6

PSP and USD end behavior flavors are already supported by SAI API today. End.DT4/6 and End.DX4/6 end behaviors are supported by SAI APIs as well. Thus there is no additional change required in SAI to support uN, uA, uDT and uDX behaviors.

Changes in orchagent,
- While processing MYSID entries, from SRV6_MY_SID_TABLE off of APPDB, handling of new actions uN, uA, uDT and uDX added in srv6orch. No APPDB schema changes required.
- SAI end behavior and end behavior flavor are determined, before calling SAI APIs, to program MYSID entries

```text
const map<string, sai_my_sid_entry_endpoint_behavior_t> end_behavior_map =
{
{"end", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_E},
{"end.x", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_X},
{"end.t", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_T},
{"end.dx6", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DX6},
{"end.dx4", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DX4},
{"end.dt4", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DT4},
{"end.dt6", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DT6},
{"end.dt46", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DT46},
{"end.b6.encaps", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_B6_ENCAPS},
{"end.b6.encaps.red", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_B6_ENCAPS_RED},
{"end.b6.insert", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_B6_INSERT},
{"end.b6.insert.red", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_B6_INSERT_RED},
+ {"udx6", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DX6},
+ {"udx4", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DX4},
+ {"udt6", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DT6},
+ {"udt4", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DT4},
+ {"udt46", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_DT46},
+ {"un", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_UN},
+ {"ua", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_UA}
};

const map<string, sai_my_sid_entry_endpoint_behavior_flavor_t> end_flavor_map =
{
{"end", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_FLAVOR_PSP_AND_USD},
{"end.x", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_FLAVOR_PSP_AND_USD},
{"end.t", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_FLAVOR_PSP_AND_USD},
+ {"un", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_FLAVOR_PSP_AND_USD},
+ {"ua", SAI_MY_SID_ENTRY_ENDPOINT_BEHAVIOR_FLAVOR_PSP_AND_USD}
};
```

## Example
uSID carrier is 128-bit IPv6 address which is specified in following format:
```text
<uSID-Block><Active-uSID><Next-uSID>...<Last-uSID><End-of-Carrier>[...<End-of-Carrier>]
```
- uSID Block: An IPv6 prefix (defines a block of SRv6 uSIDs)
- Active uSID: The first uSID
- Next uSID: The next uSID after the Active uSID.
- Last uSID: The last uSID in the carrier before the End-of-Carrier
- End-of-Carrier: A globally reserved uSID that marks the end of a uSID list. The End-of-Carrier ID is 0000. As may End-of-Carriers as required to complete full 128-bits IPv6 address

![](images/SRv6_uSID_Example.png)

- uSID block: 2001:41f0
- Active uSID: 0100
- Next uSID: 0200
- Last uSID: 0A00
- 2 End-of-Carriers (0000) to complete full 128-bits IPv6 address

A node with local uSID of 2001:41f0:0100 is to be programmed with following SRV6_MY_SID_TABLE entry, with appropriate uSID end behavior. Following shown two separate examples with 2 different end behaviors.

```text
Note: prefix of "16:8:8:8" is (locator_block_len:locator_node_len:function_len:args_len) as is currently consumed by srv6orch.

If end-behavior "un"
"SRV6_MY_SID_TABLE" : {
"16:8:8:8:2001:41f0:0100::" : {
"action": "un",
}
}

If end-behavior is "udt46"
"SRV6_MY_SID_TABLE" : {
"16:8:8:8:2001:41f0:0100::" : {
"action": "udt46",
"vrf": "VRF-1001"
}
}

A node with local uSID of 2001:41f0:0200 is to be programmed with appropriate uSID end behavior, similarly for node with 2001:41f0:0500, and for node with 2001:41f0:0A00
```
Binary file added doc/srv6/images/SRv6_uSID_Example.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.