[RTG-DIR] Rtgdir early review of draft-ietf-bess-bgp-sdwan-usage-06

[Shuping Peng via Datatracker <[e-mail]>]

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?