Re: [netconf] WGLC: draft-ietf-netconf-notification-capabilities-11

"Rob Wilton (rwilton)" <rwilton@cisco.com> Thu, 12 March 2020 10:46 UTC

Return-Path: <rwilton@cisco.com>
X-Original-To: netconf@ietfa.amsl.com
Delivered-To: netconf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0464B3A0A7E; Thu, 12 Mar 2020 03:46:28 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.6
X-Spam-Level:
X-Spam-Status: No, score=-9.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=GhD/IHAM; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=QpJdT8dk
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 d6efPwtNnGo3; Thu, 12 Mar 2020 03:46:26 -0700 (PDT)
Received: from alln-iport-5.cisco.com (alln-iport-5.cisco.com [173.37.142.92]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 448603A0A64; Thu, 12 Mar 2020 03:46:26 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=5504; q=dns/txt; s=iport; t=1584009986; x=1585219586; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=NkxxA1alzXruzGaUNJ1lsrLiRQ+yjAs7lOqqgxAOaHs=; b=GhD/IHAMBkI0yhxBVk4NE/jbQHBTvypDBnI0k1mlBnRA6g4IuH1S9w7Z ntCOYJeku3Ra35H6cfczx+ki+dnuJkGEee2ZyxG310UbBWhzItDiZBnP3 7iNsAiwkg0/jrKs2su6qFplJWh7pRxPryfjsVFK3MvEegq2HPqIZgcQTp A=;
IronPort-PHdr: =?us-ascii?q?9a23=3AzwnWkRwJiwRSqHjXCy+N+z0EezQntrPoPwUc9p?= =?us-ascii?q?sgjfdUf7+++4j5YhSN/u1j2VnOW4iTq+lJjebbqejBYSQB+t7A1RJKa5lQT1?= =?us-ascii?q?kAgMQSkRYnBZufFkz/MPnsRyc7B89FElRi+iLzPA=3D=3D?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0A5CADHEmpe/5NdJa1mHQEBAQkBEQU?= =?us-ascii?q?FAYF7gVQpJwWBRCAECyoKh1ADim6CX5gWglIDVAkBAQEMAQEtAgQBAYFPgnQ?= =?us-ascii?q?CghUkOBMCAwEBCwEBBQEBAQIBBQRthVYMhWMBAQEBAgESLgEBKgoBAgELBAI?= =?us-ascii?q?BCBEBAwEBLzIXBggCBAENBQgahU8DDiABA6AgAoE5iGKCJ4J/AQEFhRMYggw?= =?us-ascii?q?JgTiMLxqBQT+BEUeCTT6ETYNBgiyNU5Mkj0IKgjyNI4lngkqIJgWESot/jwG?= =?us-ascii?q?bWQIEAgQFAg4BAQWBaSI3gSFwFYMnUBgNgRqNA4EnAQiCQ4pVdIEpjAEBgQ8?= =?us-ascii?q?BAQ?=
X-IronPort-AV: E=Sophos;i="5.70,544,1574121600"; d="scan'208";a="454588498"
Received: from rcdn-core-11.cisco.com ([173.37.93.147]) by alln-iport-5.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 12 Mar 2020 10:46:19 +0000
Received: from XCH-ALN-003.cisco.com (xch-aln-003.cisco.com [173.36.7.13]) by rcdn-core-11.cisco.com (8.15.2/8.15.2) with ESMTPS id 02CAkI6A009093 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 12 Mar 2020 10:46:19 GMT
Received: from xhs-aln-001.cisco.com (173.37.135.118) by XCH-ALN-003.cisco.com (173.36.7.13) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 12 Mar 2020 05:46:18 -0500
Received: from xhs-rcd-003.cisco.com (173.37.227.248) by xhs-aln-001.cisco.com (173.37.135.118) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 12 Mar 2020 05:46:18 -0500
Received: from NAM11-BN8-obe.outbound.protection.outlook.com (72.163.14.9) by xhs-rcd-003.cisco.com (173.37.227.248) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Thu, 12 Mar 2020 05:46:18 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; =?utf-8?q?b=3DfRMC9hvxt/9ur6MKSiwKnI1V/t7sAMyOnHYEmhLaAZaYTEzwNT29x/F7xPj/T?= =?utf-8?q?fu6uha+EGKUCrqxg9L9n98PhWBXgFAECmyTKdEp1jRSj2/DtoFqwEvemZYlY9eFiT?= =?utf-8?q?V3wjvKhBeduBbKcGngKo0vJZwh6rfjypzpa5ISrPxNmVQ7zJhMMpndB7Nqb/S8k0e?= =?utf-8?q?aBFY4gDsvfoWENYMRhjj3aTRhANey+bHC/prc67ckcAAaYr/LPkAEECRr53KjZz+z?= =?utf-8?q?ga3qsoBWBc5kdXArtWMlgpm0DX5FbcnS2tzZDKM4tU2BSsqM05XpzilgmXpR3Ok5D?= =?utf-8?q?eh9q8T3CQQIH9PDQvLUew=3D=3D?=
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; =?utf-8?q?h=3DFrom=3ADate=3ASubject=3AMessage-ID=3ACont?= =?utf-8?q?ent-Type=3AMIME-Version=3AX-MS-Exchange-SenderADCheck=3B?= =?utf-8?q?bh=3Dbo5lBofMzqIZGvmgVXhj3+YdRP3XJdD2LhTxRSgdReQ=3D=3B_b=3DkGcEvX?= =?utf-8?q?BMk7oy1RfolsSQCxc13RW1PRAIYERheWIxXJZnCRlLwAOToeFuzySlQhhPtFO1WL4?= =?utf-8?q?WJt9ruVAfjtTCLxa1QMEkWVP2SM2QYJUFg76akbQ1TRCGPu6moe+cFzjLbtBMyEdi?= =?utf-8?q?cjoRlEwR1gKoG1pfVTWzVXq7U4zZbyu3GDPVTOshAkIsOLFszs2IdhAcioP71/IqW?= =?utf-8?q?4M/PAXIhFayEH1cbU7RnbVQS9iTgqPNk77sOi+y6XSu+EC0MS+R4F9FaRAAjiIy5r?= =?utf-8?q?YpYyTyg0MeAoiXGJ36kLOPV/DMGDEFJVsR25QbFCJA/mBpBrQ4wZfIRmyQ+sWs8qw?= =?utf-8?q?tRjHwK0nBNQ=3D=3D?=
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; =?utf-8?q?h=3DFrom=3ADate=3ASubject=3AM?= =?utf-8?q?essage-ID=3AContent-Type=3AMIME-Version=3AX-MS-Exchange-SenderADC?= =?utf-8?q?heck=3B_bh=3Dbo5lBofMzqIZGvmgVXhj3+YdRP3XJdD2LhTxRSgdReQ=3D=3B_b?= =?utf-8?q?=3DQpJdT8dk1UmX2xLbOhF3ohO4KglHlAEePZLpD9tzfWtyPMJ164AMoieb6VY9P9?= =?utf-8?q?E2o8LbpAXEtGIWe1YijwPgyu61OPfholXXeFQlKs28nKvofoFTz/K+x7ne8cmBBYv?= =?utf-8?q?qElV0gcv2VRX20u7cURVRdgrpiGIaCHaMWmRrNt25M4s=3D?=
Received: from MN2PR11MB4366.namprd11.prod.outlook.com (2603:10b6:208:190::17) by MN2PR11MB4062.namprd11.prod.outlook.com (2603:10b6:208:150::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2814.13; Thu, 12 Mar 2020 10:46:17 +0000
Received: from MN2PR11MB4366.namprd11.prod.outlook.com ([fe80::3:2164:a8e2:33b3]) by MN2PR11MB4366.namprd11.prod.outlook.com ([fe80::3:2164:a8e2:33b3%5]) with mapi id 15.20.2814.007; Thu, 12 Mar 2020 10:46:17 +0000
From: "Rob Wilton (rwilton)" <rwilton@cisco.com>
To: =?iso-8859-2?Q?Bal=E1zs_Lengyel?= <balazs.lengyel@ericsson.com>, "netconf@ietf.org" <netconf@ietf.org>, "draft-ietf-netconf-notification-capabilities@ietf.org" <draft-ietf-netconf-notification-capabilities@ietf.org>
Thread-Topic: [netconf] WGLC: draft-ietf-netconf-notification-capabilities-11
Thread-Index: AQHV5fT97GQMgsC5TEuE1GmMKTxx36ggKT0AgBjpugCAB5/HwIABLu8AgAL3rkA=
Date: Thu, 12 Mar 2020 10:46:17 +0000
Message-ID: =?utf-8?q?=3CMN2PR11MB436638F28AAFFB3E19059F65B5FD0=40MN2PR11MB4?= =?utf-8?q?366=2Enamprd11=2Eprod=2Eoutlook=2Ecom=3E?=
References: =?utf-8?q?=3C0100017055c347e5-13b624a9-04c0-4ba5-8a53-26a80f0796?= =?utf-8?q?07-000000=40email=2Eamazonses=2Ecom=3E_=3C0100017055e833dd-1a2eca?= =?utf-8?q?c4-53e4-4bb7-aab9-f75516c5fd38-000000=40email=2Eamazonses=2Ecom?= =?utf-8?q?=3E_=3C01000170a78abdde-66112ceb-d466-4c72-ad18-6058ef07ab02-0000?= =?utf-8?q?00=40email=2Eamazonses=2Ecom=3E_=3CBY5PR11MB43554B2EDF3D395D2680D?= =?utf-8?q?9C1B5FE0=40BY5PR11MB4355=2Enamprd11=2Eprod=2Eoutlook=2Ecom=3E_=3C?= =?utf-8?q?DB7PR07MB4011F0C718EF704D1B0F5385F0FF0=40DB7PR07MB4011=2Eeurprd07?= =?utf-8?q?=2Eprod=2Eoutlook=2Ecom=3E?=
In-Reply-To: =?utf-8?q?=3CDB7PR07MB4011F0C718EF704D1B0F5385F0FF0=40DB7PR07MB?= =?utf-8?q?4011=2Eeurprd07=2Eprod=2Eoutlook=2Ecom=3E?=
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=rwilton@cisco.com;
x-originating-ip: [82.15.79.32]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 54009336-da36-4a7a-6c07-08d7c6729810
x-ms-traffictypediagnostic: MN2PR11MB4062:
x-microsoft-antispam-prvs: =?utf-8?q?=3CMN2PR11MB40621A33E7B84A6CA1FBCC5AB5F?= =?utf-8?q?D0=40MN2PR11MB4062=2Enamprd11=2Eprod=2Eoutlook=2Ecom=3E?=
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 0340850FCD
x-forefront-antispam-report: SFV:NSPM; =?utf-8?q?SFS=3A=2810009020=29=284636?= =?utf-8?b?MDA5KSgzNzYwMDIpKDM0NjAwMikoMzY2MDA0KSgxMzYwMDMpKDM5ODYwNDAw?= =?utf-8?b?MDAyKSgzOTYwMDMpKDE5OTAwNCkoNzEyMDA0MDAwMDEpKDUzNTQ2MDExKSg2?= =?utf-8?q?506007=29=2886362001=29=2855016002=29=2876116006=29=2864756008=29?= =?utf-8?q?=2866946007=29=287696005=29=2866476007=29=2866446008=29=281565050?= =?utf-8?b?MDAwMSkoNjY1NTYwMDgpKDk2ODYwMDMpKDY2NTc0MDEyKSg0MzI2MDA4KSgy?= =?utf-8?q?906002=29=28186003=29=28110136005=29=288936002=29=2852536014=29?= =?utf-8?q?=288676002=29=28316002=29=285660300002=29=2881156014=29=288116600?= =?utf-8?q?6=29=2826005=29=2833656002=29=28478600001=29=3B?= DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR11MB4062; H:MN2PR11MB4366.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: =?utf-8?q?kMx8wCXeaaF+pbxf8psn6keyFP5qKwQ?= =?utf-8?q?q/C5xei/6WFe82IgZ4rUPigjq5laE2jaaSpf03fRf8NUPsi5reZVFpJU41eWeSJKl?= =?utf-8?q?XueJk3p3bHI/fV/gjo6IRnxZ7lsEkSoQ2TbQ0QQ9Dp4xOt0fNl3X9uGO6BXpY6qY1?= =?utf-8?q?JE/80lBGG+9hsHI1GzMMAYFtqVTsxZlv1iOdQO9UAXvglFNbTot/pbqOMygBlFTAF?= =?utf-8?q?siCxm/L+9fedxNKKFy0o7BHqwLwip62cx5B3EnCKbu04S+XVeTRDB9yzrYbLtSQ3F?= =?utf-8?q?hYV4DvaJAtIzgBfAOtpUWXJDEcKqZUB2Tp6QEfiCXepasdeFNa2J1foAN2vDoGiPG?= =?utf-8?q?qP/+tBIbI1ZJAc5QgqpMGPns0toyuDfkWnpmH+77r5MUzEqH0hAxxyqyxiJSAkQdg?= =?utf-8?q?lK95Q+3tgi1+2Xz9z8aasUpH3Ht?=
x-ms-exchange-antispam-messagedata: =?utf-8?q?h7BB7hoej/s1YVTXWF51/v5w2KL3Ff?= =?utf-8?q?2lRpBsVD3GxIjeAbscAp2ixphfU0gmF3Lr4zkUbp8vOPQkZ/MgV1ALNppv/vJwYtV?= =?utf-8?q?mwK7yN2vY4cfTBoDRp3izBTl/OKLc2C6+snzR0InLz74scF10AuJFcg=3D=3D?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="iso-8859-2"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 54009336-da36-4a7a-6c07-08d7c6729810
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Mar 2020 10:46:17.4230 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: =?utf-8?q?iMzXxUHxE7DnKv/cEgNlF?= =?utf-8?q?KbmFQVRwhZc5ISZ3c6ofmm38Q/TwKz6PDgrmIjaoXA2AQ272IgRtG5lgJwkcEA8Gg?= =?utf-8?q?=3D=3D?=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4062
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.13, xch-aln-003.cisco.com
X-Outbound-Node: rcdn-core-11.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/ukONXvNWJErFolclGPo4LpwtM6w>
Subject: Re: [netconf] WGLC: draft-ietf-netconf-notification-capabilities-11
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: NETCONF WG list <netconf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netconf>, <mailto:netconf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netconf/>
List-Post: <mailto:netconf@ietf.org>
List-Help: <mailto:netconf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netconf>, <mailto:netconf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 12 Mar 2020 10:46:28 -0000

Hi Balazs,

Please see inline.

> -----Original Message-----
> From: Balázs Lengyel <balazs.lengyel@ericsson.com>
> Sent: 10 March 2020 12:24
> To: Rob Wilton (rwilton) <rwilton@cisco.com>om>; netconf@ietf.org; draft-
> ietf-netconf-notification-capabilities@ietf.org
> Cc: Kent Watsen <kent+ietf@watsen.net>
> Subject: RE: [netconf] WGLC: draft-ietf-netconf-notification-capabilities-
> 11
> 
> -----Original Message-----
> From: Rob Wilton (rwilton) <rwilton@cisco.com>
> Sent: 2020. március 9., hétfő 19:34
> To: netconf@ietf.org; Balázs Lengyel <balazs.lengyel@ericsson.com>om>;
> draft-ietf-netconf-notification-capabilities@ietf.org
> Cc: Kent Watsen <kent+ietf@watsen.net>
> Subject: RE: [netconf] WGLC: draft-ietf-netconf-notification-capabilities-
> 11
> 
<snipped>
> 
> 
> 4. Section 2:
> Please state that the YANG modules are compliant to NMDA.
> BALAZS: What does it mean that a read-only module is NMDA compliant?
> Please
> explain. I would rather not add text that I do not understand.
>
[RW] 
It just means that the model is designed to conform to the architecture, and work with devices that support such an architecture.  E.g. it also means that the modules are not called "foo-state".


 
> 
> 5. YANG model:
> 
> - Does "minimum-dampening-period" only apply of "on-change-supported" is
> enabled.  If so, it might be helpful for the description to state this.  I
> would also put the "on-change-supported" leaf before the
> "minimum-dampening-period" leaf.
> BALAZS: The leaf has an if-feature "yp:on-change" substatement, which
> should
> indicate it is relevant to on-change. I would like to avoid replicating
> too
> much information from the Yang-Push RFC.
> I will rearrange the order of leaves grouping periodic and on-change
> related
> leaves.
> 
> - Does "minimum-dampening-period" only apply if dampening is requested, or
> does that mean that dampening is always required?  The text may need to be
> clarified.  Otherwise, I'm concerned about the on-change notification for
> interface statistics ... that could generate a lot of notifications ;-)
> BALAZS: The answer is not trivial. I can see logic in both cases:
> a) if there is a minimum-dampening-period declared  here, that means that
> dampening MUST always be used. Logical because if the publisher can not
> support a smaller than 5 sec dampening period, it is probably incapable to
> support on-change without dampening.
> b) the publisher MAY support on-change without dampening, but if dampening
> is requested the period MUST NOT be smaller than the
> minimum-dampening-period
> 
> - For both "minimum-dampening-period" and on-change-supported", can
> probably
> remove "data store or" from the description.
> BALAZS: OK
> 
> - Suggest moving "supported-excluded-change-type" above
> periodic-notifications-supported, to keep all the on-change options
> together.  I'm also not particularly keen on the negative enumeration
> values, having them as positive values would seem more normal to me.
> BALAZS: I will rearrange the order of leaves grouping periodic and on-
> change
> related leaves.
> I used negative numbers as the type's union also includes type
> yp:change-type which is an enumeration with implicit value assignment.
> Using
> negative numbers is a way to avoid clashing with these implicitly assigned
> numbers.
[RW] 
Okay. My concern was that not all common programming languages support negative enums, but from a quick google I could be wrong on this point. 

> 
> 6. Section 6 (Security):
> 
> Is there a risk that the capabilities information, if available offline
> and
> also from a device, could be used to fingerprint the device and gain
> information about what version of software/hardware is being run?
> BALAZS: I t could be used as part of the fingerprinting, as any readable
> data that changes infrequently. IMHO this is a generic problem, to be
> solved
> by proper access control rules. Adding it as a kind of standard boiler
> text
> to every YANG module would not help. Maybe we would need an OAM security
> RFC.
> 
[RW] 
Actually, perhaps this should just be in the security section of the instance data draft.  That would then seem to apply generally to all YANG modules used as instance data.

> 
> 7. Could you please add a brief section to thank folks who have
> contributed
> and reviewed this work.
> - I think that it easier to get reviews, if the time/effort spent in the
> review is acknowledged.
> BALAZS: I usually do it. It seems I forgot it now.
> 
> 
> 8. Appendix A:
> - Your examples look like they probably have some introduced line wrapping
> (e.g. for the description, node-selector, and others).  Perhaps we should
> be
> using the artwork draft for this, or alternatively add a comment that
> linespaces have been introduced in various fields to improve readability.
> BALAZS: I would prefer to wait till  draft-ietf-netmod-artwork-folding
> becomes a full RFC. Even creating/using the folding header without the RFC
> number is problematic.
[RW] 
I don't think that this should really be a problem.

draft-ietf-netmod-artwork-folding is int the RFC editor queue and has no missing references, i.e. it is going to become an RFC before this draft does, and the RFC editor will be able fix up the folding header.

Thanks,
Rob