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

Hongji Zhao <hongji.zhao@ericsson.com> Tue, 05 May 2020 03:52 UTC

Return-Path: <hongji.zhao@ericsson.com>
X-Original-To: pim@ietfa.amsl.com
Delivered-To: pim@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 95B5D3A13AD; Mon, 4 May 2020 20:52:56 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.101
X-Spam-Level:
X-Spam-Status: No, score=-2.101 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id p-ft__u0F0qI; Mon, 4 May 2020 20:52:54 -0700 (PDT)
Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70040.outbound.protection.outlook.com [40.107.7.40]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 390613A13AA; Mon, 4 May 2020 20:52:53 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Pqx2yjhFv5By5KrQg1Z6AonNdlx29xdlnPbF6mPuMk5D+0y9sL37K2zq6XJne5UQcnO/36n3CMdgDx9fZjeXNZw+kdO/F61S7mzW0KjIuhEvfWl1RjqknmGb023lrJhdhGVcV9vvZcpyNh68gqtlVWwJzbe8tNK1lQCFgsuZskXLu/7X7UnGAaEP1V8L3W9MRWoq107T2PqI5nZhpPyaNL4oQ1hF6J6Bva6Fa7ZqfG/1WcxPIq5bSKQIkzYe8kjmAmQwB7JjfCJYAKx+XgiJUItAc5zxrIPwx3/e9PioYkLv/I7/YWqthWuEKLGhY6LU1ejDob9m3H2I7z/9mPsT+g==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ETaPbvouWSZJzxGf/nRlS8GWrh+pLY/FEMbtYEDL3Qs=; b=cRH6zyhfU+JWe4UmAckhsvAXmj++DI5We1m14azLs7X2teBHcEKwbsTE423Wi7b8JoSETrUHTvMNf4Yj/OJEs+N+Ootx/StREq7iqftHEX9IQUC/mDJwHEGPl+Z0Ywfw2z2cQg+f0K0UAmEkcl7gt4YoeZJS7le0ma/3w9GiYeo78aWuxURJqseCQW5bz4XMevZ0IhXBlKpwF7NoCi1g3uk/m58H2zrSha7xA8nrhPQ31nVRMJ6ohDh6JLq3E9+PEMitZslCxFdxtY6IQSB58SZbDid+EiKBQMHVfeM028YDus1OqbdjtisnxrmvbGh1Wg84WNbHlJaxAL7b1t+3xg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ETaPbvouWSZJzxGf/nRlS8GWrh+pLY/FEMbtYEDL3Qs=; b=HuAfvIa3Is/r3aEsfaorPadmnvwGSjZx8egiKpmNDjlnpxQzZTgkDEAxYfQSal7pUHYe4S7oY+XV5nqpD4zPhVWCKDNTdESPU8/rWKRhr4G3hwzFpo/Td72ArRIUuPx0XmDr6RrKlMRA+YsKMQbB7cApX7Sxm+J0Ms6mmclgiBo=
Received: from HE1PR0701MB3018.eurprd07.prod.outlook.com (2603:10a6:3:49::21) by HE1PR0701MB2651.eurprd07.prod.outlook.com (2603:10a6:3:96::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2979.18; Tue, 5 May 2020 03:52:49 +0000
Received: from HE1PR0701MB3018.eurprd07.prod.outlook.com ([fe80::c4a8:8c31:3473:f84e]) by HE1PR0701MB3018.eurprd07.prod.outlook.com ([fe80::c4a8:8c31:3473:f84e%4]) with mapi id 15.20.2979.025; Tue, 5 May 2020 03:52:48 +0000
From: Hongji Zhao <hongji.zhao@ericsson.com>
To: Alvaro Retana <aretana.ietf@gmail.com>
CC: "pim@ietf.org" <pim@ietf.org>, Stig Venaas <stig@venaas.com>, "pim-chairs@ietf.org" <pim-chairs@ietf.org>
Thread-Topic: AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09
Thread-Index: AQHVzYeKaXJzNYyt5k+R4mTeZys2XKfxTt9wgJkOjoCAAONp8IAAq3eAgAlyaJCABCbGYA==
Date: Tue, 5 May 2020 03:52:48 +0000
Message-ID: <HE1PR0701MB3018F4BF64EB02972A980BC396A70@HE1PR0701MB3018.eurprd07.prod.outlook.com>
References: <CAMMESsz-MvVF-2o2_Gso6yKocUKhRFrJTp0jsUwFpcznfj5qXw@mail.gmail.com> <HE1PR07MB3148BBF5316C9EC1FAC7742A96330@HE1PR07MB3148.eurprd07.prod.outlook.com> <CAMMESsztgWZr397ca6zVCc=9=Fq5B7ib64YmePXN7KNefA5zng@mail.gmail.com> <HE1PR07MB3148318D959984C817EC11E796AE0@HE1PR07MB3148.eurprd07.prod.outlook.com> <CAMMESszGOfmsWNuBWncAms4t5jMeVXvP_nQ1c4a-qxZNYUmQaA@mail.gmail.com> <HE1PR07MB31484BB0781DC5EB19D822DA96A80@HE1PR07MB3148.eurprd07.prod.outlook.com>
In-Reply-To: <HE1PR07MB31484BB0781DC5EB19D822DA96A80@HE1PR07MB3148.eurprd07.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=ericsson.com;
x-originating-ip: [120.244.62.243]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 543e01fa-8c2b-4f88-b9af-08d7f0a7c727
x-ms-traffictypediagnostic: HE1PR0701MB2651:
x-microsoft-antispam-prvs: <HE1PR0701MB2651338FFED98324A532D3AF96A70@HE1PR0701MB2651.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 0394259C80
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: fvN2+xiLGkEh8omF77lgInPx/O78ugB+9WEIojCb6dLqjJU9aEFVfs+7d10IcT4dMthOOaQQQttJMW0zifC+na0zmvGujxM/qZ9Dm3rXMTvccoewbZAGL02aGWWUmkHnTWCC3b2sUxyorVzJqUAQuBXqb4kUnHj5RpRl5KUHKFxac6K82QKsKuQULTKyVgtYGahzgT4iUIxX3qk+fa+VHZFPoCMpBBRdzJQpR5tsvryMGAnHuv//8NXNs9RT+nnToJGUtCq8EMydYosz/mo+3i6vi784zJzGDU/hPEiw2biTd0SSwFwjTP4lZtXZ481CH3UcPe7p85XIHXmB/2rkwxJsV7xzbRwsw+A4+rLRK3BChPONrdtqtm3ck7Yv42D8iZU3AqKfPH5yEmCsDXaQ2sbO7sP27MylTQntPTE7yBpkWCZ5+P7Ike8Bmk5O3WdwQNySWrkGZlAg0VcOxD+aNgiQs6l5dZS4lGe5f5erOfdf0UE+hwvgWUlHWR2vBzfixY0bDmOtKsQbV4t2b/nb2w==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:HE1PR0701MB3018.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(39860400002)(136003)(396003)(346002)(366004)(376002)(33430700001)(86362001)(54906003)(6916009)(7696005)(33440700001)(478600001)(52536014)(4326008)(44832011)(9326002)(66946007)(26005)(8676002)(71200400001)(8936002)(5660300002)(66556008)(66476007)(2906002)(64756008)(66446008)(186003)(53546011)(76116006)(33656002)(316002)(6506007)(55016002)(9686003); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: 2VbzH2yT07RBmQzatpIVO+aJpV+61gUGhqde2y3z9+WKK96nRXoyrn6NQbizYuzdyGhw0MbkctnPlrpahnRVbdbqYCg9LVyJggudLSLO2GiE58JsvrPWw9ebSy7lhOXFzpJz/adLVqy+SLscaL3TVwNbw38CZ6VUDMH5AUzZGSl/YvPFNMbefdX8O92VNcEhPqvP4jH3muMpsQxThFCG65A7OyZwut3UXbva0jMSqNc2djNLFHWm+rzc/gKM135+Vp+aJXp+t+b58ZD2SQFCCzZxMcTR87M8g1PANVkpwK5lUerHHrugnoMX9lyWpn5spwel+GZMua6+Fcy5w4QFM82VNNP96YZN1VD2GE1mtk4wDGej09U3Xtb/LtZotWE1YdCLlrU39HYpULSu8wn0uhS6fr/qoBryiBbmhfAVLSjjscXi81djfEdaEZNV78pJVqFNT81i2CYA88PpnOnL7CbLjpnux3YYAdZwV4U/odObiWbDIyBhthG6vdN5hOhknvWkvWgxgabhxB6Yo7aMBO8rzeolaGrAkx3hg7SROn9xnytKKkIrHe+6Lw2qHgEc4yS7IZ0If4kU5HySJPoUs8O8sbSawFYtEdB9MmKailaw5GJ2w3vBWNpptkytg/1ObcpxC9aTYxSa7TIQH0XyXFxLgdbwkmk4b/h/Dqb3Dd4hMxqzOLcF0JdaSECGqvdGbWmK6ITgCllmF48ru8TISgzi6jyDHKMOeYXY9Nagp6DnbFiYzL6CsxyCRDh+fzVf1kCwZlOe/P0NynJElv9H/PjOzdq8heZNKn7pe/2NCBw=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_HE1PR0701MB3018F4BF64EB02972A980BC396A70HE1PR0701MB3018_"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 543e01fa-8c2b-4f88-b9af-08d7f0a7c727
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 May 2020 03:52:48.6129 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: y5jRvPgi3V1oXcADCoyuek935DS0yWdTOZnc6+FQGkBD5UcTl5aDp5/d5yMuSYeu4SZKTmjqHhHiFGIKz641/tU1kcrSMYaJs4c8N97VEVQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0701MB2651
Archived-At: <https://mailarchive.ietf.org/arch/msg/pim/MSfHM7RozpyoyWCHXFylQFyjS2g>
Subject: Re: [pim] AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09
X-BeenThere: pim@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Protocol Independent Multicast <pim.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pim>, <mailto:pim-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pim/>
List-Post: <mailto:pim@ietf.org>
List-Help: <mailto:pim-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pim>, <mailto:pim-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 05 May 2020 03:52:57 -0000

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 <aretana.ietf@gmail.com>
Cc: pim@ietf.org; Stig Venaas <stig@venaas.com>om>; pim-chairs@ietf.org
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 <aretana.ietf@gmail.com<mailto:aretana.ietf@gmail.com>>
Sent: Sunday, April 26, 2020 8:11 PM
To: Hongji Zhao <hongji.zhao@ericsson.com<mailto:hongji.zhao@ericsson.com>>
Cc: pim@ietf.org<mailto:pim@ietf.org>; Stig Venaas <stig@venaas.com<mailto:stig@venaas.com>>; pim-chairs@ietf.org<mailto:pim-chairs@ietf.org>
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 (hongji.zhao@ericsson.com<mailto:hongji.zhao@ericsson.com>) wrote:
We will finish addressing the comments by the end of May. Is it ok?