Re: [netmod] All IETF YANG modules MUST include revision-label statements

"Reshad Rahman (rrahman)" <rrahman@cisco.com> Tue, 31 March 2020 16:04 UTC

Return-Path: <rrahman@cisco.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 A652B3A237A for <netmod@ietfa.amsl.com>; Tue, 31 Mar 2020 09:04:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.697
X-Spam-Level:
X-Spam-Status: No, score=-7.697 tagged_above=-999 required=5 tests=[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_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, 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=fbv5QjNs; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=Oxf3oCv7
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 yJsXhCjQfwLH for <netmod@ietfa.amsl.com>; Tue, 31 Mar 2020 09:04:42 -0700 (PDT)
Received: from rcdn-iport-4.cisco.com (rcdn-iport-4.cisco.com [173.37.86.75]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1BD0B3A2378 for <netmod@ietf.org>; Tue, 31 Mar 2020 09:04:42 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=109291; q=dns/txt; s=iport; t=1585670681; x=1586880281; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=29ZFHrWj1qkIM6lddRwzemckZGrR7E8IrbfZSOfs6Z0=; b=fbv5QjNsqwgBoKuU94Mn0B0Vzp+jH3uvs7QtyyqD8K2EYLTam8JWQ08Z kv1TEXmQ/ib5rV/Xqg3twP04uozp4FogYFPn+GPh2R9daCOgotfQljDO6 7DgqeRMfxD7QZ4+qchEyU54JREtO4FzVZzjdbHgh3fkGFT5/OEHk4YTVM U=;
IronPort-PHdr: =?us-ascii?q?9a23=3AOH3ALR2R8juh1W9xsmDT+zVfbzU7u7jyIg8e44?= =?us-ascii?q?YmjLQLaKm44pD+JxKHt+51ggrPWoPWo7JfhuzavrqoeFRI4I3J8RVgOIdJSw?= =?us-ascii?q?dDjMwXmwI6B8vQE1fyLPvjaQQxHd9JUxlu+HToeUU=3D?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0AJBwBuaYNe/5NdJa1mHAEBAQEBBwE?= =?us-ascii?q?BEQEEBAEBgXuBJS9QBWxYIAQLKgqEEINFA4pugl+YHYFCgRADVAoBAQEMAQE?= =?us-ascii?q?iBwQCBAEBhEQCF4IeJDgTAgMBAQsBAQUBAQECAQUEbYVWDIVwAQEBAQIBEgg?= =?us-ascii?q?JBBkBASwLAQQHBAIBCBEDAQEBIQEGAwICAjAUCQgCBAENBRsHgwQBgX5NAw4?= =?us-ascii?q?gAQ6jFwKBOYhidX8zgn8BAQWFGBiCDAmBOIkOgyMagUE/gREnIIIfLj6CZwQ?= =?us-ascii?q?agS8hEAkNCQIGglQygiyNdA4BAy2CS4V8JIoEj08Kgj2HYY82HYJMgQKTSYR?= =?us-ascii?q?XjxuBUYdHkm0CBAIEBQIOAQEFgWkiKoEucBU7KgGCQQlHGA2OHYNzilV0AgE?= =?us-ascii?q?TgROMVAGBDwEB?=
X-IronPort-AV: E=Sophos;i="5.72,328,1580774400"; d="scan'208,217";a="744720748"
Received: from rcdn-core-11.cisco.com ([173.37.93.147]) by rcdn-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 31 Mar 2020 16:04:28 +0000
Received: from XCH-RCD-004.cisco.com (xch-rcd-004.cisco.com [173.37.102.14]) by rcdn-core-11.cisco.com (8.15.2/8.15.2) with ESMTPS id 02VG4SnV009289 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Tue, 31 Mar 2020 16:04:28 GMT
Received: from xhs-aln-002.cisco.com (173.37.135.119) by XCH-RCD-004.cisco.com (173.37.102.14) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Tue, 31 Mar 2020 11:04:27 -0500
Received: from xhs-rcd-001.cisco.com (173.37.227.246) by xhs-aln-002.cisco.com (173.37.135.119) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Tue, 31 Mar 2020 11:04:27 -0500
Received: from NAM04-SN1-obe.outbound.protection.outlook.com (72.163.14.9) by xhs-rcd-001.cisco.com (173.37.227.246) with Microsoft SMTP Server (TLS) id 15.0.1497.2 via Frontend Transport; Tue, 31 Mar 2020 11:04:27 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hVL6wVIDqvWiOViCUyJ+0ftK8Gfr/JigEWNc6ZtmMFVe3q9KX32ZsUeXf+ideo2DNssRXoISN6Tv31L6QK+J+UeaMT9qwF//MHD37M+RsAtrAOsHRmlapVTtWVE/wGRrGIg/oO9jqZiKyi9091sLw1god8tWPEtsZ2lv7S4U5pzocvnhePt7pkMm3xNw8IwNUe+1E+78jE6adItVb/WOJbl5ign4v/FGE3UH2Eo/Uhwi8FCxGlp/rTkO1EeUn4LIeJ3LycjrYGZVBIRw28jyMB1eU2pwhfGMqbBBELNqhIqDS3J+169XY/0ctqVdq1/NR1TEdgsW+xFSmc0GM+Socg==
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=29ZFHrWj1qkIM6lddRwzemckZGrR7E8IrbfZSOfs6Z0=; b=WPq4S2DI34RRYDDN3nAbpTan2Y9/wMljkIFkBfoK6Kht3y/gcsnsBCtye4SysMkf6l0SfSFEcV86LZSNc6d7+inmt3MSqfLbB2UvQRbR9INOBex4grFE2G3mx0fA2PMYi/GQZq2JFJqIAXwWOWzo3sfhJFW7/4i7ZmLCWZzcTptGyUtDw46KvD7JBUlZ0tYLruUtu2yRUJ+pww9cdduzzfWzzE4CCcf+WrV1wNiZq1gj9OF0ASf+5V52WH38cdhxzea2/Vk5Xmc93EV+w8FRUS5csHrh3i2qoxV0karxt8ffxB/g2uy8+Wb5hp1kbAbX5dBVtULtHUXWVWuC+IAEww==
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; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=29ZFHrWj1qkIM6lddRwzemckZGrR7E8IrbfZSOfs6Z0=; b=Oxf3oCv7V5pCjtuj/Y/0eeCkbVeEuccmzXUahpqu9iOnzQf78+/XkaMxSvVPtibpbkzGo5Sa4IyGxqRKUAFrNt/AN3VY4rHzp8a3mAtRAp5MHg5YhQwCSMsaZ61ZjW+XmzU9bpNC283+BSePzwqgVhBs+MV4+hlb/uY8cwAj/V4=
Received: from BN6PR11MB1748.namprd11.prod.outlook.com (2603:10b6:404:101::12) by BN6PR11MB1697.namprd11.prod.outlook.com (2603:10b6:404:41::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2856.20; Tue, 31 Mar 2020 16:04:25 +0000
Received: from BN6PR11MB1748.namprd11.prod.outlook.com ([fe80::d1f9:733e:e200:f972]) by BN6PR11MB1748.namprd11.prod.outlook.com ([fe80::d1f9:733e:e200:f972%6]) with mapi id 15.20.2856.019; Tue, 31 Mar 2020 16:04:25 +0000
From: "Reshad Rahman (rrahman)" <rrahman@cisco.com>
To: "Ivory, William" <william.ivory@intl.att.com>, "mbj+ietf@4668.se" <mbj+ietf@4668.se>, "jason.sterne@nokia.com" <jason.sterne@nokia.com>
CC: "netmod@ietf.org" <netmod@ietf.org>
Thread-Topic: [netmod] All IETF YANG modules MUST include revision-label statements
Thread-Index: AQHWBNyukTO5JXWurk2ljZZiFHPveKhhLBqAgABJ5QD//934AIAASQ+AgAAPaoCAAARfAP//wqUAgADiDwCAAEnNAA==
Date: Tue, 31 Mar 2020 16:04:25 +0000
Message-ID: <B93C60F3-9A67-4E7A-8C84-64E6937F738B@cisco.com>
References: <047FB87D-37B2-41F4-86D2-B9A03050B4EB@cisco.com> <20200330.223957.1196399215343087647.id@4668.se> <DM5PR08MB2633E6B1CA925B2D6E4B3AAE9BCB0@DM5PR08MB2633.namprd08.prod.outlook.com> <20200330.235046.60166687757387667.id@4668.se> <23333468-9959-4ECA-B529-73E1D906E3E9@cisco.com> <5f184f8673fafbb8eff0ae8b0a19f81409fa45e1.camel@intl.att.com>
In-Reply-To: <5f184f8673fafbb8eff0ae8b0a19f81409fa45e1.camel@intl.att.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.21.0.200113
authentication-results: spf=none (sender IP is ) smtp.mailfrom=rrahman@cisco.com;
x-originating-ip: [70.31.50.95]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: c5cd315e-1e78-4b0e-0cb4-08d7d58d2f12
x-ms-traffictypediagnostic: BN6PR11MB1697:
x-microsoft-antispam-prvs: <BN6PR11MB1697508227316C07C989DB72ABC80@BN6PR11MB1697.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:6108;
x-forefront-prvs: 0359162B6D
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BN6PR11MB1748.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(10009020)(4636009)(39860400002)(376002)(396003)(136003)(346002)(366004)(2906002)(110136005)(2616005)(9326002)(316002)(8936002)(71200400001)(81156014)(6512007)(81166006)(966005)(30864003)(478600001)(66574012)(26005)(4326008)(33656002)(5660300002)(8676002)(64756008)(186003)(76116006)(66556008)(66946007)(66446008)(91956017)(66476007)(6486002)(86362001)(36756003)(53546011)(6506007)(559001)(579004); DIR:OUT; SFP:1101;
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: XJ6QV6oY+qIuNUh/OchL1Lr+N1qOxe9J6cQxyqDR70Ym1C/Ba/LZwUihCDTQ7on2l16ecwL6GDzQd/TN0m+v7v2MSj5KEJRdYxrlKocttkS7+jDEbVFlYlyV/6K6NAK+YMsMjheZUZriqlPj3J+95Ml4KwPlubOe2Y5x/CrZ00uwgDdgFGlaIz90U4jQ2Uo9WnrcBpsyAMrtflN095SR3WLGeo8Eooloikoq6e9uzkI9EHJndZd4K5EyvVA1RghiNBD5j0v3S971IvoprYOrydSZb7h2XydPyLeoXTwo3IOXc7F2xSaASHT93n/qbuf3tyAC3mSg0p7OJfX/m7A7MWNejChf1+/rchXRg9okG2ukULVle1b4LJHgx2HpMhBmjnm8ZvjhZCF09XobkXhk7H1Aq+GAo+SZ/2Ust4T1hxZDyhvLA+GKml92flulQM2o90L4/094GhNziCjFNct+IyI3Cs16JVww+BjAgCSwyGK3kx6sXS5N/fVW1zkxHvpEmz9dUkkYvQJ5w/KXAS02YA==
x-ms-exchange-antispam-messagedata: kjRpHCsIQphiKw/JxHov+azVVBbGRTXpGLbFksMFbMGRIu+tCfFnp31lRaZ7LECEHNOxxmarYtDTbIDLh0+8Dlp7B03i5opCa/cmFAcP+voRh0tQOWsJUqPQl7qSYw+VpfETD4hYspUOt+BsQ+48LQ==
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_B93C60F39A674E7A8C8464E6937F738Bciscocom_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: c5cd315e-1e78-4b0e-0cb4-08d7d58d2f12
X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Mar 2020 16:04:25.1232 (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: Ob/3s0+GONqz3rQsXc4ljpXEdoAdDyqYee1kr1xpUbbOQUcMnuaJHdrJpoASt32v98bis/7rlAkPSLzsHVbBDg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR11MB1697
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.37.102.14, xch-rcd-004.cisco.com
X-Outbound-Node: rcdn-core-11.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/frBEdsfWk3vRPmpWH-WkbOiZ-aU>
Subject: Re: [netmod] All IETF YANG modules MUST include revision-label statements
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: Tue, 31 Mar 2020 16:04:48 -0000

Hi,

Thanks for the suggestion.

This was discussed, I think the reason we didn’t go with that solution is that (as you mentioned) you need the module contents with all the previous version labels. Does anyone from the design team recall any other details?

Regards,
Reshad.

From: "Ivory, William" <william.ivory@intl.att.com>
Date: Tuesday, March 31, 2020 at 3:40 AM
To: "mbj+ietf@4668.se" <mbj+ietf@4668.se>se>, "jason.sterne@nokia.com" <jason.sterne@nokia.com>om>, "Reshad Rahman (rrahman)" <rrahman@cisco.com>
Cc: "netmod@ietf.org" <netmod@ietf.org>
Subject: Re: [netmod] All IETF YANG modules MUST include revision-label statements

Apologies if this has already been suggested and deemed unworkable, but if you have access to all previous version labels for a branch then you can add 'M' only to the versions that are NBC with the previous version, and subsequent versions could drop the M until the next NBC change, ie:

1.0.0 -> 1.0.1 -> 1.0.2 > 1.0.3M -> 1.0.4 -> 1.0.5M ...

Here 1.04 is BC with 1.03 but not 1.0.0 - 1.0.2; 1.0.5 is NBC with 1.0.4 and previous versions etc.

The revision statements contain the revision-labels so you should have all the previous revision-labels present in the file, and you have all the data you need. Now you don't have the problem of the branch being poisoned as soon as the first M is added.

William

On Mon, 2020-03-30 at 22:11 +0000, Reshad Rahman (rrahman) wrote:

On 2020-03-30, 5:51 PM, "Martin Björklund" <mbj+ietf@4668.se<mailto:mbj+ietf@4668.se>> wrote:



    "Sterne, Jason (Nokia - CA/Ottawa)" <jason.sterne@nokia.com<mailto:jason.sterne@nokia.com>> wrote:

    > > But it is not true.  What happened between 1.0.2M and 1.0.3M?

    >

    > It tells you there is an NBC change between 1.0.2M and 1.0.3M.



    No.  As you note below it says that all bets are off.  The change

    between these two could be a spelling error fix.  Hence, Reshad's

    statement that "The revision label allows a user to easily figure out

    whether 2 revisions are (N)BC." is not correct.

You are correct that once a branch is poisoned with an 'M', the information provided is not as rich.

Even though you don't know whether 1.0.3M is BC/NBC with 1.0.2M, you still know that

- 1.0.2M is NBC with 1.0.1 and 1.0.0

- 1.0.3M is NBC with 1.0.1 and 1.0.0

- 1.0.1 is BC with 1.0.0



Still useful IMO.



Regards,

Reshad.



    > The M gives an indication that a branch has been "poisoned" by an

    > NBC change and that all bets are off from that point onwards in that

    > branch.





    /martin





    >

    > > -----Original Message-----

    > > From: netmod <netmod-bounces@ietf.org<mailto:netmod-bounces@ietf.org>> On Behalf Of Martin Björklund

    > > Sent: Monday, March 30, 2020 4:40 PM

    > > To: rrahman@cisco.com<mailto:rrahman@cisco.com>

    > > Cc: netmod@ietf.org<mailto:netmod@ietf.org>

    > > Subject: Re: [netmod] All IETF YANG modules MUST include revision-label

    > > statements

    > >

    > > "Reshad Rahman (rrahman)" <rrahman@cisco.com<mailto:rrahman@cisco.com>> wrote:

    > > >

    > > > On 2020-03-30, 2:20 PM, "Martin Björklund" <mbj+ietf@4668.se<mailto:mbj+ietf@4668.se>> wrote:

    > > >

    > > >     "Reshad Rahman (rrahman)" <rrahman@cisco.com<mailto:rrahman@cisco.com>> wrote:

    > > >     > On 2020-03-28, 4:41 AM, "Martin Björklund" <mbj+ietf@4668.se<mailto:mbj+ietf@4668.se>> wrote:

    > > >     >

    > > >     >     "Reshad Rahman (rrahman)" <rrahman@cisco.com<mailto:rrahman@cisco.com>> wrote:

    > > >     >     > Hi,

    > > >     >     >

    > > >     >     > https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_netmod-2Dwg_yang-2Dver-2Ddt_issues_45&d=DwIGaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=p8kyeK3u4ZYiaQ2ZPGqwkyXmQgBH6r5jpYiYWzhqJ48&m=ffH268c0xOd0DSFLQzZ2JHAmCHjVzPJVJtGPNxiiJcs&s=nyxzbv7ZWMgcXuMEW8MqjeT3oVxla6qFiF96M8SaMUY&e=

    > > >     >     >

    > > >     >     >         o  7.1

    > > >     >     >

    > > >     >     >           The text says:

    > > >     >     >

    > > >     >     >             All IETF YANG modules MUST include revision-label statements

    > > for

    > > >     >     >             all

    > > >     >     >             newly published YANG modules, and all newly published

    > > revisions of

    > > >     >     >             existing YANG modules.  The revision-label MUST take the form

    > > of a

    > > >     >     >             YANG semantic version number [I-D.verdt-netmod-yang-

    > > semver].

    > > >     >     >

    > > >     >     >           I strongly disagree with this new rule.  IETF modules use a linear

    > > >     >     >           history, so there are no reasons to use "modified semver".

    > > >     >     >

    > > >     >     >           It is ok to use rev:nbc-changes if needed, though.

    > > >     >     >

    > > >     >     > We believe some IETF models may not follow linear history, this was

    > > >     >     > brought up (I think) for IDR. Modified semver allows for non-linear

    > > >     >     > history and also doesn't preclude linear history. So even if we end up

    > > >     >     > having no IETF modules using branching, modified semver still works.

    > > >     >

    > > >     >     With the clarifiactions and updates in

    > > >     >     draft-verdt-netmod-yang-module-versioning, non-linear versioning

    > > >     >     works without modified semver.  So there is no technical reason to use

    > > >     >     modified semver in IETF modules.

    > > >     >

    > > >     > So are you proposing we use some other revision-label scheme (e.g.

    > > semver 2.0.0) for IETF modules?

    > > >     >

    > > >     > Or that IETF modules shouldn't use revision-labels?

    > > >

    > > >     That IETF shouldn't use revision labels.

    > > >

    > > > The revision label allows a user to easily figure out whether 2

    > > > revisions are (N)BC.

    > >

    > > I think you meant "modified semver as revision label allows ..."

    > >

    > > But it is not true.  What happened between 1.0.2M and 1.0.3M?

    > >

    > >

    > > /martin

    > >

    > >

    > > > Without the label, you always have to use tooling.

    > > >

    > > > Regards,

    > > > Reshad.

    > > >

    > > >     I am all for using rev:nbc-changes or rev:editorial-changes (which I

    > > >     think should be added) in IETF modules.

    > > >

    > > >

    > > >     /martin

    > > >

    > > >

    > > >     >

    > > >     > Or do you have something else in mind?

    > > >     >

    > > >     > Regards,

    > > >     > Reshad.

    > > >     >

    > > >     >     I can reluctantly accept that modified smever is published as

    > > >     >     Experimental.  But that doesn't mean that IETF modules should use it.

    > > >     >

    > > >     >

    > > >     >     /martin

    > > >     >

    > > >     >

    > > >     >     >

    > > >     >     > Regards,

    > > >     >     > Reshad.

    > > >     >     >

    > > >     >     >

    > > >     >     > On 2020-03-20, 5:08 PM, "netmod on behalf of Reshad Rahman

    > > (rrahman)"

    > > >     >     > <netmod-bounces@ietf.org<mailto:netmod-bounces@ietf.org> on behalf of

    > > >     >     > rrahman=40cisco.com@dmarc.ietf.org<mailto:rrahman=40cisco.com@dmarc.ietf.org>> wrote:

    > > >     >     >

    > > >     >     >     Hi Martin,

    > > >     >     >

    > > >     >     >     We've opened issues to track your review comments (see below).

    > > Will

    > > >     >     >     kick off separate therads for each issue.

    > > >     >     >

    > > >     >     >     https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_netmod-2Dwg_yang-2Dver-2D&d=DwIGaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=p8kyeK3u4ZYiaQ2ZPGqwkyXmQgBH6r5jpYiYWzhqJ48&m=ffH268c0xOd0DSFLQzZ2JHAmCHjVzPJVJtGPNxiiJcs&s=HjVuj69fVsCLulvyNUajxCbtSKPAVkUZVJNK8s-f-Ho&e=

    > > dt/issues?q=is%3Aissue+is%3Aopen+label%3Aupdated-mod-rev-handling

    > > >     >     >

    > > >     >     >     Regards,

    > > >     >     >     Reshad.

    > > >     >     >

    > > >     >     >     On 2020-03-10, 3:31 PM, "netmod on behalf of Martin Björklund"

    > > >     >     >     <netmod-bounces@ietf.org<mailto:netmod-bounces@ietf.org> on behalf of mbj+ietf@4668.se<mailto:mbj+ietf@4668.se>> wrote:

    > > >     >     >

    > > >     >     >         Hi,

    > > >     >     >

    > > >     >     >         Here are my review comments of

    > > >     >     >         draft-verdt-netmod-yang-module-versioning-01.

    > > >     >     >

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  3.1.1

    > > >     >     >

    > > >     >     >             o  In statements that have any data definition statements as

    > > >     >     >                substatements, those data definition substatements MAY be

    > > >     >     >                reordered, as long as they do not change the ordering or any

    > > >     >     >                "rpc"

    > > >     >     >                "input" substatements.

    > > >     >     >

    > > >     >     >           I think this needs to capture that no descendant statements to

    > > >     >     >           "input" can be reordered.  Same for "output" (note, "input" and

    > > >     >     >           "output" in both "rpc" and "action").

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  3.3

    > > >     >     >

    > > >     >     >             All revision labels that match the pattern for the "version"

    > > >     >     >             typedef in the ietf-yang-semver YANG module MUST be

    > > interpreted as

    > > >     >     >             YANG semantic version numbers.

    > > >     >     >

    > > >     >     >           I don't think this is a good idea.  Seems like a layer violation.

    > > >     >     >           What if my project use another dialect of semver, that wouldn't

    > > be

    > > >     >     >           possible with this rule.  I think this needs to be removed.

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  3.3

    > > >     >     >

    > > >     >     >             Submodules MUST NOT use revision label schemes that could

    > > be

    > > >     >     >             confused

    > > >     >     >             with the including module's revision label scheme.

    > > >     >     >

    > > >     >     >           Hmm, how do I ensure that this MUST NOT is handled correctly?

    > > What

    > > >     >     >           exactly does "could be confused with" mean?

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  3.3

    > > >     >     >

    > > >     >     >               In the filename of a YANG module, where it takes the form:

    > > >     >     >               module-

    > > >     >     >               or-submodule-name ['@' revision-label] ( '.yang' / '.yin' )

    > > >     >     >

    > > >     >     >           Should this section update 5.2 of RFC 7950?  I know that 5.2 just

    > > >     >     >           says "SHOULD".  But existing tools implement this SHOULD, and

    > > they

    > > >     >     >           need to be updated to handle this new convention.

    > > >     >     >

    > > >     >     >           But I wonder if this a good idea.  It means that a tool that looks

    > > >     >     >           for a module with a certain revision date cannot simply check

    > > the

    > > >     >     >           filenames, but need to parse all available modules (wijust to

    > > find

    > > >     >     >           the

    > > >     >     >

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  3.4

    > > >     >     >

    > > >     >     >              leaf imperial-temperature {

    > > >     >     >                type int64;

    > > >     >     >                units "degrees Fahrenheit";

    > > >     >     >                status deprecated {

    > > >     >     >                  rev:status-description

    > > >     >     >                    "Imperial measurements are being phased out in favor

    > > >     >     >                     of their metric equivalents.  Use metric-temperature

    > > >     >     >                     instead.";

    > > >     >     >                }

    > > >     >     >                description

    > > >     >     >                  "Temperature in degrees Fahrenheit.";

    > > >     >     >              }

    > > >     >     >

    > > >     >     >           I don't think rev:status-description is necessary / worth it.  This

    > > >     >     >           can easily be written with the normal description statement

    > > instead:

    > > >     >     >

    > > >     >     >              leaf imperial-temperature {

    > > >     >     >                type int64;

    > > >     >     >                units "degrees Fahrenheit";

    > > >     >     >                status deprecated;

    > > >     >     >                description

    > > >     >     >                    "Imperial measurements are being phased out in favor

    > > >     >     >                     of their metric equivalents.  Use metric-temperature

    > > >     >     >                     instead.

    > > >     >     >

    > > >     >     >                     Temperature in degrees Fahrenheit.";

    > > >     >     >              }

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  3.5

    > > >     >     >

    > > >     >     >           The example modules should be legal YANG modules.  Use e.g.

    > > >     >     >           "urn:example:module" as namespace.

    > > >     >     >

    > > >     >     >           Also, the modules are missing the last "}", which confuses the

    > > >     >     >           "rfcstrip" tool.

    > > >     >     >

    > > >     >     >

    > > >     >     >         o 4.1.1

    > > >     >     >

    > > >     >     >             Alternatively, the first example could have used the revision

    > > >     >     >             label

    > > >     >     >             "1.0.0" instead, which selects the same set of

    > > revisions/versions.

    > > >     >     >

    > > >     >     >             import example-module {

    > > >     >     >               rev:revision-or-derived 1.0.0;

    > > >     >     >             }

    > > >     >     >

    > > >     >     >           Shouldn't this be s/1.0.0/2.0.0/g ?

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  5

    > > >     >     >

    > > >     >     >           I think the module name "ietf-yl-revisions" should be changed

    > > to

    > > >     >     >           "ietf-yang-library-revisions".   "yl" is not a well-known acronym.

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  5.2.2

    > > >     >     >

    > > >     >     >           Wouldn't it be better if the leaf "deprecated-nodes-

    > > implemented" and

    > > >     >     >           "obsolete-nodes-absent" were of type "boolean" rather than

    > > type

    > > >     >     >           "empty"?

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  7.1

    > > >     >     >

    > > >     >     >           The text says:

    > > >     >     >

    > > >     >     >             All IETF YANG modules MUST include revision-label statements

    > > for

    > > >     >     >             all

    > > >     >     >             newly published YANG modules, and all newly published

    > > revisions of

    > > >     >     >             existing YANG modules.  The revision-label MUST take the form

    > > of a

    > > >     >     >             YANG semantic version number [I-D.verdt-netmod-yang-

    > > semver].

    > > >     >     >

    > > >     >     >           I strongly disagree with this new rule.  IETF modules use a linear

    > > >     >     >           history, so there are no reasons to use "modified semver".

    > > >     >     >

    > > >     >     >           It is ok to use rev:nbc-changes if needed, though.

    > > >     >     >

    > > >     >     >

    > > >     >     >         o 7.1.1

    > > >     >     >

    > > >     >     >           There is a missing " in:

    > > >     >     >

    > > >     >     >            4.  For status "obsolete", it is RECOMMENDED to keep the

    > > "status-

    > > >     >     >                description" information, from when the node had status

    > > >     >     >                "deprecated, which is still relevant.

    > > >     >     >          HERE  -----------^

    > > >     >     >

    > > >     >     >

    > > >     >     >         o  8

    > > >     >     >

    > > >     >     >           s/CODE ENDS>/<CODE ENDS>/

    > > >     >     >

    > > >     >     >

    > > >     >     >         o Both YANG modules

    > > >     >     >

    > > >     >     >           All extensions should specify the grammar; i.e., in which

    > > statements

    > > >     >     >           they can be present and which substatements they can have.

    > > >     >     >

    > > >     >     >

    > > >     >     >

    > > >     >     >         /martin

    > > >     >     >

    > > >     >     >         _______________________________________________

    > > >     >     >         netmod mailing list

    > > >     >     >         netmod@ietf.org<mailto:netmod@ietf.org>

    > > >     >     >         https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_netmod&d=DwIGaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=p8kyeK3u4ZYiaQ2ZPGqwkyXmQgBH6r5jpYiYWzhqJ48&m=ffH268c0xOd0DSFLQzZ2JHAmCHjVzPJVJtGPNxiiJcs&s=z5LiDOlko48vuqlIgA0Gm7dcsxmHOtwfod6wC46lRU0&e=

    > > >     >     >

    > > >     >     >

    > > >     >     >     _______________________________________________

    > > >     >     >     netmod mailing list

    > > >     >     >     netmod@ietf.org<mailto:netmod@ietf.org>

    > > >     >     >     https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_netmod&d=DwIGaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=p8kyeK3u4ZYiaQ2ZPGqwkyXmQgBH6r5jpYiYWzhqJ48&m=ffH268c0xOd0DSFLQzZ2JHAmCHjVzPJVJtGPNxiiJcs&s=z5LiDOlko48vuqlIgA0Gm7dcsxmHOtwfod6wC46lRU0&e=

    > > >     >     >

    > > >     >     >

    > > >     >

    > > >     >

    > > >

    > > >

    > > _______________________________________________

    > > netmod mailing list

    > > netmod@ietf.org<mailto:netmod@ietf.org>

    > > https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_netmod&d=DwIGaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=p8kyeK3u4ZYiaQ2ZPGqwkyXmQgBH6r5jpYiYWzhqJ48&m=ffH268c0xOd0DSFLQzZ2JHAmCHjVzPJVJtGPNxiiJcs&s=z5LiDOlko48vuqlIgA0Gm7dcsxmHOtwfod6wC46lRU0&e=





_______________________________________________

netmod mailing list

netmod@ietf.org<mailto:netmod@ietf.org>

https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_netmod&d=DwIGaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=p8kyeK3u4ZYiaQ2ZPGqwkyXmQgBH6r5jpYiYWzhqJ48&m=ffH268c0xOd0DSFLQzZ2JHAmCHjVzPJVJtGPNxiiJcs&s=z5LiDOlko48vuqlIgA0Gm7dcsxmHOtwfod6wC46lRU0&e=