Hi Bruno, Thank you very much for your valuable comments.
My responses/explanations are inline below with [HC]. Best Regards, Huaimo on behalf of co-authors ________________________________ From: spring <spring-boun...@ietf.org> on behalf of bruno.decra...@orange.com <bruno.decra...@orange.com> Sent: Wednesday, February 2, 2022 8:26 AM To: slitkows.i...@gmail.com <slitkows.i...@gmail.com>; 'SPRING WG' <spring@ietf.org>; Huzhibo <huzh...@huawei.com> Subject: Re: [spring] WG adoption call - draft-hu-spring-segment-routing-proxy-forwarding Hi authors of both documents, WG, [Speaking as individual contributor.] It’s good to see technical discussions on the restoration of failed SIDs used by SR policy. 1. From a functional point of view, can we summarize the benefit to signal the node proxy capability? e.g. - drop the traffic earlier if the PLR does not support proxy capability. (helps with congestion) [HC]: When the PLR supports proxy capability, it provides protection for the failure of its adjacent node with BSIDs. The BSIDs are protected. When the PLR does not support proxy capability, it provides protection for the failure of its adjacent node, but the BSIDs on the node are not protected. - use another proxy off the shortest path (increase congestion but reduce loss) [HC]: When there is a node on the shortest path not supporting proxy, another proxy capable node off the shortest path to a neighbor of the failed node is used. The traffic is sent to the neighbor, which re-routes the traffic around the failed node towards the destination. The traffic is protected. In fact, the failed node is a loose hop on the SR path with the SID of the node. >From a upstream node to the failed node, there are a few hops in general. When any of these hops fails, the traffic will be re-routed around the failure. This should be considered by the controller that computes the SR path. This should not increase congestion. Using proxy capable node off the shortest path to a neighbor of the failed node to some extend should not increase congestion. Using another proxy capable node off the shortest path reduces traffic loss and should not increase congestion to some extend. - possibly help identifying the proxy (nominal is not in the reachable topology anymore) [HC]: When a node failed and becomes unreachable, it helps identifying the proxy capable path to a neighbor of the failed node. … Or agree on the absence of significant benefits? [HC]: It provides more protection coverage in some cases as compared to the other existing draft. This improves the reliability of networks, and QoE. This should be a significant benefit. In addition, when a node fails, the nodes of the entire network converge to the latest state consistently in time. After a node failed, comparing to the other existing draft regarding to holding off the FIB during the HoldTimer period, when the network changes again, our solution continues to converge at any time. 1. draft-ietf-spring-node-protection-for-sr-te-paths “If the Node-SID or Prefix-SID becomes unreachable, the event and resulting forwarding changes should not communicated to the forwarding planes on all configured routers (including PLRs for the failed node) until the hold-timer expires.” * It’s not crystal clear to me how it would work in reality, so I would welcome more prescriptive text. In particular: * “node failure” is not an IGP message. IGP nodes sees multiple “adjacency loss” messages which are not atomic and could be handled in multiple SPFs. Hence different nodes will freeze their FIB based on a different topology (link1 for some, link2 for others) leading to inconsistent routing and forwarding loops. * How is the FIB modified in cases of consecutives IGP events? (freezed on hold topology may lead to drops, updating entries would need to be specified. * On a side node, this text requires a global behavior of all IGP nodes. That seem a bit out of scope of a non-normative sentence, in an informational document, describing a local behavior on the PLR. [HC]: In addition to the above, it seems better to describe procedures about link/node up and down events regarding to FIB entries such as those being held off (not to be changed) during the period from HoldTimer start to end. For example, after a node is down, some FIB entries are held off (not to be changed), then the node is up, but some of its links are up and the other links are still down (maybe for a long time). How to handle the FIB entries (hold off or change) in this case? 1. draft-hu-spring-segment-routing-proxy-forwarding Rather than defining a new “Proxy Forwarding” capability in IGP why don’t you use the existing Mirroring Segment (from RFC https://datatracker.ietf.org/doc/html/rfc8402#section-5.1<https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fhtml%2Frfc8402%23section-5.1&data=04%7C01%7Chuaimo.chen%40futurewei.com%7C12af08c51f9646ac60e308d9e64f9c0c%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C1%7C637794051887643063%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&sdata=ENHobem9VChu%2Fh2wTK5QXtC60ypc18rRRy9LgMfXz4o%3D&reserved=0>) whose signaling is already standardized? https://datatracker.ietf.org/doc/html/rfc8667#section-2.4.1<https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fhtml%2Frfc8667%23section-2.4.1&data=04%7C01%7Chuaimo.chen%40futurewei.com%7C12af08c51f9646ac60e308d9e64f9c0c%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C1%7C637794051887643063%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&sdata=%2BpplasnFxP69Ox3kv%2BoqhhsfdS7%2FQUooMyaygZ96f4Y%3D&reserved=0> [HC]: “Proxy Forwarding” capability of a node may be alternatively indicated by the mirror SIDs advertised by the node. Mirror SID for a failed node can be used as context to forward the traffic with the SID of the failed node to the next hop of the failed node. A node advertises a mirror SID for each of its neighbor nodes. When a node failed, its node SID is not reachable. A remote node of the failed node sends the traffic with the SID of the failed node to a neighbor of the failed node. The neighbor sends (i.e., re-route) the traffic to the next hop of the failed node when the neighbor advertises the mirror SID for the failed node. When the neighbor does not advertise the mirror SID for the failed node, it pushes the mirror SID advertised by another neighbor (say AN) and sends the traffic to AN. AN pops its mirror SID as context and uses the SID of the failed node to re-route the traffic to the next hop of the failed node. 1. What about the following solution: * Use mirror SID * Tunnel to the “proxy-forwarding” advertising mirror SID [HC]: If tunnels are used, there may be tunnels from each of the remote nodes to each of the neighbors of a failed node. This may use some network resources. It seems that tunnels may be replaced by the shortest path to the neighbor along the nodes advertising mirror SIDs. I would see the following benefits: * No new protocol extensions (cf “3)” * Consistent routing in case of multiple SPFs (cf “2)”) * Benefit from the signaling of the proxy (cf “1)”) [HC]: We can see these. Thanks, Regards, --Bruno Orange Restricted From: slitkows.i...@gmail.com <slitkows.i...@gmail.com> Sent: Tuesday, January 25, 2022 6:13 PM To: DECRAENE Bruno INNOV/NET <bruno.decra...@orange.com>; 'SPRING WG' <spring@ietf.org> Subject: RE: [spring] WG adoption call - draft-hu-spring-segment-routing-proxy-forwarding Hi, I’m NOT supporting this draft for the following reasons: 1. The WG already have a WG document which is dealing with this problem, I don’t think that WG should come with multiple documents/solutions for the same solution space as it may just confuse the industry and create deployment issues as different vendors may pick different solutions. 1. Adding protocols extensions adds complexity in the solution without adding a strong value. The document claims that “[I-D.ietf-spring-segment-protection-sr-te-paths] … may not work for some cases such as some of nodes in the network not supporting this solution.”. While this is true, the proposed solution in draft-hu-spring-segment-routing-proxy-forwarding has exactly the same caveat and requires all nodes in the network to support the solution. Considering the following straight line network: A -B -C -D – E – F - G -H and an SR policy from A to H using SID_G, routers A to F have to support the extension to make the solution working, if one of the router doesn’t support the extension, traffic will be dropped. Then, there is no value compared to the timer-based solution of [I-D.ietf-spring-segment-protection-sr-te-paths] Authors of draft-hu-spring-segment-routing-proxy-forwarding argued that G may have multiple upstream neighbors let’s say F and F’ and the solution allows for F’ to support the extension while F may not support, so the solution will send the traffic to F’. Well yes, but this still requires all routers upstream to F’ to support this extension and maybe F is on the path to F’. So, I don’t think the argument is valid as it may possibly work tactically depending on the network topology when we look at a small portion of the network, but when we look at the whole network, operator will have to upgrade all their nodes to support the extension to ensure the benefit is there. In addition, in term of traffic, forwarding traffic to a neighbor of the failed node which wasn’t initially on the path, could lead to traffic congestion or high traffic peaks on links that were not sized to carry this traffic. We could easily expect some traffic tromboning, where traffic goes to this non-natural neighbor of the failed node and then goes back over some part of the same path before reaching the destination. So these protocol extensions are bringing complexity for no value here. 1. Regarding BSID, I’m not fan of advertising BSIDs in IGP as there may be hundreds or thousands of BSID on a node which again will create a lot of burden in IGP. The proposed way will have to be discussed in LSR, not in SPRING (see next comment). Note that [I-D.ietf-spring-segment-protection-sr-te-paths] could also work with BSIDs as long as BSID information of failed node is available in the control-plane of PLRs by whatever mechanism. I think this BSID handling is orthogonal to the proxy-forwarding controlplane behavior. The forwarding operations for BSID will have to be discussed more in details, we could not expect all HW to be able to do 3 or 4 lookups without any perf degradation. 1. The document is currently a bit borderline between SPRING and LSR as it talks in good details about IGP protocol extensions. If it’s a SPRING doc, it should detail reqs for protocols but nothing beyond. Brgds, Stephane From: spring <spring-boun...@ietf.org<mailto:spring-boun...@ietf.org>> On Behalf Of bruno.decra...@orange.com<mailto:bruno.decra...@orange.com> Sent: jeudi 13 janvier 2022 11:19 To: SPRING WG <spring@ietf.org<mailto:spring@ietf.org>> Subject: [spring] WG adoption call - draft-hu-spring-segment-routing-proxy-forwarding Dear WG, This message starts a 2 week WG adoption call, ending 27/01/2022, for draft-hu-spring-segment-routing-proxy-forwarding https://datatracker.ietf.org/doc/draft-hu-spring-segment-routing-proxy-forwarding/<https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fdraft-hu-spring-segment-routing-proxy-forwarding%2F&data=04%7C01%7Chuaimo.chen%40futurewei.com%7C12af08c51f9646ac60e308d9e64f9c0c%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C1%7C637794051887643063%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&sdata=mTBBwsQquqH7GTfnwgHB8BRMndNK3IhgV5fJr7eXx9E%3D&reserved=0> After review of the document please indicate support (or not) for WG adoption of the document to the mailing list. Please also provide comments/reasons for your support (or lack thereof) as this is a stronger way to indicate your (non) support as this is not a vote. If you are willing to work on or review the document, please state this explicitly. This gives the chairs an indication of the energy level of people in the working group willing to work on the document. Thanks! Bruno, Jim, Joel _________________________________________________________________________________________________________________________ Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration, Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci. This message and its attachments may contain confidential or privileged information that may be protected by law; they should not be distributed, used or copied without authorisation. If you have received this email in error, please notify the sender and delete this message and its attachments. As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified. Thank you. _________________________________________________________________________________________________________________________ Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration, Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci. This message and its attachments may contain confidential or privileged information that may be protected by law; they should not be distributed, used or copied without authorisation. If you have received this email in error, please notify the sender and delete this message and its attachments. As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified. Thank you.
_______________________________________________ spring mailing list spring@ietf.org https://www.ietf.org/mailman/listinfo/spring