Re: [netmod] [Netmod-ver-dt] Adding a feature is it BC or NBC ? [was: RE: YANG packages draft - now ready for review please]
Balázs Lengyel <balazs.lengyel@ericsson.com> Fri, 11 October 2019 13:21 UTC
Return-Path: <balazs.lengyel@ericsson.com>
X-Original-To: netmod@ietfa.amsl.com
Delivered-To: netmod@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E71D5120074 for <netmod@ietfa.amsl.com>; Fri, 11 Oct 2019 06:21:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.001
X-Spam-Level:
X-Spam-Status: No, score=-2.001 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, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-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 tpaqk5z3yFSt for <netmod@ietfa.amsl.com>; Fri, 11 Oct 2019 06:21:39 -0700 (PDT)
Received: from EUR02-VE1-obe.outbound.protection.outlook.com (mail-eopbgr20044.outbound.protection.outlook.com [40.107.2.44]) (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 DBA12120088 for <netmod@ietf.org>; Fri, 11 Oct 2019 06:21:38 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=F09yTAxlZ6CDHY8buEv1eqCmfbmZ4wKTPtZ+onEjcvzE/oyzvndd529CwAuLHOrAJsaKw76Ok0q77KHITFSo4AdIEEcUYrxqRcOQT9vkZ/FPPF2TD0/L7M3ipkKN5CsfAXlBekZb/bEWDUCTx8wY3ntT3kSQ+hcuV9SLHdS7QvF2ZiQgWVZXhWrUMMcOqNpNz3GEqT82g5h2XBhsXtzxAFuYI2ZFmocPPfOqdhXlGOJkh2HvPJoUt18cspjxS/EXcNEXP08DMMJJf4va7OGt59SM3ybeiGew+QMwUrP69HuV3s1687F+6h/x/mxRYJarQFHtMSicsmsOciF8Ummk1w==
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=ph39rPDdTOjlTe6rQOPEPIGkRSqBW2WkCiYcjuB3TmA=; b=KkY7G4NvN7GcE0zoEIE6vLBqrIarv0cPBjjkloLYtDz2/r5aluLnFCoHBAKdkCaPZneXvbedEQ9AR+8OWLXsrkBMI/cjGOeggflyyu6ePiBEuyommWaT//WuDt4+KqMIqnctlqfGrsSlabqlsZ4/xvx/RVpoj5SeAsSdzDWeN13rk7AA9uSj4BtDZTxW89hU0Qq1GyrbqjoRRLraODGzC/fw4GDyWmRkxAPzb4NZWI6NAL1xdyaEc8trlkfQ2jHMaLRTbQ4/G5YKxXil3f9XpRi4v591CfRJbbc/gEHdHTba6bMvWM4B2tcxnFFEAfAKCiBqZzFuwHMGAjRg9zTwtg==
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=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ph39rPDdTOjlTe6rQOPEPIGkRSqBW2WkCiYcjuB3TmA=; b=ehQGGA9bOdxk6TVHuJtzUWkJTzDUBuB371yT3bMzjy428/swDpbAVNGfihuy5c+0KWcSe7tWElZm+KKM6SgpYkkRVfB7PtcTRv1WU8uEY3AiDRgOQ4dcrpZo5yWVLoZuUEc58rVQpqStSBi9re4Lu8rAOl8FzQniwKv2Hml5tDg=
Received: from VI1PR0701MB2286.eurprd07.prod.outlook.com (10.169.137.153) by VI1PR0701MB2893.eurprd07.prod.outlook.com (10.173.76.12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2347.10; Fri, 11 Oct 2019 13:21:36 +0000
Received: from VI1PR0701MB2286.eurprd07.prod.outlook.com ([fe80::2d49:4ace:81d8:2fbc]) by VI1PR0701MB2286.eurprd07.prod.outlook.com ([fe80::2d49:4ace:81d8:2fbc%12]) with mapi id 15.20.2347.021; Fri, 11 Oct 2019 13:21:35 +0000
From: Balázs Lengyel <balazs.lengyel@ericsson.com>
To: "Rob Wilton (rwilton)" <rwilton@cisco.com>, "netmod@ietf.org" <netmod@ietf.org>
Thread-Topic: [Netmod-ver-dt] Adding a feature is it BC or NBC ? [was: RE: YANG packages draft - now ready for review please]
Thread-Index: AQHVgCG7st4xHnFXDU2FAFmC68Y67adVbGxA
Date: Fri, 11 Oct 2019 13:21:35 +0000
Message-ID: <VI1PR0701MB2286348DCEEFD7B5560731CEF0970@VI1PR0701MB2286.eurprd07.prod.outlook.com>
References: <VI1PR0701MB2286E1EAFBAE8992F08832E1F0940@VI1PR0701MB2286.eurprd07.prod.outlook.com> <MN2PR11MB436653CD5A9D715BA148F46AB5970@MN2PR11MB4366.namprd11.prod.outlook.com>
In-Reply-To: <MN2PR11MB436653CD5A9D715BA148F46AB5970@MN2PR11MB4366.namprd11.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=balazs.lengyel@ericsson.com;
x-originating-ip: [89.135.192.225]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 53cca068-4aa0-4a0f-a8c6-08d74e4df10f
x-ms-traffictypediagnostic: VI1PR0701MB2893:
x-microsoft-antispam-prvs: <VI1PR0701MB2893A16857B7CE85666B9BC2F0970@VI1PR0701MB2893.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:6790;
x-forefront-prvs: 0187F3EA14
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(366004)(376002)(396003)(39860400002)(136003)(346002)(51914003)(51444003)(189003)(199004)(54896002)(446003)(11346002)(14454004)(186003)(66066001)(76116006)(81166006)(85182001)(76176011)(7696005)(81156014)(26005)(8676002)(6506007)(102836004)(53546011)(790700001)(6116002)(236005)(3846002)(9686003)(71190400001)(71200400001)(2906002)(6436002)(66446008)(64756008)(66556008)(66946007)(66476007)(66616009)(33656002)(229853002)(55016002)(2501003)(6306002)(74316002)(316002)(110136005)(256004)(14444005)(5024004)(478600001)(25786009)(6246003)(86362001)(5660300002)(486006)(7736002)(476003)(66574012)(99936001)(52536014)(85202003)(9326002)(8936002)(99286004)(21314003); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0701MB2893; H:VI1PR0701MB2286.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: xq0p7dpbYhBv254/LLxla6a/29c1YPVF0Ia4p5NSIrGFTQHdZ6UQA96wkMkLymHk8DDmu8R8dtb0dIEBcVzM243Jlku6hnjn1IcmFcy95nj4wDBSKk3zYeNxMIOU38Bs/0j/SHCJJozLbqj1vbmgkpZhonw85WunhHlG59hGSyCVvH8ERf4Qdutp15mj/kYymGfsFj+BVoGV3crZGzVi+3hfkRkHuWPt1GHfyhn0qUmICnhsnabBEPaLsvwLzph4vUxsVPZ0lC+LyhFhSASmCymNUH17D0rr23WcpIsaSS7easRlgKGQ59uONAo1I8J0iVoGQ1D1YpUc9klwLH3Onwn+T9VOePmh//9T6ex1Ev12OSxUa4R0j6dpZTSuAvhN2EBzDVyj4gGniwRuuVGGIfPGx2tJtfhbRUIPIErPy3Q=
x-ms-exchange-transport-forked: True
Content-Type: multipart/signed; protocol="application/x-pkcs7-signature"; micalg="SHA1"; boundary="----=_NextPart_000_0489_01D58047.91301B20"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 53cca068-4aa0-4a0f-a8c6-08d74e4df10f
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Oct 2019 13:21:35.8458 (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: JBu9JLyXFT4JGf6hIdEdi2wDBQlJ0I9FMmUFCUWfR1AqAKfhDusVVjOVzBQndJeRmVzZ4Vt4GLNZ/WkPMwcBI2VRUh7DTQe6fRs79neYolA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0701MB2893
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/5BKKmtszCQPVuTKsklgUwEElOxI>
Subject: Re: [netmod] [Netmod-ver-dt] Adding a feature is it BC or NBC ? [was: RE: YANG packages draft - now ready for review please]
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: NETMOD WG list <netmod.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netmod>, <mailto:netmod-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netmod/>
List-Post: <mailto:netmod@ietf.org>
List-Help: <mailto:netmod-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netmod>, <mailto:netmod-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 11 Oct 2019 13:21:43 -0000
Hello Rob, I agree: so introducing the “not” operator into if-feature IMHO was a mistake. It can have its uses, but it is very dangerous. regards Balazs (Sending it to the full netmod group.) From: Rob Wilton (rwilton) <rwilton@cisco.com> Sent: 2019. október 11., péntek 12:50 To: Balázs Lengyel <balazs.lengyel@ericsson.com>; Reshad Rahman (rrahman) <rrahman@cisco.com>; netmod-ver-dt@ietf.org Subject: RE: [Netmod-ver-dt] Adding a feature is it BC or NBC ? [was: RE: YANG packages draft - now ready for review please] Hi Balazs, This is an interesting example. It doesn’t just break packages, but breaks YANG conformance more generally, particularly if the leaf was also marked as mandatory. I.e. whether a server implements the radius feature will determine whether or not the configuration is valid. Basically, I think that this is an unwise use of if-feature. Generally, I think that the principle behind features is that they represent functionality that is optional to support, and probably should not be used in this way to remove nodes from the underlying model. I think that this is probably a bug/flaw in YANG. Thanks, Rob From: Netmod-ver-dt <netmod-ver-dt-bounces@ietf.org <mailto:netmod-ver-dt-bounces@ietf.org> > On Behalf Of Balázs Lengyel Sent: 10 October 2019 23:57 To: Rob Wilton (rwilton) <rwilton@cisco.com <mailto:rwilton@cisco.com> >; Reshad Rahman (rrahman) <rrahman@cisco.com <mailto:rrahman@cisco.com> >; netmod-ver-dt@ietf.org <mailto:netmod-ver-dt@ietf.org> Subject: [Netmod-ver-dt] Adding a feature is it BC or NBC ? [was: RE: YANG packages draft - now ready for review please] Hello, Adding a new feature MAY or MAY NOT be backward compatible. Think about the following YANG: feature radius {} leaf xxxx { if-feature “not radius”; } So if I add radius it actually removes the leaf xxx. NBC. regards Balazs Regards Balazs From: Netmod-ver-dt <netmod-ver-dt-bounces@ietf.org <mailto:netmod-ver-dt-bounces@ietf.org> > On Behalf Of Rob Wilton (rwilton) Sent: 2019. október 10., csütörtök 13:17 To: Reshad Rahman (rrahman) <rrahman@cisco.com <mailto:rrahman@cisco.com> >; netmod-ver-dt@ietf.org <mailto:netmod-ver-dt@ietf.org> Subject: Re: [Netmod-ver-dt] YANG packages draft - now ready for review please Hi Reshad, Thanks for the comments. I’ve fixed all of the typos. For the Abstract: Old: This document defines YANG packages, a versioned organizational structure holding a set of related YANG modules, that can be used to simplify the conformance and sharing of YANG schema. It describes how YANG instance data documents are used to define YANG packages, and how the YANG library information published by a server can be augmented with packaging related information. Proposed: This document defines YANG packages, a versioned organizational structure holding a set of related YANG modules, that collectively define a YANG schema. It describes how YANG instance data documents are used to define YANG packages, and how the YANG library information published by a server can be augmented with packaging related information. For 5.2.1.1 Re: “Should this list also include/state “any NBC changes to a module in the package”?” Ah, I see. This is what the following current text of the second paragraph was meant to mean Old: <t>Changing a package import to select a package version that is non-backwards-compatible to the prior package version, or removing a previously imported package.</t> <t>Changing a module import to select a module revision that is non-backwards-compatible to the prior module revision, or removing a previously implemented module.</t> <t>Removing a previously supported feature.</t> <t>Adding, changing, or removing a deviation that is considered a non-backwards-compatible change to the affected data node in the schema associated with the prior package version.</t> Perhaps the following text would be more clear: <t>Changing an 'imported-packages' list entry to select a package version that is non-backwards-compatible to the prior package version, or removing a previously imported package.</t> <t>Changing a 'modules' or 'import-only-modules' list entry to select a module revision that is non-backwards-compatible to the prior module revision, or removing a previously implemented module.</t> <t>Removing a feature from the 'supported-feature' leaf-list.</t> <t>Adding, changing, or removing a deviation that is considered a non-backwards-compatible change to the affected data node in the schema associated with the prior package version.</t> Re 5.5.1 “MUST” instead of “SHOULD” for uniqueness. Probably we should discuss today, but my default position is that I think that I will change this. I think that my question was that how do you enforce global uniqueness …. Thanks, Rob From: Reshad Rahman (rrahman) <rrahman@cisco.com <mailto:rrahman@cisco.com> > Sent: 10 October 2019 03:57 To: Rob Wilton (rwilton) <rwilton@cisco.com <mailto:rwilton@cisco.com> >; netmod-ver-dt@ietf.org <mailto:netmod-ver-dt@ietf.org> Subject: Re: [Netmod-ver-dt] YANG packages draft - now ready for review please Hi Rob, Looks good, but TBH I haven’t gone over the minute details of the examples etc. Some comments. Abstract The sentence with “simplify the conformance and sharing of YANG schema” doesn’t jive well to me. I think I know what you mean (we probably discussed this previously), I don’t have better phrasing to propose. Only suggestion I have is to consider shortening the abstract. Terminology s/oraganization/organization/ Introduction s/appendicies/appendices/? Objectives s/were a client/where a client/ YANG Package definition s/oraganization/organization/ s/collectively define/collectively defines/? 3rd paragraph starting with “Each version of a YANG package defines…”. Do this in bullet form instead? 5.1 Package definition rules s/decendent/descendent/ s/supercedes/supersedes/ 5.2.1 Leaf is ‘nbc-changes’ (not ‘nbc-change’) 5.2.1.1. Non-Backwards-compatible changes Should this list also include/state “any NBC changes to a module in the package”? 5.3 s/libary/library/ 5.5.1 Why SHOULD is used for uniqueness? Why not MUST? s/publically/publicly/? 6 s/pacakge/package/ Regards, Reshad. From: Netmod-ver-dt <netmod-ver-dt-bounces@ietf..org <mailto:netmod-ver-dt-bounces@ietf.org> > on behalf of "Rob Wilton (rwilton)" <rwilton@cisco.com <mailto:rwilton@cisco.com> > Date: Monday, October 7, 2019 at 9:58 AM To: "netmod-ver-dt@ietf.org <mailto:netmod-ver-dt@ietf.org> " <netmod-ver-dt@ietf.org <mailto:netmod-ver-dt@ietf.org> > Subject: [Netmod-ver-dt] YANG packages draft - now ready for review please Hi, I think that this document has had sufficient updates that it is ready for review. The latest version is available at per the commit information above, or attached. I appreciate that it is quite short notice, but if it is possible for folks to review and provide comments before Thursday’s meeting that probably be helpful. Thanks, Rob
- Re: [netmod] [Netmod-ver-dt] Adding a feature is … Balázs Lengyel