Re: [pim] Robert Wilton's Discuss on draft-ietf-pim-igmp-mld-proxy-yang-08: (with DISCUSS and COMMENT)
Hongji Zhao <hongji.zhao@ericsson.com> Tue, 06 December 2022 04:17 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 3A5E1C14CF13; Mon, 5 Dec 2022 20:17:03 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level:
X-Spam-Status: No, score=-2.098 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, RCVD_IN_DNSWL_BLOCKED=0.001, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=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 ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id tqj-Y2eV_BoK; Mon, 5 Dec 2022 20:16:58 -0800 (PST)
Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-vi1eur04on2070.outbound.protection.outlook.com [40.107.8.70]) (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 039ADC14CF10; Mon, 5 Dec 2022 20:16:57 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=T/vDwStJ7XQdY+yx3U7f6FA/L6+xvNP2bUdkVpt/GJYidBtZr1HBcOkVoZ7w6x3ILljNe2KWIajL9TzNKr0maKPxSqSv7A7nFFJalv+8drbfHq1YcmVlqj23nCmciP0QGxAOVCvPXDqnWCviYEGHl/jqVlQcH0BhSOvOQLGjQK8SIwsM9voq4N2MgUMF9JxolaxpOimSLFxKdUXWfBrFtG27UxMRJ54+QDARPJmCmE9bLBDIKopbsp2EC+Fyy4WnrVyk+VMT4vVexclmHCjuvH+8Kw6fq8c/iWapyJB3tPSOZOVXmzst+jbVtiQjonYW7Ck16SGMBeXhyyUq9Rjxwg==
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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=kh5c2dIv7FyRY1ZfrsCbS4eAVtX3N+y7XAK/lKowENs=; b=e4aqWHvx7jIusN1j/LC2X6tQcL/6GMnGGw0949UCL6EIYQIEQVNasTPhVCygPMapRJwcewJbWdEdJvKYKVx3/5NIzTYF8STRUg9ZHM1ixO2jOA7nTcZpxZcpvkH7SMbuXgpYOI8Gwu9NmFDJHsWVOtAoUsaOYxYpP2DN+16SnkvWemuswYFqjaas2JShf8+CVLcJpOQJYJnZjrG8c135VcEX+blhYXZ8M+flfqg5om/uF2NWSzrDzLvqpKCdlQzdznta6R/UnrdRgQS4yzIwJM9+zyS7wRoqs6aimDYIgXpgEQu1T1RAMtn8GjbiFllmwrkPDzb+oW84e0ShLyOvfg==
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=kh5c2dIv7FyRY1ZfrsCbS4eAVtX3N+y7XAK/lKowENs=; b=e1ETTGI0fpOSgdGFigzD5hmKt99PpZzNx3PSVx75ttDvh7kpXUQ1LCRs37FDI+a2/Im9AJYernKDFAyet7HbOChvh53BJ05a7lSZk0ivV7fqNnjcngh/2EMKYtIOmNJjVt064PRSabmZaF+EB6GywS3374lMSQJBGpcpK88JFmc=
Received: from AS2PR07MB8979.eurprd07.prod.outlook.com (2603:10a6:20b:556::7) by DB9PR07MB8702.eurprd07.prod.outlook.com (2603:10a6:10:30d::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5880.13; Tue, 6 Dec 2022 04:16:53 +0000
Received: from AS2PR07MB8979.eurprd07.prod.outlook.com ([fe80::fda8:e5bb:89e4:3157]) by AS2PR07MB8979.eurprd07.prod.outlook.com ([fe80::fda8:e5bb:89e4:3157%8]) with mapi id 15.20.5880.014; Tue, 6 Dec 2022 04:16:53 +0000
From: Hongji Zhao <hongji.zhao@ericsson.com>
To: "Acee Lindem (acee)" <acee@cisco.com>, "Rob Wilton (rwilton)" <rwilton@cisco.com>, The IESG <iesg@ietf.org>
CC: "pim-chairs@ietf.org" <pim-chairs@ietf.org>, "pim@ietf.org" <pim@ietf.org>, "draft-ietf-pim-igmp-mld-proxy-yang@ietf.org" <draft-ietf-pim-igmp-mld-proxy-yang@ietf.org>
Thread-Topic: [pim] Robert Wilton's Discuss on draft-ietf-pim-igmp-mld-proxy-yang-08: (with DISCUSS and COMMENT)
Thread-Index: AQHZBYnkjl9p+VncTkq/EMrPzASoca5ZGxOAgAcslsA=
Date: Tue, 06 Dec 2022 04:16:53 +0000
Message-ID: <AS2PR07MB8979AC3C46ED8E7FEDA9EFE9961B9@AS2PR07MB8979.eurprd07.prod.outlook.com>
References: <166990176748.52357.11679071816887044251@ietfa.amsl.com> <4E0F04B3-2AD8-42B4-8013-CD7421BB104C@cisco.com>
In-Reply-To: <4E0F04B3-2AD8-42B4-8013-CD7421BB104C@cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=ericsson.com;
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: AS2PR07MB8979:EE_|DB9PR07MB8702:EE_
x-ms-office365-filtering-correlation-id: d6a73a48-3a9d-49d3-b8f0-08dad740b487
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: /jIfKJb8Hkj1Gawp1xdHp7jmDMO+ZlOhhMiewwfqZoxklMJZ6MOlDqd+kEaFM2uuUF4DR5zSUzI/+leOpmj8rGKQBWxIpKFISKZxtsb+zym7o1mGoXx7+iMy2VzBEcrs+IC82G8gldPQqIYCgxsVDSzR01umgVYF1nNIMGc75VcrcJlWBXHr8zMFngHjjRDUd7nicw7Q7zuEahDPs54tY3BwzDei9nLJWcxRwzSYtRWJEoCvmSyBM1XQP1GioIG0eqd6Jy9Bb8pohS7bHzpa5nUXgg1v/XMlx2UCMwTCkaR6Y8ZyngZYEq9Bh2AKghmzENeO6qHeLrPe3vHQw3GmDkl2WyioMftCIxWUu92GvlyGCOHwS9pTS+Dy5xfulqIaa0nhJLy5ak/TI+ZT3KSHfWOVm8dCPcl41zo1TRaVHMm/XeHrMNmi5KB7BW9yTpo21D2tpqIv86oAy7a4+pcJmpaTS8ajIZ0hZDDnJmUAxnCkIBkqOA1+Rh8+2xzWOgy/j1B2IxcG0gJgAxjVS8ObEd5NVHTRTKWAwFCOjGGhEizqS7PTCJyB3m2ZNkPQ8hxnuNL+lrDjgPlq4SWKIF1F0LIqt7XWTMBvDAXp4/+zaloKRxzI201RtYpd2kiDu+lLcsAIUgftZD6WWkJFdSNNoCGB3gmOLiC3Opm3p8DhWyXcQ6fj6/NtkVJmdb2bl3DsWKaUPqXeV6O4AhC8T7sEJlpeUej/mEkOToEsnYOw4vN++1dPU1yxqx3J1wTAv4qludvsyqxebeUYHLwHJPb+qw==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AS2PR07MB8979.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(4636009)(346002)(366004)(136003)(376002)(39860400002)(396003)(451199015)(7696005)(316002)(26005)(53546011)(966005)(6506007)(186003)(110136005)(9686003)(54906003)(33656002)(55016003)(2906002)(86362001)(83380400001)(66556008)(64756008)(66476007)(4326008)(38100700002)(66446008)(8676002)(122000001)(76116006)(66946007)(82960400001)(71200400001)(41300700001)(38070700005)(478600001)(52536014)(8936002)(44832011)(5660300002); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: jWMKYVziNmXKk5OIcCVKmPHor7ENzlHrXL+eAWkhr7mhz1xpVxkv76v9L6Hho7CED0HEXqA+Qvo9Tk9JSkhhcscQS2U047UEu0B87XobR5lf0FeaWUuPcbrYmcHDOOF/mHM2xhLrWC4UGIdsI+7oVD52kiHp1eDNVSCJgwKecQ4XLCQBd0P05e4jEFFCQvA98AfqGOgWEJZjF9GkeyqlwKIVNb7Nai0KAJhY6p9RnNuvYFyF3ECE9o4U3RwGZlDKHOhNayH0mBw+uHBSTr5wgRoeO0ymgRsj9uokVbV1ZSyLrhtQyT2z7M9oKYEZ/X/951peR0aJQhDiYUr8jiX1k2ecrvEx5G98FQfNs/1aSZIfbXNZTbzOodfDQT5zOSO2p6gMg6dSwwyvLWQYGUSpNCga+y333BdunF4wDX6dt4WKEqzIpRXcyRG83OQrVOHLhofX5niEi0ufD4EwOZVtap0xN/gelTnuRFYC2Im4ijKLQIe5V37PQRXKpSx4yoni9zUu6TvAPpcPxVo8MVZu8dGOM8ya/2G5olSoQuNhBh2bygHOnGS+ctaHnqj9Ygpen4l0NuMOQkpqNfj5HQVTfMzvP9ZPgDDKxx5v2Hj+cPXzhwxc630dd9ZznaSIZMaAQw2JqJVpBj18UXHeYArL391JyFsQxRqL0rGpel/raa4ywTLYc1PAwpYUYzECZMmLXZ0vz7xlGlvTJKO5PklQtuPJ6pA/yAoiwOZTc+bHeDvSvnXYtKdk2zHT26a5eN15Te+Pzwi0NRKTPx2J2sSMzo024ZcEDMDGuCxhUGqyd3Zwkrw2Y6k+Prf1FPZKE/ebt/g1hBMakHvdYODzGR2Zzfm4lq1Kt+Xpzm0hDPH0aCumqf7rAXMtIOvGykLaLdYzqu7id6flZfw5NLRtBktnllLhSy5QavN+Ms8ZyYjoII9OZXbsfca9QSs9inlORPAEITQarRQqqs+0QxB3onaHsCEgSPddZx2rAFrcimc9IQ3OqBhZ6vl1o5a/qY6vlRZLNI4e41TibWjklBuRH4VScPXDPF0nicobhFiyXnXjMw43k2t3z/thiYeTCrozcfZZgL8/xMx+6xnBkmX5RH6LAsXPNP5KQnb59yOeGfIW+ncmjazEBGfzLRzzE+yH4kVFvkIhCafUTmfiOUHXfcLIyGan7pTgfUAcMwbHseNVjOqYym9Po8dMf6cbVIKdTb6xIknvqeiQFuV//yS6lQB/09us+hWL4xyS8pCNneAQTsM/X/oXItauaMEGL9six+u8w64UmRqJgyPbOSZbIX8xbpXD6hZE+ehOMNbaDCjxxhe1pZuSf2FtGZPtjaGFmbEq0Lm3iFU2Od+8kfx9D532NU5vD+x5/VfYKBE2CdXhNs3Ii7fou7cC0tz74DT/jkg6V2TaFH9U8IPbbo9tTGJWeTus8W9XYeAeb/oHZ1Z2yXXD6ujlxbvjMrq1oAWARs27Bc5PMU2ZPwHkFXjRiebH8Caf+lT2moHHjvVT+w6wk3r/G50aZM6xErVPSECrJrDwbC6WFDbDkcZtYEul8W70P/Kfz0zeO/J5BP/9DL9D/sOJS0JUpykO3jMWDXV8PdU1
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AS2PR07MB8979.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d6a73a48-3a9d-49d3-b8f0-08dad740b487
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Dec 2022 04:16:53.2706 (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: +WK0zUCu9uok8sbIFxt/bOIxZaHQonk+IDogIk/wRB9QPLbi/qJTYp7k1A+CIyWGkjcTtjnNrl6TDm/OSPkoCrDB1GYzPLyD8kOiLx9oyww=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR07MB8702
Archived-At: <https://mailarchive.ietf.org/arch/msg/pim/yQKY9wCxBxvOqbIJ25zftjqagvM>
Subject: Re: [pim] Robert Wilton's Discuss on draft-ietf-pim-igmp-mld-proxy-yang-08: (with DISCUSS and COMMENT)
X-BeenThere: pim@ietf.org
X-Mailman-Version: 2.1.39
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, 06 Dec 2022 04:17:03 -0000
Hi Acee, In order to be consistent with other IETF models, We will change "enable" to "enabled". Thanks a lot. BR/Hongji -----Original Message----- From: Acee Lindem (acee) <acee@cisco.com> Sent: 2022年12月1日 22:42 To: Rob Wilton (rwilton) <rwilton@cisco.com>; The IESG <iesg@ietf.org> Cc: pim-chairs@ietf.org; pim@ietf.org; draft-ietf-pim-igmp-mld-proxy-yang@ietf.org Subject: Re: [pim] Robert Wilton's Discuss on draft-ietf-pim-igmp-mld-proxy-yang-08: (with DISCUSS and COMMENT) Hi Rob, See one inline. On 12/1/22, 8:36 AM, "pim on behalf of Robert Wilton via Datatracker" <pim-bounces@ietf.org on behalf of noreply@ietf.org> wrote: Robert Wilton has entered the following ballot position for draft-ietf-pim-igmp-mld-proxy-yang-08: Discuss When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ for more information about how to handle DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-pim-igmp-mld-proxy-yang/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- Hi, My discuss covers two of comments below that I would like to have some discussion on to resolve (further details are in my comments below): (1) The addition/default of the "enable" leaf. Do you recall our discussions on the routing protocols models? We decided on "enabled" rather than "enable". I was in favor of the latter but was in the minority and we'd already published models using the former. Thanks, Acee (2) Name of the interface_name list key rather than just name. Regards, Rob ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Hi, Thanks for this document. I note the formatting of the .txt file is strange (e.g., my commenting script isn't able to parse it because it seems to have extra line breaks in unexpected places). I also note that there is no XML file uploaded. So, sorry but my comments are in a slightly more raw format! 1. Introduction: The "Network Management Datastore Architecture" (NMDA) adds the ability to inspect the current operational values for configuration, allowing clients to use identical paths for retrieving the configured values and the operational values. ## I don't think that this paragraph is required. Other IETF YANG RFCs just cite NMDA. 2. Design of Data Model This document provides freedom for vendors to adapt the data model to their product implementations. For example, some vendors could support configuring IGMP Robustness Variable under the interface which enabled IGMP Proxy. They could make their own augmentation. ## I'm not sure that this paragraph is necessary, since this applies to all YANG models and isn't specific to what is defined here. 2.2. Optional Capabilities There is also value in widely supported features being standardized, to provide a standardized way to access these features, to save work for individual vendors, and so that mapping between different vendors' configuration is not needlessly complicated. Therefore, this model declares a number of features representing capabilities that not all deployed devices support. # I don't think that this paragraph is accurate for the YANG contained in this draft. The model below only defines two features, one covering IGMP Proxy and one covering MLD Proxy. The extensive use of feature declarations should also substantially simplify the capability negotiation process for a vendor's IGMP / MLD Proxy implementations. # Again, I don't think that this paragraph is accurate and should be removed. The YANG data model defined in this document conforms to the Network Management Datastore Architecture (NMDA) [RFC8342]. The operational state data is combined with the associated configuration data in the same hierarchy [RFC8407]. ## I think that this paragraph is redundant and can be removed. The igmp-version represents version of IGMP protocol, and default value is 2. ### represents version -> represents the version, and default -> and the default If the value of enable is true, it means IGMP Proxy is enabled. ## I would make this a separate paragraph. I'm also not entirely sure why we need this leaf, since I would expect IGMP proxy to be enabled on an interface simply because an interface entry exists in the list. Please can you give me an explanation as to why it needed, and if it is needed whether it would be better to default to true rather than false. leaf interface-name { type if:interface-ref; must "not( current() = /rt:routing"+ "/rt:control-plane-protocols/pim-base:pim"+ "/pim-base:interfaces/pim-base:interface"+ "/pim-base:name )" { description "The upstream interface for IGMP proxy must not be configured to use PIM."; } description "The upstream interface name."; # It would be more consistent with other IETF models to just use "name" for the interface name. Is there a good reason to not be consistent here? Both here, and for downstream-interface and also for MLD. _______________________________________________ pim mailing list pim@ietf.org https://www.ietf.org/mailman/listinfo/pim
- [pim] Robert Wilton's Discuss on draft-ietf-pim-i… Robert Wilton via Datatracker
- Re: [pim] Robert Wilton's Discuss on draft-ietf-p… Acee Lindem (acee)
- Re: [pim] Robert Wilton's Discuss on draft-ietf-p… Hongji Zhao
- Re: [pim] Robert Wilton's Discuss on draft-ietf-p… Hongji Zhao
- Re: [pim] Robert Wilton's Discuss on draft-ietf-p… Alvaro Retana
- Re: [pim] Robert Wilton's Discuss on draft-ietf-p… Rob Wilton (rwilton)