Skip to main content

Early Review of draft-ietf-bess-evpn-mh-pa-09

Request Review of draft-ietf-bess-evpn-mh-pa
Requested revision No specific revision (document currently at 10)
Type Early Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2023-11-10
Requested 2023-10-18
Requested by Stephane Litkowski
Authors Patrice Brissette , Luc André Burdet , Bin Wen , Eddie Leyton , Jorge Rabadan
I-D last updated 2023-11-03
Completed reviews Genart Early review of -09 by Paul Kyzivat (diff)
Rtgdir Early review of -07 by Ketan Talaulikar (diff)
Assignment Reviewer Paul Kyzivat
State Completed
Request Early review on draft-ietf-bess-evpn-mh-pa by General Area Review Team (Gen-ART) Assigned
Posted at
Reviewed revision 09 (document currently at 10)
Result Ready w/issues
Completed 2023-11-03
I am the assigned Gen-ART reviewer for this draft. For background on 
Gen-ART, please see the FAQ at <>.

Please resolve these comments along with any other comments you may receive.

Document: draft-ietf-bess-evpn-mh-pa-09
Reviewer: Paul Kyzivat
Review Date: 2023-11-10
IETF LC End Date: ?
IESG Telechat date: ?


This draft is on the right track but has open issues, described in the 

Issues: 2
Nits: 0

Note: This reviewer is not burdened with any knowledge of the subject 
matter. It is possible that the points I raise below may be entirely 
clear to a subject matter expert. Just take this as input from a fresh 
and unbiased set of eyes.

Otherwise the draft seems in good shape.

1) ISSUE: Ambiguity, Section 3.2(c):

I find the normative statement here to be ambiguous:

"PEs in the redundancy group MAY exchange only the Ethernet-Segment (ES) 
route (Route Type‑4) when ESI is configured on a Layer‑3 interface."

To what does "MAY" apply? What is the alternative? Does this mean that 
exchanging the ES route is optional? Or is this excluding exchanging 
other forms of route? Or something else?

2) ISSUE: Confusing algorithm in 4.3(2):

I find the expression:

"2. BDF(Es) = Sk| Weight(Es, Si) >= Weight(Es, Sk), and Weight(Es, Sk) 
 >= Weight(Es, Sj)."

confusing, even with the subsequent qualifier:

"BDF(Es) is defined as that PE with address Sk for which the computed 
Weight is the next highest after the Weight of the DF. j is the running 
index from 0 to N-1; i and k are selected values."

Can you find a clearer way to state this?