Skip to main content

Last Call Review of draft-ietf-bess-evpn-lsp-ping-08
review-ietf-bess-evpn-lsp-ping-08-tsvart-lc-eddy-2022-10-09-00

Request Review of draft-ietf-bess-evpn-lsp-ping
Requested revision No specific revision (document currently at 11)
Type IETF Last Call Review
Team Transport Area Review Team (tsvart)
Deadline 2022-10-18
Requested 2022-10-04
Authors Parag Jain , Ali Sajassi , Samer Salam , Sami Boutros , Greg Mirsky
I-D last updated 2023-11-09 (Latest revision 2023-05-29)
Completed reviews Genart IETF Last Call review of -08 by Joel M. Halpern (diff)
Secdir IETF Last Call review of -08 by Rifaat Shekh-Yusef (diff)
Tsvart IETF Last Call review of -08 by Wesley Eddy (diff)
Dnsdir IETF Last Call review of -08 by Di Ma (diff)
Secdir Telechat review of -09 by Rifaat Shekh-Yusef (diff)
Rtgdir IETF Last Call review of -07 by Joel M. Halpern (diff)
Assignment Reviewer Wesley Eddy
State Completed
Request IETF Last Call review on draft-ietf-bess-evpn-lsp-ping by Transport Area Review Team Assigned
Posted at https://mailarchive.ietf.org/arch/msg/tsv-art/l9YKSCpJMplYB9KkQWqoPbUopMg
Reviewed revision 08 (document currently at 11)
Result Ready
Completed 2022-10-09
review-ietf-bess-evpn-lsp-ping-08-tsvart-lc-eddy-2022-10-09-00
This document has been reviewed as part of the transport area review team's
ongoing effort to review key IETF documents. These comments were written
primarily for the transport area directors, but are copied to the document's
authors and WG to allow them to address any issues raised and also to the IETF
discussion list for information.

When done at the time of IETF Last Call, the authors should consider this
review as part of the last-call comments they receive. Please always CC
tsv-art@ietf.org if you reply to or forward this review.

Some comments:
(1) Is there an 'e' missing at the end of this sentence in Section 1?:
validate data plane against the control plan.
->
validate data plane against the control plane.

(2) It seems like some words are missing in this sentence in Section 4.1:
The EVPN MAC/IP Sub-TLV identifies the MAC or ARP/ND for an EVPN
->
The EVPN MAC/IP Sub-TLV identifies the target MAC address for ARP/ND for an EVPN

(3) For IP address used for examples (e.g. in Section 6.1), consider using the
documentation prefix (RFC 5737).