Re: [Detnet] Comments on detnet-yang-15 Re: [Last-Call] Yangdoctors last call review of draft-ietf-detnet-yang-14

Don Fedyk <dfedyk@labn.net> Wed, 12 January 2022 14:07 UTC

Return-Path: <dfedyk@labn.net>
X-Original-To: detnet@ietfa.amsl.com
Delivered-To: detnet@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7EFDC3A0EE0; Wed, 12 Jan 2022 06:07:53 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=labn.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 FaI0x6tS1xpj; Wed, 12 Jan 2022 06:07:49 -0800 (PST)
Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2112.outbound.protection.outlook.com [40.107.223.112]) (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 F35873A0EDF; Wed, 12 Jan 2022 06:07:48 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IxuqWjgOT8ezzzAwAAaMvh4LnZeJPX5Rt0tQmHothXBSmeJuKO9B7XqtDC8ETXOuJ/iZHB7fvn40DWDPBcawzxC7pFMvl2VrS1/+mbGi4QtUxVQmgWeginbxADeFQCHckbVYBO7r5rnzqf6s9uQ8mizvy5bjUMnep6gKSjO+W0yr97WrdN07WaCVKn66jV9yuHwwaMi8gQEOqVv5qm256jcfes8kiHmNzC+bur2+1EgvhHpRnB7WpWsvpXRkohbPHAgJ93W/9Lg/N+a0UMUQcycO6j3M4hXFTizwIVaOmQQ9a2Ty5rEe+pSOkxzMHGkl+3w/7n5NA+8YgQZxhp76Fw==
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=P6cbJaqbnQq4AfVR6QHnj+z+qo09tyvEud+8gDC6/0E=; b=PAgL2q5mbxrk2e1eCFMIV8SQwVZvlWEwCxIzb46UE5wZ8EDQTMUdC5s3PlXmpzqJxptbJwlJlYM687zOa++UxQMbI/R4OQlpEEqQLm8biWEwj/sTRGj5QUTApQnDyNuW3CIuauiyeWR7r55LmVwbiTOefhAoUfvq+lZS3gs7oMbSsH9y8XGVmUPmzul+uwii15lMWtt29I67SkWRTMpkHFztHthrikUJGKh4kknmxT4SX9QIgr4/HvkAJFQoUXnWlIbrMp9Vlx135XwQLmQC00aJQUKReS5A4mYvMa203v752lXg+9ZMhjkp8UkYX+fu3nlF2j3ZR2+m6PFncolxkQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=labn.net; dmarc=pass action=none header.from=labn.net; dkim=pass header.d=labn.net; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=labn.onmicrosoft.com; s=selector2-labn-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=P6cbJaqbnQq4AfVR6QHnj+z+qo09tyvEud+8gDC6/0E=; b=Jmb6sQwTzC23Ukg4WsHHHsevRCOJN+dmVFLUdZIN+4SAA/Qzhf91/tmkR3s3Aoxeb5UYoHrKQP7OUm9VkroDijlwE0kVjrmFSIJvcDdLQFkSt4kFOQPwzx3vsG8vNM102dRCx5VrdvrC8a62s2U8CodQgg2BrhO/JddsvioGoeg=
Received: from MN2PR14MB4030.namprd14.prod.outlook.com (2603:10b6:208:1dc::14) by MWHPR14MB1391.namprd14.prod.outlook.com (2603:10b6:300:b8::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4867.9; Wed, 12 Jan 2022 14:07:44 +0000
Received: from MN2PR14MB4030.namprd14.prod.outlook.com ([fe80::29a7:f994:54f5:b486]) by MN2PR14MB4030.namprd14.prod.outlook.com ([fe80::29a7:f994:54f5:b486%7]) with mapi id 15.20.4888.009; Wed, 12 Jan 2022 14:07:44 +0000
From: Don Fedyk <dfedyk@labn.net>
To: Don Fedyk <dfedyk@labn.net>, tom petch <daedulus@btconnect.com>
CC: "detnet@ietf.org" <detnet@ietf.org>, "draft-ietf-detnet-yang.all@ietf.org" <draft-ietf-detnet-yang.all@ietf.org>
Thread-Topic: Comments on detnet-yang-15 Re: [Last-Call] Yangdoctors last call review of draft-ietf-detnet-yang-14
Thread-Index: AQHX1aVVzOJCkt7zEEO2OKLaojNpkKxAOHCAgBTg/mCAAXLNAIAABy7wgAk64MA=
Date: Wed, 12 Jan 2022 14:07:44 +0000
Message-ID: <MN2PR14MB4030A544B79E73DE20248DA4BB529@MN2PR14MB4030.namprd14.prod.outlook.com>
References: <163648840575.19216.561627083204230489@ietfa.amsl.com> <61C46025.5040800@btconnect.com> <MN2PR14MB403066E8D1953043A52D3FE3BB4B9@MN2PR14MB4030.namprd14.prod.outlook.com> <61D71AE6.80502@btconnect.com> <MN2PR14MB403051ABB0AFCAFEDD34E1BBBB4C9@MN2PR14MB4030.namprd14.prod.outlook.com>
In-Reply-To: <MN2PR14MB403051ABB0AFCAFEDD34E1BBBB4C9@MN2PR14MB4030.namprd14.prod.outlook.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=labn.net;
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 89fc5b9d-ebf2-4eb2-0699-08d9d5d4e7b1
x-ms-traffictypediagnostic: MWHPR14MB1391:EE_
x-microsoft-antispam-prvs: <MWHPR14MB1391BD81AC3F4067C755F29BBB529@MWHPR14MB1391.namprd14.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:8882;
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: s5r1pi/PgPXwuR9r3WqjvUYBVUSUvFlGYmXPJyR3pyaKjdrge/czJZv046C2H0ia+MMv1yLF8naX1K4dUWt8/a1mC2UClOGCeGIrB03xgeA5wvU8BzDuMtYcBmD0+Kn4TUlFtHUZHogjOuTlk5u3Jj2pKere55B5BNsc5LjNd/zzR+MdZ8Q3pmMwDHhPU9PyNd7rWPzTTWY0jj14XwID9oISVjWpXZpKz3J3Rtz1eB2HDpxhRFnB7RguwBnFI7cNwGHGbLPlt7xWib0MQKgu3OZRp1XCnhyKhlMepnnWEXipkW/i8jUn8yuHx9kpsYxwlcU8P8wkmsX9uq93iJ7CWbAl23DlRQL+jm+hdpVtnVEWeo87ZhKqR6LAixLq/AChNqGBv4fc2Qr6VzotRlJTJ7Jdgyq8zhYCXAjj2adiXLqInfDgdNp44LA+ng7SfFA6Q54VGHzDD28NLVPL66nTVBTwg5HoL6XM0AI0lX8avie4DzHbR2gcS34ST/X3qN1aTTAZKPC1VtBZZqdT5WD0zO9KmfGzHU3w6EL0jsGw/eSEx02MbbfqIcJjdC5u1riuFLSvaV8fcNSS7iQAyv4UnT8cBhNa9EiBs5BXWinrF7M3w2JVOdZjzsGJId7X96+U5a0edbtrB0YQSOiux3dVG9+Fz51vO/9re9auFXXWc7bux3svb6n/H7FSBm0XuJyaobzVKeBBMjCWgM37QDaSnpL/bhO9PazHZavgWTJupTRCsLYTIOmOFkAr9r56VPXyboJqSJRGJ54NZEx6CTFjF50hLONcfhn1cCCva4yK6ls=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MN2PR14MB4030.namprd14.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(7696005)(2906002)(86362001)(54906003)(110136005)(316002)(4326008)(508600001)(33656002)(5660300002)(53546011)(966005)(83380400001)(6506007)(8936002)(186003)(71200400001)(52536014)(8676002)(38100700002)(26005)(38070700005)(66446008)(66556008)(76116006)(64756008)(66476007)(66946007)(9686003)(122000001)(55016003)(296002); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?utf-8?B?VW53VkIxVE50Z0VFUTgwaHpoQ0p1ZXlJdy90SUtKVzlJTU1VNjVlUGUvNUMw?= =?utf-8?B?Z2k0bkFOeDNieEluZW9Qa0xDcW1ZZ1RiY2FJMGd4RW1ya2t6d2hVME9xcGt3?= =?utf-8?B?aEVUUDlWVXNjSTkvNVJPbTIyQUZPVXlSZUViVWdBbVJTZUlOcTViMHFvVWx2?= =?utf-8?B?RFJjTlVuaGVmQWhlamJ0S1ZSa3I4Q1NUemVQeENRVVBWRGRaRURadTRuQTdZ?= =?utf-8?B?d2xLckg2TFhtWEVqUERsVFAwcEI4eVg1emFhSkhJL1Y5NWpLSVdUZ3VBWmQ2?= =?utf-8?B?TnVYZ2tjZ0RIL1o4VXpaYytTYjVpaXA3ZzhialFhNnhBWmM5aTZtRmd6VkpS?= =?utf-8?B?YVNYcHlhdHV0TWdKUk9oZ05UdWdJOEc5eFVHdVhvVm9zQzZkNGlzaVhLRWhH?= =?utf-8?B?cW50WjhMMFhLUFRNamFJYlNXbm9lNkJKbFp4V3o5S3BxaE9Lb3ZWeTU4RHQ1?= =?utf-8?B?ZjhGdExqV3h6TENaSWpiSzA1R3pwdi9hU2txY2h1cUFKRGZSSDlBbGJGcjBY?= =?utf-8?B?ajYxcHVnQUF0N1hPV01OdVdESlkzTENvT2s3MzZoLzArZnA3YW10d1U1ODFC?= =?utf-8?B?RkFLRGFFbXhMMlJMUXNJVzZTSGcvMS8xR2ZhRVdHTTZlU0ZTaThkbllkMlBG?= =?utf-8?B?VFBaZTZNOHZXcVRiVWNBbHpjZnNrc3NZUnlabmRFWHh5TE82S283aEpWWC8x?= =?utf-8?B?NFZmNWlsWWFRUFRJdW55alVhK0hVNURmNlFwSm56Q2dlMTV6akh1TUhKaU43?= =?utf-8?B?NWlrR1htKzA0K1lIVVhMd0drMmpwV0c4RWZmVWdOSnM0dXpxWXpyT2p4VE1E?= =?utf-8?B?YzRQQjdmZzhDWW1Penl3MFNPd21ONHM3cWM0dTEzMk96dGFyWDR6cVFxeGZY?= =?utf-8?B?bVFkS1VJaCtFNjd3QU14Zi9LU25pWlpHa01zTDJCRHQvZnRLUkRxMEg1L1ho?= =?utf-8?B?b09nVUJNVnFndzJQWlNsM1pqSVM3eDhzQjVSNG9aQkVidXVpaGRkTmk2NUNp?= =?utf-8?B?Q1pjOVVOMk5VQ050bVVTTUd4UWF4Z29kWUJKTXlFMzB1UWgvemlUUTRqM29O?= =?utf-8?B?LzVqOFFnS05rNWNqZG9KQlJ2dTJRRW8wT0ZLRXZqQnZ6cDFCcjQvS1loWXk1?= =?utf-8?B?RW00QkJ0NUdLMXVjTHpZQzNEUXVwMFNrTFBYV2MyZGZia2VlSGRUdm9Mci9F?= =?utf-8?B?QU5FeDRUa3Y0UkI1UWxOTFBpYlk4ekdrUjVtZE1uUlh0S0R0VVB4amRtaERE?= =?utf-8?B?T1ZtU3prSXJwMVExczVMSHBhTkMyM0krbi9VM2VDNExIeXlGdXBTemtnMit4?= =?utf-8?B?dVdTanZETm1VSk5BUjBYV0pZbzByM2hQUHd3L09JYzVEMitaZWNwU2E5blpR?= =?utf-8?B?Tk92YmVtNjlwemxWeWxpZHVNQ2RidHJrS1J6RXhGbVRNZGUxa0Jvd2lkbjBG?= =?utf-8?B?YXFUWW1PZDZWLzR0c0lhSWVSSjRvVHJUOXhkL2pDb2NWaUZWQloybnhzZE1C?= =?utf-8?B?VUl3Ui82NUliUVp2SSs3Vjd0QjgwaWMzeVQ2SmJEMlkzc3phQy90QUVOWkt3?= =?utf-8?B?L05iNVk2SnM0anBqbkVPbVRSeW0xVFJKaklNbEFjQ3EvdkhRQkR6NkE0dGdi?= =?utf-8?B?Y0ZoS3hPYnhkUnVyRUVtYnA3MDZIaGVFZ3RZV0hLb3ZzTVVsaFA4R0xhZ2M3?= =?utf-8?B?UzlpZU8wWmNtamdEcUFXQmlyUm9mb0FxOTlDMEFzeEtmN05OTUVEckVSYXZE?= =?utf-8?B?d2JjVkRsclNncy95TWllYVFMM21oN05YR0RWeEZpQXRkVWdLUEVqNjg1ZDQr?= =?utf-8?B?Wnk1S3FBS0FRTGhKYWFKR2ZNTXk0ZWMxMUQvVU96Y1lRUTN3eFFjMG5iUk9k?= =?utf-8?B?RkhacEZVWU5tZ09LT280VFBZMnFTMEl4bFFudEVySzBFRTNLZURTSVI0MERQ?= =?utf-8?B?RERtL01QaWlRUGVKdGpBSEo1UnlHT1pLRm1Kd2M3SmR2UmRXS3E3MTlWakkv?= =?utf-8?B?RXN0U2hMQkJ0NHVCZ2JsT01SanlNaXc0QlQ1ZmtWVkNBNTFjR01SZHkzMHhq?= =?utf-8?B?aDhBRkZxNVNjeUg2L1RMVzV4NDVjOXJQVFpQSk8wVVpYcUl6OFRZcE5RQWM2?= =?utf-8?B?cmpNL01lNVdKMXAyNE9WL2xndi8yTVZOOW1NdENvOXRwWXRtVVNtcDVQL1ZY?= =?utf-8?B?WFE9PQ==?=
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: labn.net
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MN2PR14MB4030.namprd14.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 89fc5b9d-ebf2-4eb2-0699-08d9d5d4e7b1
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Jan 2022 14:07:44.5151 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eb60ac54-2184-4344-9b60-40c8b2b72561
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 9krRYwFkFDbxE7gQyY+s1fPH5BuakAaW8MzmpTM3m2tZrycgPTSOM6ip83mKEmXemSC0Qk0gB8E9mgy9hWbdnw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR14MB1391
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/X9UeFVhumlxsBJW_tPj7JswLzug>
Subject: Re: [Detnet] Comments on detnet-yang-15 Re: [Last-Call] Yangdoctors last call review of draft-ietf-detnet-yang-14
X-BeenThere: detnet@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussions on Deterministic Networking BoF and Proposed WG <detnet.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/detnet>, <mailto:detnet-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/detnet/>
List-Post: <mailto:detnet@ietf.org>
List-Help: <mailto:detnet-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/detnet>, <mailto:detnet-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 12 Jan 2022 14:07:54 -0000

FYI

After Tom pointed this out, I did find the new site under author tools that generates all formats correctly including SVG from the latest version 3 xml. 
https://author-tools.ietf.org/


Cheers
Don

-----Original Message-----
From: Don Fedyk <dfedyk@labn.net> 
Sent: Thursday, January 6, 2022 2:01 PM
To: tom petch <daedulus@btconnect.com>
Cc: detnet@ietf.org; draft-ietf-detnet-yang.all@ietf.org
Subject: RE: Comments on detnet-yang-15 Re: [Last-Call] Yangdoctors last call review of draft-ietf-detnet-yang-14

Hi Tom 

Ok got it, I needed to update pyang to version 2.5.2 - pyang --ietf checks the copyright boiler plate and substituting Revised for Simplified created an error in version 2.5.0. Now it is correctly processed. 

Interesting the Trust and legal Provisions (TLP) comes from an XML tag ipr="trust200902" for the document which locally for me uses the older boiler plate text with Simplified, etc., in the draft copywrite (the YANG copywrite is pure text so I corrected that). My tools for xml2rfc v3 are ~8 month old which is the environment for processing SVG. 
However, the same tag on IETF submission substitutes the newly revised text so version 15 txt and html is corrected. But, the PDF version gets the older text since I submit my rendered PDF along with the xml because the IETF does not process XML to pdf quite the same way and the only way to get the SVG diagrams in PDF was for me to submit it. 
 
Thanks
Don
 

-----Original Message-----
From: tom petch <daedulus@btconnect.com>
Sent: Thursday, January 6, 2022 11:38 AM
To: Don Fedyk <dfedyk@labn.net>
Cc: detnet@ietf.org; draft-ietf-detnet-yang.all@ietf.org
Subject: Re: Comments on detnet-yang-15 Re: [Last-Call] Yangdoctors last call review of draft-ietf-detnet-yang-14

On 05/01/2022 20:04, Don Fedyk wrote:
> Hi Tom
>
> Comments Inline [Don]
> Also incorporated, https://github.com/detnet-wg/draft-ietf-detnet-yang

One follow-up in line

Tom Petch

> Thanks
> Don
>
> -----Original Message-----
> From: tom petch <daedulus@btconnect.com>
> Sent: Thursday, December 23, 2021 6:40 AM
>
> Borrowing a useful cc: with a related topic
>
> two documents in the YANG module need adding to the I-D References
> RFC8938
> [Don] OK on this one
> IEEE 802.1Qcx-2020
> [Don] I see. Yes
>
> contact URL
> insecure, obsolete - https://datatracker ...
> [Don] Fixed
>
> Editor: Mach Chen
> not in the authors of the I-D
> [Don] Fixed
>
> TLP out of date
> [Don] TLP?  I changed the date reference in the XML header.  It should line up with the module date.

Look in the YANG module and the TLP refer to Simplified BSD License [sic].

This changed in 2020 - I cannot recall an announcement thereof - and the new one is Revised BSD License as in
draft-ietf-lsr-isis-yang-augmentation-v1

Tom Petch


> Reference: this ID when published
> Reference: "RFC XXXX: Deterministic Networking (DetNet) YANG Model 
> [Don] I think the RFC editor correct this on publication.
>
>
> Tom Petch
>
>
> On 09/11/2021 20:06, Xufeng Liu via Datatracker wrote:
>> Reviewer: Xufeng Liu
>> Review result: Ready with Nits
>>
>> Thanks to authors for addressing the previous review comments.
>>
>> The updates look good. The followings are a few additional nits:
>>
>> 1) In the model, “container flow-spec” has been changed to “container 
>> traffic-spec”, but the description has not been updated, shown as below:
>>
>>            container traffic-spec {
>>              description
>>                "Flow-specification specifies how the Source transmits
>>                 packets for the flow.  This is the promise/request of the
>>                 Source to the network.  The network uses this flow
>>                 specification to allocate resources and adjust queue
>>                 parameters in network nodes.";
>>
>> 2) Most names of list and leaf-list have been fixes. The following 
>> three were
>> missed: “leaf-list member-apps” should be “leaf-list member-app”
>> “leaf-list member-services” should be “leaf-list member-service”
>> “leaf-list member-fwd-sublayers” should be “leaf-list member-fwd-sublayer”
>>
>> 3) Section 10.  Security Considerations would need to include a list 
>> of “sensitive or vulnerable” nodes.  RFC 8349 shows an example.
>>
>> Thanks,
>> - Xufeng