Re: [Bier] Warren Kumari's No Objection on draft-ietf-bier-evpn-11: (with COMMENT)

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

Hi Warren,

Thank you for your review and comments. I have posted the -12 revision that addresses comments from you, John, and Zaheduzzaman: https://author-tools.ietf.org/iddiff?url1=draft-ietf-bier-evpn-11&url2=draft-ietf-bier-evpn-12&difftype=--html

Please see zzh> below for some clarifications.


Juniper Business Use Only
-----Original Message-----
From: Warren Kumari via Datatracker <[e-mail]>
Sent: Wednesday, November 29, 2023 3:07 PM
To: The IESG <[e-mail]>
Cc: [e-mail]; [e-mail]; [e-mail]; [e-mail]; [e-mail]
Subject: Warren Kumari's No Objection on draft-ietf-bier-evpn-11: (with COMMENT)

[External Email. Be cautious of content]


Warren Kumari has entered the following ballot position for
draft-ietf-bier-evpn-11: No Objection

When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.)


Please refer to https://urldefense.com/v3/__https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/__;!!NEt6yMaO-gk!Hwe2UdCIQYeKPcW6IM38LRuj53Ysj6qTK74LW1JCCWdITBbvAe27V4lisuoJFQrFnGxVZganKe_jI7Y$
for more information about how to handle DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://urldefense.com/v3/__https://datatracker.ietf.org/doc/draft-ietf-bier-evpn/__;!!NEt6yMaO-gk!Hwe2UdCIQYeKPcW6IM38LRuj53Ysj6qTK74LW1JCCWdITBbvAe27V4lisuoJFQrFnGxVZganEvjzGxk$



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Thank you for writing this document; I found it an interesting read.

I have a few comments / readability suggestions:

Section 3. Multihoming Split Horizon
"For EVPN-MPLS, [RFC7432] specifies the use of ESI labels to identify the ES from which a BUM packet originates. A PE receiving that packet from the core side will not forward it to the same ES."

It is unclear what exactly "that packet" refers to - from my (limited) understanding the packets themselves don't carry ESI labels.

Zzh> when a packet from a multi-homing ES is forwarded by the receiving PE1 to other PEs, say PE2, an ESI label is indeed added. When PE2 receives that packet from the core side, from the ESI label PE2 knows it is from the ES that PE2 is also on, so PE2 does not forward it back to the ES.

"For EVPN-VXLAN/NVGRE/GENEVE, [RFC8365] specifies local bias procedures, with which a PE receiving a BUM packet from the core side knows from encapsulation the ingress PE so it does not forward the packet to any multihoming ESes that the ingress PE is on, because the ingress PE already forwarded the packet to those ESes, regardless of whether the ingress PE is a DF for those ESes."

This sentence is somewhat of a run-on. Perhaps moving the "from encapsulation"
much earlier would help? Or perhaps splitting this into multiple sentences would help?

zzh> I split it to multiple sentences.
Zzh> Thanks!
Zzh> Jeffrey