-
Notifications
You must be signed in to change notification settings - Fork 654
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
Add vpws local and remote identifiers to EVPN-VPWS configuration #930
Add vpws local and remote identifiers to EVPN-VPWS configuration #930
Conversation
Extend openconfig-evpn.yang model with those two leafs for setting both local and remote vpws service identifiers within container evpn/evpn-instances/evpn-instance
No major YANG version changes in commit a517d7f |
Compatibility Report for commit 963ec52: |
|
Reviewed at OC Operators meeting on Aug 1, 2023. Seems like Huawei and Nokia examples do not include an interface, but Cisco and Juniper do. Is the interface reference required to complete the configuration? Also, this model only allows one VPWS per network instance. Probably there should be a list of of the VPWS? |
In case of Nokia, our "network-instance" is defined as a generic point-to-point service, where you have interfaces and optionally you can have one interface plus an "evpn-instance". So there is a 1:1 mapping, and the interface is not included in the evpn-instance, but in the network-instance. As you do for the case of MAC-VRFs.. My suggestion would be to do something similar to the MAC-VRF with EVPN model, that is, an EVPN-VPWS service would be configured as follows:
To satisfy models that support multiple VPWS per "routing-instance" (e.g. JunOS), we could define a new NETWORK_INSTANCE_TYPE, e.g. MULTI-L2P2P (or similar), and then for that type we could include the interface in the evpn-instance... My two cents.. |
This PR was discussed in the public OC special meeting. An alternative proposal was discussed in which connection points were used and enhanced to be able to have multiple point-to-point connections in the same network instance reusing the same EVPN instance. All vendors implementing EVPN-VPWS support the one-to-one mapping case between point-to-point service, while not all support the case of reusing the EVPN insance. The consensus was that is was better to decouple both functions. Hence, with this PR, ONE Network instance will represent ONE point to point service with ONE EVPN instance (route distinguishsçer and target). The proposed solution in this PR was reasonable and implementable for the vendors present in the call. For the "multi Point to point connection" use case, a new type of network instance needs to be created. This will be done in a separate PR triggered by the need of that use case. |
Last call expires today. Since there are no new comments, I will merge this. /gcbrun |
@oscargdd can you resolve the merge conflicts? |
A P2P service establish by VPWS is between a pair of ACs. Hence, it's essential to specify the local interface associated with a P2P service within an EVPN VPWS instance. Each EVPN VPWS instance can accommodate one or more P2P services, provided each P2P service has its unique AC and its local VPWS service ID. Therefore, it is crucial to ensure that we can incorporate multiple P2P services under the same EVPN VPWS instance. I'm also curious about the 'multicast-group,' 'multicast-mask,' and 'replication-mode' attributes within an EVPN VPWS instance as these attributes do not apply to VPWS P2P services. it might be worthwhile to remove them, but please let me know if I've overlooked any specific relevance or context. |
HI, in the public openconfig call we agreed on limiting the functionality of this use case to the one-to-one mapping case, which is the most common. There was another proposal, to add in another PR in which , with a different network instance type you can support multiple P2P over the same EVPN instance. Not all vendors support this way of working, that is why we decided to keep them separate. |
Reviewed in today's in OC Operators meeting. Operators did not have a use case for 'flow-label-enabled' for the EVPN VPWS instance at this time. It seems like this could be added in a future PR, with the necessary references and operational use cases described. @oscargdd can you respond regarding the presence of |
Re-reviewed in April 30, 2024 OC operators meeting. @oscargdd to resolve conflicts and respond on the multicast related leafs. Then I think this is ready to merge. |
/gcbrun |
@oscargdd care to comment on the multicast leaves? That is the only open issue. |
/gcbrun |
/gcbrun |
@wlin212 As you mention, the presence of multicast-group multicast-mask do not apply for the EVPN-VPWS use case. In order not to complicate, when the services is evpn-vpws, these leafs are ignored by the implementation, as they are out of scope and not related. |
Re-reviewed in July 16, 2024 OC Operators meeting without objection and rough consensus that the multicast question can be handled via a separate pull request. |
…nconfig#930) * Update openconfig-evpn.yang with vpws identifiers Extend openconfig-evpn.yang model with those two leafs for setting both local and remote vpws service identifiers within container evpn/evpn-instances/evpn-instance --------- Co-authored-by: Óscar González de Dios <[email protected]>
…nconfig#930) * Update openconfig-evpn.yang with vpws identifiers Extend openconfig-evpn.yang model with those two leafs for setting both local and remote vpws service identifiers within container evpn/evpn-instances/evpn-instance --------- Co-authored-by: Óscar González de Dios <[email protected]>
Change Scope
Platform Implementations
https://www.cisco.com/c/en/us/td/docs/routers/asr9000/software/asr9k-r7-8/lxvpn/configuration/guide/b-l2vpn-cg-asr9000-78x/evpn-virtual-private-wire-service.html
https://www.juniper.net/documentation/us/en/software/junos/evpn-vxlan/topics/example/example-configuring-vpws-service-with-evpn-signaling-mechanisms.html
https://support.huawei.com/enterprise/en/doc/EDOC1000173015/229228a3/configuring-evpn-vpws-over-mpls-functions
https://infocenter.nokia.com/public/7750SR225R1A/index.jsp?topic=%2Fcom.nokia.L2_Services_and_EVPN_Guide%2Fevpn_for_mpls_t-ai9enrmqfa.html
Examples
CISCO
JUNIPER
HUAWEI
NOKIA