Re: [bess] Genart last call review of draft-ietf-bess-mvpn-evpn-aggregation-label-10

["Jeffrey (Zhaohui) Zhang" <[e-mail]>]

Hi Russ,

I have posted the -11 revision, which addresses your comments and comments from others.

https://author-tools.ietf.org/iddiff?url1=draft-ietf-bess-mvpn-evpn-aggregation-label-10&url2=draft-ietf-bess-mvpn-evpn-aggregation-label-11&difftype=--html

Please let me know if you have any other comments.

Thanks!
Jeffrey


Juniper Business Use Only
-----Original Message-----
From: Jeffrey (Zhaohui) Zhang
Sent: Tuesday, July 11, 2023 10:43 PM
To: Russ Housley <[e-mail]>; [e-mail]
Cc: [e-mail]; [e-mail]; [e-mail]
Subject: RE: Genart last call review of draft-ietf-bess-mvpn-evpn-aggregation-label-10

Hi Russ,

Thanks for your review and comments.
Please see zzh> below.

-----Original Message-----
From: Russ Housley via Datatracker <[e-mail]>
Sent: Friday, June 30, 2023 11:34 AM
To: [e-mail]
Cc: [e-mail]; [e-mail]; [e-mail]
Subject: Genart last call review of draft-ietf-bess-mvpn-evpn-aggregation-label-10

[External Email. Be cautious of content]


Reviewer: Russ Housley
Review result: Almost Ready

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://urldefense.com/v3/__http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq__;!!NEt6yMaO-gk!Bn70GYYpMVzCajcXY6lg0iBcKkTYHdlqiTryvf4m2OjdKMVqmA7FDSbsFpOYzSz92OgNRiv14HmwByA$ >.

Document: draft-ietf-bess-mvpn-evpn-aggregation-label-10
Reviewer: Russ Housley
Review Date: 2023-06-30
IETF LC End Date: 2023-07-12
IESG Telechat date: unknown


Summary: Almost Ready


Major Concerns: None.


Minor Concerns:

Section 2.2: Please provide more discussion of "central authority".  I do not think that a Internet-wide authority is required to meet this requirement.  I think that the authority needs to be recognized throughout a provider network.

Zzh> Correct. I changed the first to "central entity in the provider network", and the rest to "central entity".
Zzh> I fixed the following nits as well - will post the revision after the quiescence period is over.

Zzh> Jeffrey

Nits:

Section 1: please avoid splitting "I/S-PMSI" across multiple lines.

Section 2: s/ingress PE's default label space/default label space/

Section 2: s/packet's ingress PE/ingress PE of the packet/

Section 2.2.2.1: s/Route-2 respectively/Route-2, respectively/

Section 3.2: s/PTA's Label field/label field of the PTA/

Section 3.2: s/source PE's label space/label space of the source PE/

Section 4: s/third party's label space/label space of a third party/ (2 places)

Section 4: please spell out "LFIBs"; this term is not in the definitions.