Re: [pim] Last Call: <draft-ietf-pim-igmp-mld-proxy-yang-07.txt> (A Yang Data Model for IGMP/MLD Proxy) to Proposed Standard

[tom petch <[e-mail]>]

On 15/11/2022 02:47, Hongji Zhao wrote:
> Hi Tom,
> 
> Thanks a lot for your comments. Please check inline.
> 
> BR/Hongji
> 
> -----Original Message-----
> From: tom petch <[e-mail]>
> Sent: 2022年11月14日 20:06
> 
> On 25/10/2022 20:27, The IESG wrote:
>>
>> The IESG has received a request from the Protocols for IP Multicast WG
>> (pim) to consider the following document: - 'A Yang Data Model for IGMP/MLD Proxy'
>>     <draft-ietf-pim-igmp-mld-proxy-yang-07.txt> as Proposed Standard
>>
>> The IESG plans to make a decision in the next few weeks, and solicits
>> final comments on this action. Please send substantive comments to the
>> [e-mail] mailing lists by 2022-11-15. Exceptionally,
>> comments may be sent to [e-mail] instead. In either case, please
>> retain the beginning of the Subject line to allow automated sorting.
>>
> 
> Last time I got this wrong but I believe that 225.0 is not appropriate for documentation use.  It should be 233.252.0/24 as per RFC5771.acc
> [Authors] Accepted
> 
> In places, the spacing is double line spaced (which has appeared in
> other recent I-Ds).    I see this in parts of s.5 and s.6.1 and in thee
> YANG Editors.
> [Authors] Accepted
> 
> Simplified BSD should now be Revised BSD.
> [Authors] Accepted
> 
> YANG has a 'should not' but no reference to RFC8174.
> [Authors] YANG has a 'must not' constraint, shall I use 'must not' in the description instead of 'should not'?
>                    RFC8174 clarify the UPPERCASE usage of the key words, but I would like to use lowercase. 
   Is it necessary to reference it?

Using lower case is fine if that is what you want to do and in such a
case I see no reference to RFC8174 as needed.  I flagged it as something
that is often raised as a query by the IESG but if it is the consensus
of the WG then that is ok.  This is a corner case since the text is in a
message and if you make that UPPER case then users not familiar with the
quirks of the IETF may wonder what is going on!

Tom Petch

> 
> 'membership report or leave' would be clearer to me with another word or two, such as 'message'
> [Authors] Accepted
> 
> Tom Petch
> 
>> Abstract
>>
>>
>>      This document defines a YANG data model that can be used to
>> configure and manage Internet Group Management Protocol (IGMP) or
>> Multicast Listener Discovery (MLD) proxy devices. The YANG module in
>> this document conforms to Network Management Datastore Architecture
>> (NMDA).
>>
>>
>>
>>
>> The file can be obtained via
>> https://datatracker.ietf.org/doc/draft-ietf-pim-igmp-mld-proxy-yang/
>>
>>
>>
>> No IPR declarations have been submitted directly on this I-D.
>>
>>
>>
>>
>>
>> _______________________________________________
>> IETF-Announce mailing list
>> [e-mail]
>> https://www.ietf.org/mailman/listinfo/ietf-announce
>> .
>>
> .
>