Re: [pim] AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09

[Hongji Zhao <[e-mail]>]

Hi Alvaro,

When RFC 4541 is referenced as Normative reference,  it will raise following Downref error.
** Downref: Normative reference to an Informational RFC: RFC 4541

So I have to set it as informative reference.  Is it ok?  Or it still needs to be set as normative reference?

BR/Hongji

From: Hongji Zhao
Sent: Saturday, May 2, 2020 8:40 PM
To: Alvaro Retana <[e-mail]>
Cc: [e-mail]; Stig Venaas <[e-mail]>; [e-mail]
Subject: RE: AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09

Hi Alvaro,

We have already addressed all your comments and uploaded the latest version as draft-ietf-pim-igmp-mld-snooping-yang-10.
Below are the answers to your questions in your comments. Please check inline.

1)
175           augment /rt:routing/rt:control-plane-protocols/rt:control-plane-protocol:
176             +--rw igmp-snooping-instance {feature-igmp-snooping}?
177             |  +--rw scenario?                            snooping-scenario-type
178             |  +--rw enable?                              boolean
179             |  +--rw forwarding-mode?                     enumeration
180             |  +--rw explicit-tracking?                   boolean
{explicit-tracking}?
181             |  +--rw exclude-lite?                        boolean {exclude-lite}?
182             |  +--rw send-query?                          boolean
183             |  +--rw immediate-leave?                     empty {immediate-leave}?
184             |  +--rw last-member-query-interval?          uint16
185             |  +--rw query-interval?                      uint16
186             |  +--rw query-max-response-time?             uint16

[minor] Why are timer-value-* (rfc8294) not used?  Please consider using them for the variables above and others...
   [Authors] Regarding last-member-query-interval or query-interval, it should not be set infinity or not-set. So timer-value-* is not fit for it.

2)
    leaf igmp-version {
      type uint8 {
        range "1..3";
      }
      default 2;
      description "IGMP version.";
    }
[major] If the previous guess is correct...  What does this version mean?  Given that the default is 2, does it mean that only IGMPv2 packets should be snooped?  I would assume that deployments already supporting IGMPv3 are common...
  [Authors] The version here refers to the igmp version.  By default is 2, it means only IGMPv2 packets could be snooped. In rfc8652, igmp default version is also 2.   If IGMPv3 packets need to be snooped, it should be set 3. So does the MLD.

BR/Hongji

From: Alvaro Retana <[e-mail]<mailto:[e-mail]>>
Sent: Sunday, April 26, 2020 8:11 PM
To: Hongji Zhao <[e-mail]<mailto:[e-mail]>>
Cc: [e-mail]<mailto:[e-mail]>; Stig Venaas <[e-mail]<mailto:[e-mail]>>; [e-mail]<mailto:[e-mail]>
Subject: RE: AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09

Hi!

Yes, as long as there is a plan and progress is being made. :-)

Thanks!

Alvaro.


On April 25, 2020 at 10:06:03 PM, Hongji Zhao ([e-mail]<mailto:[e-mail]>) wrote:
We will finish addressing the comments by the end of May. Is it ok?