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> Fri, 18 February 2022 15:57 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 AA0473A07EB; Fri, 18 Feb 2022 07:57:43 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.9
X-Spam-Level:
X-Spam-Status: No, score=-6.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_HI=-5, 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 ZaV_d_3L29wr; Fri, 18 Feb 2022 07:57:38 -0800 (PST)
Received: from NAM10-BN7-obe.outbound.protection.outlook.com (mail-bn7nam10on2072e.outbound.protection.outlook.com [IPv6:2a01:111:f400:7e8a::72e]) (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 D3D533A07AC; Fri, 18 Feb 2022 07:57:37 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K72nYldYZQ0KAAMRbNsvwzeuvFpEZH9TsS8axVHTJgILqxotA71dkLFde7m/x6hYPP5L6zOQoDE4ChAZikIRxiawYq7oXQ4G3rj1HyTNyE2doau+C2AQGJWt+A+QP1Xl/y67S0knu4DrxssWgik2tTuo6bO3+pzH1l+BfG2nVddFyQ95IV/TVLthvelydUksBnww9K0L7ln0BCqqj4kT9uNKfa9HOxoASZiRLqTTydVR0VlCfdz++MkjThhxxBt8wsPzZu4YVBkzCKZ8RsNQl+j2BccDGNfJDOIPSqD6oOv6sZHeulkp9iZpk+h8BfddY1HZmfqnOFcu8bPsss6t+A==
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=KBoWR6tMFHm6yV4zz8mVbB8EgPQRmvkStCUu85+IabE=; b=E0TiQDPK2nrh+9TM+u5qOu18YoDF2HtW8vttee29pImnhrflOQcXWJbkGTvTZD6ERMZ7fNOFauUKH0Q3NYbVz6p/MTq2m+bfmR7sWlVdt6YCNoEW8PuIncjyJ/loD5d5xYxnhnODhU2dXmptzFDGWpCoivV58C/pKTB92SpsQJDFc4x1ZFpG4mTGDM4wA8EjJQmEgwzenLytZ6R54FQEGgPEe8Tha9I0tCiwY9EGVUKF6ge6TF/DDqIZKkhvmJT9StopnQQMvxddCaOorN+8XwKoFYW4kt1WCVer1Yujf1U227i9t9OktLFbT5PMx292BTCkiayi8HXLA0clv9bYMA==
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=KBoWR6tMFHm6yV4zz8mVbB8EgPQRmvkStCUu85+IabE=; b=HkGMY/0FoAHgJlCCU8CUfqnKqhexUxjOnLxECf/XwRqtJMnUmDaLPurkcmHyEWhgRpGPhCrzcLEGlnmAo/XltTZWyovT83k/V9va+Pt9ZQVoRowUh4bPtwVYLp9kVqhQ5RLPXepydD/07p9PkwB2FZFetoIzed5bU7j0bWUqocg=
Received: from MN2PR14MB4030.namprd14.prod.outlook.com (2603:10b6:208:1dc::14) by MWHPR14MB1102.namprd14.prod.outlook.com (2603:10b6:300:89::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4995.16; Fri, 18 Feb 2022 15:57:31 +0000
Received: from MN2PR14MB4030.namprd14.prod.outlook.com ([fe80::4ce2:9d0c:ff32:928d]) by MN2PR14MB4030.namprd14.prod.outlook.com ([fe80::4ce2:9d0c:ff32:928d%9]) with mapi id 15.20.4995.024; Fri, 18 Feb 2022 15:57:31 +0000
From: Don Fedyk <dfedyk@labn.net>
To: 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/mCAAXLNAIAABy7wgAk64MCANvKogIABvJng
Date: Fri, 18 Feb 2022 15:57:31 +0000
Message-ID: <MN2PR14MB4030082721A510DA7A466FA6BB379@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> <MN2PR14MB4030A544B79E73DE20248DA4BB529@MN2PR14MB4030.namprd14.prod.outlook.com> <620CF703.9030605@btconnect.com>
In-Reply-To: <620CF703.9030605@btconnect.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: 1f28c958-df86-4934-2b03-08d9f2f75ef0
x-ms-traffictypediagnostic: MWHPR14MB1102:EE_
x-microsoft-antispam-prvs: <MWHPR14MB1102085B9BC6BF354EFBFB56BB379@MWHPR14MB1102.namprd14.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: zytP+ch7eWjkGve0CnV14QkcPl6MXRbK+/bhYOnw0wjf58Gyn5LHR7zy+JStdAHY7SzMA6Tbkl2gttkmwMpLy+y3gY72ChqNJDG4Fy7X9gQs76SCBNwLIbrnSbdE6hIFwpSDOsGrah/YygbFgPxwXkAFo5T6qhDtAp4VSXXK2z9SnOjtbyL3wbWPTGVPlwYE087M2RLnn7GCcKlg7jWeftp8chJ5JUZzte0yuF/eBt5P5s7MfH3EQMo6nlRQ+LudQ9+3i8FOjEsv9f1Mv/rhIrPV+mhPjhUZldLzlAyRHsPkCZDfoRI+48Rwu8RiJyZYEHayCC4us3cUyDxg98Fxe6SHogsvYZTKztxDBRnjdaI/5CKupZfTdj1nN2RNhFeGB0+7RPi+nK+v9mNA1pcmGDw28+sHivuH2UpfVJm5RpAWWLtELdhkNjj9sm54KAieM/3Pa2fkdZgwilylCXulTbJlg+5XRcVONw22nYk/WhMtAJFG9nRJThmTSRj7sPLntsQNueUDgL7KrenCS+AXbVRN4a9cupYk3/Z+n91sBgEImUfSMhAwpmBY2SckLZHggw5m93Lqfef8PNerpff7d4VB4coWlAjxSm9XflAjypPhnKDaln/LauUy2D16JeC7OWEDOLBWyii/aWtURBD6uHmYGzHEW6PgK8DQwc3tT0vuA3aY+Jz7XuvAayOoc8w/kyU5WG8cgEGiFC00dE/BLbZwGvvc1OH8ORq6Y2OJHXcCcZEibuEJjMnvrZ2QpmdNwOSqjDrgxoDu1KGzsGOcVF9u/GPUh7lg89qHMmC7jBU=
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:(13230001)(396003)(136003)(39830400003)(376002)(346002)(366004)(38100700002)(122000001)(33656002)(296002)(316002)(76116006)(66946007)(54906003)(6916009)(66556008)(66446008)(5660300002)(8936002)(2906002)(66476007)(86362001)(55016003)(64756008)(4326008)(8676002)(52536014)(9686003)(83380400001)(186003)(53546011)(966005)(38070700005)(71200400001)(508600001)(7696005)(6506007); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 8YgHm2iyghD+vHoitYVOEydCu8x91guXvcKWgby4m+E5KJ1Xn9DOLxZ9EVd4pJN2HStVkZJB+oZ9o28TYhcVFDZZu1JG6kgUXfM0025QM2tfvskWLlYxQakA+ZKYqR5hRYH60IKHdWbXKXLeyXU/H2TaAa3H5XpMAZe0jLjjhtTP6gGW0piRw+lwlLIRP+cUAoNAKocWrrUxTyfWCkYBivnHHwYn5kU6TQFhwoPMx6ziLwBinVu4X2qxzjogutf7O6DTMfSYGf4A9lmq+2F4VP1wPiMVjXqlmlNFyRO0StrKHekVpXjdtoGRTOw/ZQ7/8FWRv9y+p5NqDdSWAWIIoZnrnntmfFhi5b+q/Wz2Ot2OsqEzPqKG0K2JRT6dPPLR00EWfsOk6n13XN/4MCjJ+UQ5pPvGqaC1HqIQXGhLfL7KieIt/YrYDkNGB/ANLMzchaxYFZac5nopzQ6kV+WH1m+woTgsjWasgw67xtjZUlFxYrwja1NcUhwxU1s+sJg/8Y5f9gtlMVNUengBqFG7HLaLoh1A8By39m4a35LxTTYP7rOe5Tq+JMMJMR70MchohXx+H9oSrkidUqahdH1XmInEPNkbk+c/JHrQp2FCl/I+aIQ2kkk15CgwghndY3ja+noUwZZ8mfdN39bzoSp/2GuK09GUe9ovd8LEdAhRt8VeOq0ph4Itee9MAp/KBF/bJxNrfAWnBYyL07ajMWPSo5ZOQDb2zEN3MS7+afTMexDclzv7+1PxQ6RJ6WtemHFsZPHzok/fkj6J95D/7CoVCMi1GMxtRKqeb3SAaKpUuy+RrjqFdm0PvS7BrOPfPOATPXaAMZbwO29USCwkjmygqnRzox3KybUKjasFD/YpTigo9fGLJTNum97R+CsBnwGM+n7W9xTA9Y07IC7WHJ4a5MYPShPCL7FQInnmqF/dIbrj5TECkrSeuGg+35B9zmXhc+5OaZjEsgbk9SVnsLW1hPVoPDViboiD2ePxhFRCdUtpKbJT7E2/Pm00e8Aaq1016Icgg3H8nnYZHkEuoIgH+m5vgcLwIGv3B5Mp8Jrv+e4X1BJ+ccWGvW83XE/0Yzmz/8hCaC5vI3QBieQdhuepPlerjM7L7sPQTnAkMYrN4rbat9VtlabtmiXpHsbIIkWyjUirSExqpTy/cf/tNuxek4QFPa3tdHgNqecjbvwRTQjHsPR8Frv0X+0X3vQaxcsSoWjN0EU7O0BmaKbLUyLPGqvOIsYTUsvr1QyX6Jxp8vQ4kRIKHaUp14LsSMJWNTBe/NudAjjMXvqZGNXTwNIm0c1J4nXUrVn5tLdlsUGdbL/MusgYlSEJNi7zN9i69eA83L5gQio7vUvRbLbkkgKfOKkOiu66kV85eK79QFL972YkpIsYx3dmAfANQPWJ/adrbA1JogJMAMdY9R1FdjBVdsEGgghbiZ9vXLcr3B82S8XD8XsmAEmKaKjvC/Qv9DvBgCqAMGOpPnjMRQ0YdgxkzEqWmFjytuzp548ew5Vmg2tcYdDrvqMpUyI/ScWTzdgKb6/pqAUAn2UaAEhVV7wuqgPVEbLs4W13A8FMtaWy9T6kfCxAO3SwsWYEq1aRXhmGQFuziXAH1ZuqrXb7wwf4xz3ImybAwRhwxjlIkyEZDPDjDA8+aUghkypr+ze1f52icLOi+SBShDKNDGJYpwf1mqfqchJnsW/rxPQihsQ4nMQ=
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: 1f28c958-df86-4934-2b03-08d9f2f75ef0
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Feb 2022 15:57:31.2027 (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: JZaqUaC0l3iaGuSm3vXDM4OZk+xLFU7XUtZLO8v12tAHMmFLlIuWfuTJ1NTupg/BHeVN0Nlw8i1spfqlBKHr+Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR14MB1102
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/ETrMhqAjk2uScomBEK4I__L8cXs>
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: Fri, 18 Feb 2022 15:57:44 -0000

Hi Tom

Looked into this and found RFC 7224 iana-if-type.yang uses this: 

module iana-if-type {
  namespace "urn:ietf:params:xml:ns:yang:iana-if-type";
  prefix ianaift;

In the examples the string "iana-if-type" comes from the XML namespace not the prefix declared. Detnet YANG does not import iana-if-type.yang but it imports ietf-interface.yang module that imports iana-if-type.yang. When validating with Yanglint I have no control of the display "iana-if-yang" in examples - I use short form prefixes for those on input that do not show up in the Yanglint output. Yanglint expands the prefix used in the XML test files to the namespace. 
While we can add a reference RFC 7224, it is a namespace not a prefix that shows up in the output. Every object in the examples has its specified namespace. You pointed to a part of an example using ietf-interfaces namespace which prepends the iana-if-type namespace just for the typedef object. 

Cheers,

Don   


-----Original Message-----
From: tom petch <daedulus@btconnect.com> 
Sent: Wednesday, February 16, 2022 8:07 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 12/01/2022 14:07, Don Fedyk wrote:

Don

I am not sure where this is in processing terms - I see no Telechat date yet.

I have (finally) looked at the examples having been put off by the size previously.

I see that they reference iana-if-type but
- there is no I-D reference for this IANA-maintained YANG module and I think that there should be
- the wrong prefix is used - the module declares a prefix of ianaft so that is what an IETF document should use.

I am still looking at the examples.

Tom Petch

> 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