Hi authors,
Please find below my chair/shepherd's review of
draft-ietf-bess-mvpn-evpn-sr-p2mp.
Introduction:
* "A SR P2MP tree is defined by a SR P2MP Policy and instantiated via a PCE"
* I would use the name controller instead of PCE. PCE is really tied to
PCEP protocol IMO. If we agree, then you should change it across the doc. I
appears in other sections too.
Section 2:
* "A Replication segment of a SR P2MP tree can be instantiated..."
* Shoudln't you provide informational refs here ?
Section 3:
* I would enhance the tunnel-type description with a list, something like
" * Tunnel Type:
* 0x0c for SR-MPLS P2MP tree
* TBD for SRv6 P2MP Tree
"
Section 3.1.2
s/"Domain- wide"/"Domain-wide" (remove space)
Section 4.1.1
Use an XML reference for RFC6514 Section 9.1.1 instead of hardcoding in text.
(same in 4.1.2, 4.2.1, 4.2.2, 4.3.2...).
When you refer to "condition (c)", it's not clear, where it's defined.
Section 10
Please fix last name of Luc Andre (there are two "t" instead of t, it should be
Burdet).
Thanks,
Stephane
_______________________________________________
BESS mailing list -- [email protected]
To unsubscribe send an email to [email protected]
