Skip to main content

Last Call Review of draft-ietf-bess-evpn-optimized-ir-08
review-ietf-bess-evpn-optimized-ir-08-tsvart-lc-tuexen-2021-09-07-00

Request Review of draft-ietf-bess-evpn-optimized-ir
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team Transport Area Review Team (tsvart)
Deadline 2021-09-07
Requested 2021-08-24
Authors Jorge Rabadan , Senthil Sathappan , Wen Lin , Mukul Katiyar , Ali Sajassi
I-D last updated 2021-09-07
Completed reviews Rtgdir Early review of -09 by Julien Meuric (diff)
Tsvart Last Call review of -08 by Michael Tüxen (diff)
Secdir Last Call review of -09 by Derek Atkins (diff)
Genart Last Call review of -09 by Gyan Mishra (diff)
Opsdir Last Call review of -09 by Tim Chown (diff)
Intdir Telechat review of -09 by Pascal Thubert (diff)
Assignment Reviewer Michael Tüxen
State Completed
Request Last Call review on draft-ietf-bess-evpn-optimized-ir by Transport Area Review Team Assigned
Posted at https://mailarchive.ietf.org/arch/msg/tsv-art/mPy1FOQ3Y9CvQhgEf4jdEMeQ5wc
Reviewed revision 08 (document currently at 12)
Result Ready w/nits
Completed 2021-09-07
review-ietf-bess-evpn-optimized-ir-08-tsvart-lc-tuexen-2021-09-07-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.

I have not found issues related to transport.
However, I have two questions:

Section 5.2
What is the timer value for AR-REPLICATOR-activation-timer?
Are there dependencies to other parameters?

Section 6.2
What is the timer value for timer t?
Are there dependencies to other parameters?

Nits:

Abstract
Resolve BUM, resolve acronyms on first occurrence

Section 1
BUM resolved after being used.

Section 10
A implementation following -> An implementation following