Re: [bess] Shepherd review comment draft-ietf-bess-mvpn-msdp-sa-interoperation-04

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

Hi Acee,

Thanks for clarifying that.

Hi Mankamana,

Please see zzh> below.

From: Acee Lindem (acee) <[e-mail]>
Sent: Sunday, April 12, 2020 7:53 AM
To: Mankamana Mishra (mankamis) <mankamis=[e-mail]>; [e-mail]
Cc: Bocci, Matthew (Nokia - GB) <[e-mail]>; [e-mail]
Subject: Re: [bess] Shepherd review comment draft-ietf-bess-mvpn-msdp-sa-interoperation-04

[External Email. Be cautious of content]

Hi Mankamana,

From: BESS <[e-mail]<mailto:[e-mail]>> on behalf of "Mankamana Mishra (mankamis)" <mankamis=[e-mail]<mailto:mankamis=[e-mail]>>
Date: Saturday, April 11, 2020 at 4:41 PM
To: "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>
Cc: "Bocci, Matthew (Nokia - GB)" <[e-mail]<mailto:[e-mail]>>, "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>
Subject: [bess] Shepherd review comment draft-ietf-bess-mvpn-msdp-sa-interoperation-04

Authors,
As part of BESS Shepherd review process, went over document. Its in good shape to move forward. But I have some comments .


·        Abstract : I think new directive is not to have any RFC reference in Abstract, so you can remove reference of RFC 6514 from abstract

As long as the referring text isn’t in the form of an IETF reference, i.e.,  [RFCXXXX], it is fine. In fact, this type of statement is required in a document that updates an RFC.

Thanks,
Acee


·        Terminology : Since it uses many terms from RFC 7761 (PIM), 6513 and 5614 . it may be good to state that familiarity with terms used in these RFC would be useful.

·        Introduction :

o   In introduction I see it talks about PE1 , PE2. But there Is no picture / topology in document . it may be useful to have one sample topology created .

o   Introduction does mention one statement about Extranet, but rest of the document does not have any mention of it. Do you want to add small paragraph which talks about what is expected in case of extranet ?

Zzh> Nothing specific to extranet. It’s just that policies for extranet can control how BGP MVPN SA routes are advertised/imported (independent of this interoperation procedure), and then imported MVPN SA routes can be turned into MSDP SA routes (independent of extranet).


·        Section 3:  Statement from this section “In that case, if the selected best MVPN SA route does not have the "MVPN SA RP-address EC" but another route for the same (C-S, C-G) does, then the best route with the EC SHOULD be chosen.” Does it not make sense to make it as MUST ?

Zzh> Making it MUST will require changes to BGP route selection procedures; besides, an incoming MVPN SA routes may not carry that EC and the receiving PE needs to handle that anyway, so I think SHOULD is appropriate here.
Zzh> Thanks!
Zzh> Jeffrey


Mankamana