Subject: Re: [pim] Benjamin Kaduk's Discuss on draft-ietf-pim-igmp-mld-snooping-yang-17: (with DISCUSS and COMMENT)

From: Hongji Zhao <[email-masked]>
Date: Sat, 15 Aug 2020 04:18:10 +0000

Hi Benjamin,

I have addressed your comments and post the latest version as draft-ietf-pim-igmp-mld-snooping-yang-18.  Thanks a lot!

Btw, The last-reporter shows one IGMP host, which has sent either an unsolicited IGMP Join or IGMP Report for that particular group.   Most vendors have it implemented.

BR/Hongji

-----Original Message-----
From: Benjamin Kaduk <[email-masked]> 
Sent: Tuesday, July 14, 2020 11:40 AM
To: Hongji Zhao <[email-masked]>
Cc: The IESG <[email-masked]>; [email-masked]; [email-masked]; [email-masked]; Stig Venaas <[email-masked]>; [email-masked]
Subject: Re: Benjamin Kaduk's Discuss on draft-ietf-pim-igmp-mld-snooping-yang-17: (with DISCUSS and COMMENT)

Hi Hongji,

Yes, that looks like it will work quite well in terms of indicating what behavior will result.  (I do not have enough YANG experience to comment on whether this usage of a "type empty" is conventional, but trust that the responsible AD will get additional review if needed.)

Thank you,

Ben

On Tue, Jul 14, 2020 at 03:04:49AM +0000, Hongji Zhao wrote:
> Hi Benjamin,
> 
> Thanks a lot for your comments. 
> 
> Regarding your discuss,  we plan to modify exclude-lite leaf as below. 
> Is it ok? Thanks.
> 
> leaf lite-exclude-filter {
>       if-feature lite-exclude-filter;
>       type empty;
>       description
>            "For IGMP Snooping, the presence of this
>             leaf enables the support of the simplified EXCLUDE filter 
>             in the Lightweight IGMPv3 protocol, which simplifies the
>             standard versions of IGMPv3.
>             For MLD Snooping, the presence of this
>             leaf enables the support of the simplified EXCLUDE filter
>             in the Lightweight MLDv2 protocol, which simplifies the
>             standard versions of MLDv2.";
>       reference
>            "RFC 5790: Lightweight Internet Group Management Protocol
>             Version 3 (IGMPv3) and Multicast Listener Discovery
>             Version 2 (MLDv2) Protocols";
>     }
> 
> BR/Hongji
> 
>