Re: [pim] AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09
Hongji Zhao <hongji.zhao@ericsson.com> Sat, 16 May 2020 08:40 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 658453A094C; Sat, 16 May 2020 01:40:14 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.202
X-Spam-Level:
X-Spam-Status: No, score=-0.202 tagged_above=-999 required=5 tests=[DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, 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 VVt0kn67Kvjz; Sat, 16 May 2020 01:40:11 -0700 (PDT)
Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05on2088.outbound.protection.outlook.com [40.107.21.88]) (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 0842C3A094B; Sat, 16 May 2020 01:40:10 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cF4+HltTQ04bzt1o+7aukO+RLNFnY3XVWVosa42pEVUS4el+C4u4hw/Ftn2+WLsI8IQMKWxAseK3eN1skOfDkbySs30+iotY7kxkWvh1fpmDQfxCTXeQ4ieHVev+pVjJRQK+MUfh4vVgtx/fKMMGNju3QlW5BXyTiv+kqet9f3QYXetoSaKbyd0ceHU5WqVpWnRMRYo4L1wmvHoFPKW8Wd4Wide/aAffjHudbwAqSm5eIU5t5lux+m1MRjO8Qd7DxQ+EaSRQjimmm2V2cR8/1PUfo5/x07iz39RJh98Z3yHoJrQc+QdkuyYyOK3+ax87k3YQw3HrpPHxaTf2TngbVg==
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=1tDHsjPusVapnuhVeNED6WbF5vphv9MIMUrbkd3zQEk=; b=jGDjqn28aJ7e4N1b90Ox6BukPgzQ6EBpTPl6TISm2oDo1equjIx6hEeCAQrSgZsSWANhYHdev3zQTIqs7DDoq/11HJqF1JpO4LNkROjYulC+qz726tp5O2AKPNNvSKLWa+Vy1kN4+T+hpjYXqw19HTlcA+566+vgCnYCC4MSW+cfgg5GQr6fcEH0ulrwWGIAbQkkUY7CMEbWxJUi84bFP6a0K/f0zIIRZZe6OBUpW35CAj+4ouT/4Xt63PBEsl7akZzrhW0uOJcdUuJaMYDAaC7MWDLAlskaGd+fO4EHifUMt6TlxX79McXf4sFGzapt0KU8WUtzzbKVYtHbECV0QQ==
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=1tDHsjPusVapnuhVeNED6WbF5vphv9MIMUrbkd3zQEk=; b=oFBxshrjrfSq66NEWocxolygnkA1oBxSAm4SYCxV0uGodc8euIbwQbAbnnXFBSVwZrQ/YReg0XL1jAJH2w3S3uzmiG3v+HhS55aeR6cbCuC1Enb0Z7Wg8jH/mocsUQOQ+t/GdTsB5rPeAlnThMIUENgCvQhgTNUF6rBIezwzt5o=
Received: from HE1PR0701MB3018.eurprd07.prod.outlook.com (2603:10a6:3:49::21) by HE1PR0701MB2713.eurprd07.prod.outlook.com (2603:10a6:3:98::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3021.12; Sat, 16 May 2020 08:40:07 +0000
Received: from HE1PR0701MB3018.eurprd07.prod.outlook.com ([fe80::cd0d:a67b:2882:319e]) by HE1PR0701MB3018.eurprd07.prod.outlook.com ([fe80::cd0d:a67b:2882:319e%8]) with mapi id 15.20.3021.013; Sat, 16 May 2020 08:40:06 +0000
From: Hongji Zhao <hongji.zhao@ericsson.com>
To: Alvaro Retana <aretana.ietf@gmail.com>
CC: Stig Venaas <stig@venaas.com>, "pim-chairs@ietf.org" <pim-chairs@ietf.org>, "pim@ietf.org" <pim@ietf.org>
Thread-Topic: AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09
Thread-Index: AQHVzYeKaXJzNYyt5k+R4mTeZys2XKfxTt9wgJkOjoCAAONp8IAAq3eAgAlyaJCADrhRAIAHCK1w
Date: Sat, 16 May 2020 08:40:06 +0000
Message-ID: <HE1PR0701MB3018E31B696DBA6F53DEDC0D96BA0@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> <CAMMESsxYCLQoWbtCB0R7bySExcmeOXexZFFCO2nyHJmz1kNRfw@mail.gmail.com>
In-Reply-To: <CAMMESsxYCLQoWbtCB0R7bySExcmeOXexZFFCO2nyHJmz1kNRfw@mail.gmail.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.163]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 799edce7-f30c-4df3-f4f1-08d7f974bc69
x-ms-traffictypediagnostic: HE1PR0701MB2713:
x-microsoft-antispam-prvs: <HE1PR0701MB271360C2DF3B86C128FE6B4296BA0@HE1PR0701MB2713.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:4714;
x-forefront-prvs: 040513D301
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: wV5sOowzr0I5raQEXOePtUD0ONAIJlSblMCvSvumyNf3OPovRzlt+5/q6wrpYh9bE3cpIeWZUshX/AL09tTf1zD90Di9WqWMCozjdBEzzyp3oDYnVM+7o6Yf/ic7XpSM6scFklvCZcXYP3B0H3WiPr5hUPKpyaZDaEOfUwBYo9yGTQsUay20epGxIDlBfhIXgI6aUXQbw6tI9dAo8NDTY2acb0tym3xS4mAtgKudXCnWYsbpgqmH5QL1sgbzG96gN8g+XVnfHH6pZPtw2rS13GU7fa9BNss6mK5QRVvHGE+BljARdr3LP2N1CMvZkWf0xXF6iA7WH9DZ0WdVn5f4RPxwm6nYmV8nEEesjKHJ7J9HRxnOXA21FyY/jaBk5IpbYbJEqiNrtH84kGIt+cTVuyFMZELU799ZIdjVGvjRdbT+39p/bQN2Ks+bRPbPNncg6uW3XqwAkdLaM51XRDBnxrC177tdlAZaLzF3gOlPT+M=
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)(136003)(346002)(366004)(376002)(396003)(39860400002)(6506007)(71200400001)(8676002)(8936002)(478600001)(6916009)(54906003)(86362001)(33656002)(4326008)(9686003)(316002)(5660300002)(26005)(53546011)(66574014)(186003)(7696005)(52536014)(55016002)(76116006)(2906002)(66556008)(66946007)(44832011)(64756008)(66446008)(66476007)(560514002)(357404004); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: S958Pe+8VBt+Xvt1B3ntMz41VmkvFsA7YoOWFIpHGIq5wCROzwxEo854Xw33klp/xMxkMkyeBEpgmtZR64JOrg97HvSIXWEPmNEgVKg5ocaPz7mIBZK1XZoyQmB7DNzNiGtQjKW6AFpuGJ90lzGTMkE4sDzQGqLZerDsnTtfYeMaEZvTN/bfF2g037g5/O+WAMx065otCyhXC1+6UD4jwQhY9ycmGQXlZ3qjAENfPLQJ+NF/Sc+1OF4LtEV5GM7kEkLfoKAE9cXiP7XI7V3whEQJAHskLyuVfzQgEx+eWmFXRic/+08DxlOW27k0+urDednFx9T2R33dqYF6r+3wKPdwSF0p5gOzeDCkI2NyEoZjT2THqve21LKPrsLro0WHKTQKp3HfzFllddfMc45RASSO5J204Tbrxa/F/vCLliv3EH0yoj5bIAb0/oc4l3gWhf+VRm3aDZFIdNE/L87JKOAkUGmac6TGgmbSWG7jlrHLCgtSfoHxCkAAIq1wKNyH
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 799edce7-f30c-4df3-f4f1-08d7f974bc69
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 May 2020 08:40:06.7418 (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: 1LB/eWVzhZQ7IuG/Y9O8KthFf4ihnD3oJghCinjQyMtVevu+zCSlAGoW1BG5kBkqeuRH/HG9SJ4ngdxAKrMP7iMMA6FEYfW0sgu1iDFWgsA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0701MB2713
Archived-At: <https://mailarchive.ietf.org/arch/msg/pim/OuyzkSvBVMLHm02xktVfGebnTHg>
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: Sat, 16 May 2020 08:40:14 -0000
Hi Alvaro, Thanks a lot for your comments. We will address them ASAP and post the new version. BR/Hongji -----Original Message----- From: Alvaro Retana <aretana.ietf@gmail.com> Sent: Tuesday, May 12, 2020 5:14 AM To: Hongji Zhao <hongji.zhao@ericsson.com> Cc: Stig Venaas <stig@venaas.com>; pim-chairs@ietf.org; pim@ietf.org Subject: RE: AD Review of draft-ietf-pim-igmp-mld-snooping-yang-09 On May 2, 2020 at 8:39:48 AM, Hongji Zhao wrote: Hongji: Hi! How are you? > We have already addressed all your comments and uploaded the latest > version as draft-ietf-pim-igmp-mld-snooping-yang-10. First, to answer your question about rfc4541: Yes, it must be a Normative reference as that is where snooping is defined. We will take care of the DownRef registry as we get the document approved. I need this reference in the right place so that the IETF Last Call call its out. Along with that, I have some comments on -10 (see below). Thanks! Alvaro. ... 134 1.2. Tree Diagrams 136 Tree diagrams used in this document follow the notation defined in 138 [RFC8340]. [nit] There's an extra line above... ... 180 +----------+-----------------------+---------------------------------+ 182 | l2vpn | ietf-l2vpn | [draft-ietf-bess-l2vpn-yang] | 183 +----------+-----------------------+---------------------------------+ 185 | dot1q | ieee802-dot1q-bridge | [dot1Qcp] | 187 +----------+-----------------------+---------------------------------+ 189 Table 1: Prefixes and Corresponding YANG Modules [nit] This table seems to also have extra lines. 191 2. Design of Data Model [major] It may be confusing to other readers the reason for not augmenting, or even requiring, the IGMP/MLD model (rfc8652). Please include some text to explain the relationship between snooping and the IGMP/MLD protocols themselves. Specifically, the fact that the switches don't really need to run the protocols. ... 197 In recent years, a number of commercial vendors have introduced products 198 described as "IGMP snooping switches" to the market. These devices do 199 not adhere to the conceptual model that provides the strict separation 200 of functionality between different communications layers in the ISO 201 model, and instead utilize information in the upper level protocol 202 headers as factors to be considered in processing at the lower levels 203 [RFC4541]. [major] This paragraph is a copy from rfc4541...which was written in 2006, so the "in recent years" phrase doesn't really apply anymore. Please remove it. 205 IGMP Snooping switches utilize IGMP, and could support IGMPv1, IGMPv2, 206 and IGMPv3. IGMP snooping switches may maintain forwarding tables based 207 on either MAC addresses or IP addresses [RFC4541]. MLD Snooping switches 208 utilize MLD, and could support MLDv1 and MLDv2. [major] This paragraph is a great opportunity to include appropriate references for all the protocols above. If they are not Normative already, any new references can be Informative. [minor] "IGMP snooping switches may maintain forwarding tables based on either MAC addresses or IP addresses [RFC4541]." This sentence also comes directly from rfc4541. Do we need it? What about MLD Snooping? Given that the model includes a mac-address for both the IGMP and MLD instances, it seems to me that we can simply take it out to avoid confusion. ... 229 2.2. Optional Capabilities 231 This model is designed to represent the capabilities of IGMP and MLD 232 switches with various specifications, including the basic capability 233 subsets of IGMP and MLD Snooping. The main design goals of this document 234 are that the basic capabilities described in the model are supported by 235 any major now-existing implementation, and that the configuration of all 236 implementations meeting the specifications is easy to express through 237 some combination of the optional features in the model and simple vendor 238 augmentations. [minor] This is not the IGMP/MLD module: s/This model is designed to represent the capabilities of IGMP and MLD switches with various specifications, including the basic capability subsets of IGMP and MLD Snooping./This model is designed to represent the basic capability subsets of IGMP and MLD Snooping. ... 258 2.3. Position of Address Family in Hierarchy ... 271 * The structure is consistent with other YANG data models such as 272 [RFC8344], which uses separate branches for IPv4 and IPv6. [minor] s/[RFC8344]/[RFC8652] This would be a more appropriate justification. ... 540 3.3. Using IGMP and MLD Snooping Instances ... 550 It also augments /dot1q:bridges/dot1q:bridge/dot1q:component/ 551 dot1q:bridge-vlan/dot1q:vlan to use igmp-snooping-instance. It means 552 IGMP Snooping is enabled in the certain VLAN of the bridge. [nit] s/in the certain VLAN of the bridge/in the specified VLAN on the bridge 554 augment /dot1q:bridges/dot1q:bridge: 555 +--rw igmp-snooping-instance? igmp-mld-snooping-instance-ref 556 +--rw mld-snooping-instance? igmp-mld-snooping-instance-ref 557 augment /dot1q:bridges/dot1q:bridge/dot1q:component 558 /dot1q:bridge-vlan/dot1q:vlan: 559 +--rw igmp-snooping-instance? igmp-mld-snooping-instance-ref 560 +--rw mld-snooping-instance? igmp-mld-snooping-instance-ref [nit] Add a space between the 2 augmentations...for readability. ... 594 4. IGMP and MLD Snooping YANG Module 596 This module references 597 [RFC2236],[RFC3376],[RFC3810],[RFC4286],[RFC4541],[RFC4604],[RFC4607], 598 [RFC6020],[RFC6241],[RFC6636],[RFC6991],[RFC7950],[RFC8040],[RFC8342], 599 [RFC8343],[RFC8340],[RFC8529],[RFC8652],[dot1Qcp], and [draft-ietf-bess- 600 l2vpn-yang]. [major] RFC2236, RFC3810, RFC4286, RFC4604, RFC4607, RFC6020, RFC6241, RFC7950, RFC8040, RFC8342. RFC8340 and RFC8652 are not referenced *inside* the model. Please don't include them in this sentence. 602 <CODE BEGINS> file ietf-igmp-mld-snooping@2020-04-29.yang ... 1707 <CODE ENDS> 1708 5. Security Considerations [nit] Add an extra line for readability. ... 1730 /rt:routing/rt:control-plane-protocols 1732 /rt:control-plane-protocol:/ims:igmp-snooping-instance 1734 /rt:routing/rt:control-plane-protocols 1736 /rt:control-plane-protocol:/ims:mld-snooping-instance [minor] The locations can be summarized: Under /rt:routing/rt:control-plane-protocols/rt:control-plane-protocol:/ ims:igmp-snooping-instance ims:mld-snooping-instance Please apply the same format to other entries in this section. ... 1791 6. IANA Considerations ... 1804 Registrant Contact: The IESG. [major] s/IESG/IETF ... 1826 7.1. Normative References 1828 [dot1Qcp] Holness, M., "IEEE 802.1Qcp-2018 Bridges and Bridged 1829 Networks - Amendment: YANG Data Model", 2018. [major] This reference is not complete. Let's try this: [dot1Qcp] IEEE, "Standard for Local and metropolitan area networks--Bridges and Bridged Networks--Amendment 30: YANG Data Model", IEEE Std 802.1Qcp-2018 (Revision of IEEE Std 802.1Q-2014), September 2018, <https://ieeexplore.ieee.org/servlet/opac?punumber=8467505> ... 1872 [RFC7951] L. Lhotka, "JSON Encoding of Data Modeled with YANG", RFC 1873 7951, August 2016. [minor] This reference can be Informative. ... 1920 7.2. Informative References ... 1926 [RFC4541] M. Christensen, K. Kimball, F. Solensky, "Considerations 1927 for Internet Group Management Protocol (IGMP) and Multicast 1928 Listener Discovery (MLD) Snooping Switches", RFC 4541, May 1929 2006. [major] This reference must be Normative as that is where snooping is described. We will take care of the downref registry after the document is approved.
- [pim] AD Review of draft-ietf-pim-igmp-mld-snoopi… Alvaro Retana
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Hongji Zhao
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Alvaro Retana
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Hongji Zhao
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Alvaro Retana
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Hongji Zhao
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Hongji Zhao
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Alvaro Retana
- Re: [pim] AD Review of draft-ietf-pim-igmp-mld-sn… Hongji Zhao