Reviewer: Shuping Peng Review result: Has Nits Hello
I have been selected to do a routing directorate “early” review of this draft. https://datatracker.ietf.org/doc/html/draft-ietf-bess-bgp-sdwan-usage-06 The routing directorate will, on request from the working group chair, perform an “early” review of a draft before it is submitted for publication to the IESG. The early review can be performed at any time during the draft’s lifetime as a working group document. The purpose of the early review depends on the stage that the document has reached. As this document is in working group last call, my focus for the review was to determine whether the document is ready to be published. Please consider my comments along with the other working group last call comments. For more information about the Routing Directorate, please see http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir Document: draft-ietf-bess-bgp-sdwan-usage-06.txt Reviewer: Shuping Peng Review Date: 25-Oct-22 Intended Status: Informational Summary: This document is basically ready for publication, but has nits that should be considered prior to being submitted to the IESG. Comments: 3.1.3 "The SD-WAN Traffic Segmentation is enabled on a single SD-WAN Service to a single subscriber." Here the word "subscriber" is used for the first time. More associations with the context or more explanations would be appreciated. 5.2 UPDATE #1b for Route Route 12.1.1.x/24: - MP-NLRI Path Attribute: 12.1.1.x/24 Nexthop: 2.2.2.2 (C-PE2) - Encapsulation Extended Community: Type= SDWAN-Hybrid - Color Extended Community: YELLOW Should the Type be "MPLS" instead of "SDWAN-Hybrid" here since it is mentioned above that "...suppose that Route 10.1.x.x/16 can be carried by either MPLS or IPsec and Route 12.1.1.x/24 can only be carried by MPLS..."? Nits: 1. SDWAN or SD-WAN? 2. page 15, s/for the Node Loopback address:/for the Node Loopback address. 3. page 17, 18, s/Type, Tunneltype, Tunnel-type/TYPE? _______________________________________________ BESS mailing list [email protected] https://www.ietf.org/mailman/listinfo/bess
