Last Call Review of draft-ietf-bess-evpn-pref-df-10
review-ietf-bess-evpn-pref-df-10-genart-lc-gurbani-2023-07-02-00

Request Review of draft-ietf-bess-evpn-pref-df
Requested revision No specific revision (document currently at 13)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2023-07-11
Requested 2023-06-27
Authors Jorge Rabadan , Senthil Sathappan , Wen Lin , John Drake , Ali Sajassi
I-D last updated 2023-07-02
Completed reviews Secdir Last Call review of -11 by Peter E. Yee (diff)
Genart Last Call review of -10 by Vijay K. Gurbani (diff)
Rtgdir Last Call review of -09 by Stewart Bryant (diff)
Assignment Reviewer Vijay K. Gurbani
State Completed
Request Last Call review on draft-ietf-bess-evpn-pref-df by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/yyKue3u0e4F2LuAMSzjlxaEKHrQ
Reviewed revision 10 (document currently at 13)
Result Ready w/nits
Completed 2023-07-02
review-ietf-bess-evpn-pref-df-10-genart-lc-gurbani-2023-07-02-00
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-bess-evpn-pref-df-10
Reviewer: Vijay K. Gurbani
Review Date: 2023-07-02
IETF LC End Date: 2023-07-11
IESG Telechat date: Not scheduled for a telechat

Summary: The draft is ready for publications as a Proposed Standard with a few
nite.

Major issues: 0

Minor issues: 1 (please see below)

Nits/editorial comments: 3 (please see below)

Minor

- Sec 2, you may consider ordering the abbreviations alphabetically as it may
be easier for the reader to find what they are looking for.

Nits:

- Sec 1.1, s/two new DF Algs/two new DF algorithms/ (unless, of course, in your
nomenclature, "Alg" means something specific).

- Sec 3, the opening sentence is too long.  You may consider the following
edit: s/route, by replacing/route.  It does so by replacing/

- Sec 4.1, bullet d, s/E.g., 50/e.g., 50/