[Netmod-ver-dt] Comments on draft-verdt-netmod-yang-module-versioning.txt (May 16 version)

"Sterne, Jason (Nokia - CA/Ottawa)" <jason.sterne@nokia.com> Thu, 23 May 2019 13:57 UTC

Return-Path: <jason.sterne@nokia.com>
X-Original-To: netmod-ver-dt@ietfa.amsl.com
Delivered-To: netmod-ver-dt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A522412008A for <netmod-ver-dt@ietfa.amsl.com>; Thu, 23 May 2019 06:57:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.91
X-Spam-Level:
X-Spam-Status: No, score=-1.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=nokia.onmicrosoft.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 nvjnaemhU9pi for <netmod-ver-dt@ietfa.amsl.com>; Thu, 23 May 2019 06:57:32 -0700 (PDT)
Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70131.outbound.protection.outlook.com [40.107.7.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 047D612004C for <netmod-ver-dt@ietf.org>; Thu, 23 May 2019 06:57:31 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=weMWIAWbDL87zdAAsruZRUx2CE999u2DB1d6UG8l5vM=; b=caeUo2cSZSLsR1K4WJFPsD+8m/h4GMhqYKqPlnbdSMYGWScKhhE5QzwmR6GI48oSyVmoVlkoYyfqZUTdQojC0/nQAS49vCVVQH40A3Bx/Z5uBiirzYCn+WpI6YyD6m42+5NvsTbOjkQfriOn8q1KHdwWcc/IPw7CJU/q5nQFi/Y=
Received: from VI1PR07MB3981.eurprd07.prod.outlook.com (52.134.28.141) by VI1PR07MB5439.eurprd07.prod.outlook.com (20.178.15.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1943.10; Thu, 23 May 2019 13:57:28 +0000
Received: from VI1PR07MB3981.eurprd07.prod.outlook.com ([fe80::25cb:d0f3:dd14:3b68]) by VI1PR07MB3981.eurprd07.prod.outlook.com ([fe80::25cb:d0f3:dd14:3b68%6]) with mapi id 15.20.1922.016; Thu, 23 May 2019 13:57:27 +0000
From: "Sterne, Jason (Nokia - CA/Ottawa)" <jason.sterne@nokia.com>
To: "netmod-ver-dt@ietf.org" <netmod-ver-dt@ietf.org>
Thread-Topic: Comments on draft-verdt-netmod-yang-module-versioning.txt (May 16 version)
Thread-Index: AdURb21sAkuH8qSrRduJlr3VI5Xo1g==
Date: Thu, 23 May 2019 13:57:27 +0000
Message-ID: <VI1PR07MB39811366C619F54ECD73EE3D9B010@VI1PR07MB3981.eurprd07.prod.outlook.com>
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=jason.sterne@nokia.com;
x-originating-ip: [45.72.251.26]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 8d730b17-5bbc-42df-c47f-08d6df869780
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600141)(711020)(4605104)(4618075)(2017052603328)(7193020); SRVR:VI1PR07MB5439;
x-ms-traffictypediagnostic: VI1PR07MB5439:
x-ms-exchange-purlcount: 1
x-microsoft-antispam-prvs: <VI1PR07MB5439C61A91A6270CBA51831D9B010@VI1PR07MB5439.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 00462943DE
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(396003)(376002)(366004)(39860400002)(346002)(136003)(189003)(199004)(99286004)(14444005)(256004)(81166006)(81156014)(486006)(186003)(66946007)(478600001)(26005)(76116006)(66476007)(66556008)(64756008)(66446008)(73956011)(7736002)(8676002)(53936002)(74316002)(68736007)(6916009)(476003)(316002)(2501003)(6506007)(7696005)(606006)(102836004)(25786009)(2906002)(2351001)(6116002)(66066001)(3846002)(71190400001)(5660300002)(55016002)(71200400001)(6436002)(790700001)(86362001)(5640700003)(561944003)(14454004)(966005)(52536014)(8936002)(9686003)(236005)(6306002)(54896002)(33656002); DIR:OUT; SFP:1102; SCL:1; SRVR:VI1PR07MB5439; H:VI1PR07MB3981.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: nokia.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: U6edybJhT2Aix9EKXuaQJ9/Bg995yaEUFIU1xKJf5oxEeto8mb179iCBOQccKIXxRdjiXCDazuZm6fGxASm7v9LudoYTWAjN/1/ARNxWUmpbzd2WONhch/iUps57rPXJ4LmCrGC+3XLjgKpBpm/gbEwBPYJ36T9e+uOkrrxKfXxWSsUmaOa3qBZvQvoXFWPOyGGpGEcFExprYWyc0j79LkHLv9yduUU2x1JpvqSbyzNpB+JA+CR0J1UcI9Lkq3H7p8YGzNu8pIOvlYcJCTRVX10ptCEXE/nHrpYPGx1qZMK1qUH5ENqyD0ydeqkmRVOVA7F1BKXr9x+kKydfQXTOAqjEwauE8pPc/r0m0iWXziLKctTF9TGaFueXOTZUvcX9CUltIOz+wskPA3Xsrw6OsSM3cyu7B2koVBaWu5nC2Q8=
Content-Type: multipart/alternative; boundary="_000_VI1PR07MB39811366C619F54ECD73EE3D9B010VI1PR07MB3981eurp_"
MIME-Version: 1.0
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8d730b17-5bbc-42df-c47f-08d6df869780
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 May 2019 13:57:27.8994 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: jason.sterne@nokia.com
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB5439
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod-ver-dt/Ugn4JNDTO-62-sBINqqHcBrpzTc>
Subject: [Netmod-ver-dt] Comments on draft-verdt-netmod-yang-module-versioning.txt (May 16 version)
X-BeenThere: netmod-ver-dt@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: NetMod WG YANG Model Versioning Design Team <netmod-ver-dt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netmod-ver-dt>, <mailto:netmod-ver-dt-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netmod-ver-dt/>
List-Post: <mailto:netmod-ver-dt@ietf.org>
List-Help: <mailto:netmod-ver-dt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netmod-ver-dt>, <mailto:netmod-ver-dt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 23 May 2019 13:57:36 -0000

Hi guys,

Some comments on this file (May 16 version):
https://github.com/netmod-wg/yang-ver-dt/blob/develop/yang-mod-ver/draft-verdt-netmod-yang-module-versioning.txt

(A)
Replace this abstract:

   This document specifies a new YANG module update procedure to allow
   for limited non-backwards-compatible changes, as an alternative
   proposal to module update rules in the YANG 1.1 specifications.  This
document updates RFC 7950, RFC 8407 and RFC 8525.

with this (to avoid the implication that we are simply trying to "allow" NBC changes):

   This document specifies a method for documenting non-backwards-compatible changes that have occurred in a YANG module. Guidelines for managing the lifecycle of YANG modules are also provided. This document updates RFC 7950 (primarily the module update rules), RFC 8407 and RFC 8525.

(B)
replace this paragraph in section 1:

   Specifically, this document recognises a need to sometimes allow YANG
   modules to evolve with non-backwards-compatible changes, which could
cause breakage to clients and importing YANG modules.

with the following:

   Specifically, this document recognises a need to describe when a non-backwards-compatible (NBC) change has occurred in a YANG module [optional: in cases where the NBC change has been deemed necessary].  An NBC change can cause an incompatibility between a client using an older revision of the YANG module and a server using a newer revision.

(C)
In section 2, remove the following sentence from its paragraph/bullet and separate it down into its own paragraph fully left justified.  It isn't really part of the list of 'updates' to RFC7950.  We're just reinforcing that a rule is still in place.

As per [RFC7950], YANG module revisions
      continue to be uniquely identified by the module's revision-date,
      and hence all revisions of a module MUST have unique revision
dates.

(D)
In section 2.1, replace this text:

   This section updates [RFC7950] chapter 11 to provide a more flexible
   approach that allows for some non-backwards-compatible changes during
   YANG module updates.

   Where pragmatic, updates to YANG modules SHOULD be backwards-
compatible, as described in Section 2.1.1.

with this (the guidelines can recommend to avoid NBC changes, and what we're changing here in this particular section doesn't really directly allow or support NBC changes anyways):

This section updates [RFC7950] chapter 11 to modify and clarify the definition of a non-backwards-compatible (NBC) change, and to allow a method of describing when an NBC change occurs in a YANG module.

(E)
section 2.1.2 -> change ' brekaage' to 'breakage'

I'm also not sure if the examples of NBC changes makes sense to include here. Especially the 2nd, 4th 5th and 6th bullets which are clearly described already in section 11 of RFC 7950.  Maybe just keep examples 1 & 3 and any others that are somewhat new based on our updates, or aren't terribly clear in section 11 of RFC 7950

That's as far as I've gotten for now.

Jason