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

tom petch <daedulus@btconnect.com> Sat, 19 February 2022 12:23 UTC

Return-Path: <daedulus@btconnect.com>
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 04E9E3A095E; Sat, 19 Feb 2022 04:23:43 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.615
X-Spam-Level:
X-Spam-Status: No, score=-2.615 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, NICE_REPLY_A=-0.714, 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=btconnect.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 b5ftLnIHaDy4; Sat, 19 Feb 2022 04:23:38 -0800 (PST)
Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2112.outbound.protection.outlook.com [40.107.22.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 66D433A0963; Sat, 19 Feb 2022 04:23:36 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=k6cbLZxa2VJE5J9YYW49EDoFR9+KJ3NrB7JIrlRCvHDseZT70oSWSPGR6ugzWvRrdyogGqlHwrpmQdSdjM7yCY/wVy5R5341fOv9+memqgQV1CTSvZaukmTTCeLrJByegxMDSAjWcuu2lbNJ/BuPZbTICR+6C9yFV71MHQrnEm752J0t3OvEP+vXFXxCSpO4qJOAHpMFe9fSwPP0UQHqmVoNSxEQDhB+I4b9l3du8haZFjF81LQ7neVWIjI/XJuzVTLe1Wb0uKmawuGN/WnzHGFgPmtdavPS9NzHSALpeVqSxRZV9jW/oFZyZeQ6rqZHzVvQPw6QecccOv67zJrJMQ==
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=D0d+9pnoMI9KD+P9aKIBcwN+TlfH9YCUz/DGLLxMNuY=; b=fJsa+2IB38PkWt9Wz9OOprWoGIOrz8x7KcabKNAdXrl98NVP+mqqZPGxgVZdjVcmm/GMaVdNhAO/xLOw7IOqxlbqf8TPb74ZOPn46BDuhXbpAawCEgofJjmOfDHwCJXgnHFtUjWPSKJa88MFL1UOwgCGqeTzRBXEbYB74IyBOvhSd09yoGneT5f2omdN6AXOv93xRBRLLXuYJSHGQR5ijeyuqppkpNfXJNWzUsnOJKzGz5F4whByhaO9N47pv0RDm2tY3FOtE5Mao+TeZaijiPPhEpzv46c9tI8JybSjHFZ9/qO6ShuMAI/IyyuDq7dIJbq6KFbzoL5geoRpV8wHbw==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=btconnect.com; dmarc=pass action=none header.from=btconnect.com; dkim=pass header.d=btconnect.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector2-btconnect-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=D0d+9pnoMI9KD+P9aKIBcwN+TlfH9YCUz/DGLLxMNuY=; b=fHXKMGhu2oZL5tbFpQb0kbQzAJFbrP2yZxSarqWuE/2uXd8rS/ywVIGBW0u32iAagK4X/22oRmEx8XIC4tMQvWI0oK0H9Sk8tezy7TJXKZFyU+E0SLi+11AlKbiYbcV0HJWWqwRq83hBnBDrRmhJdDBCrASzL4oK6Jb7hLL9Ptg=
Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=btconnect.com;
Received: from VI1PR07MB6704.eurprd07.prod.outlook.com (2603:10a6:800:18b::8) by PA4PR07MB7229.eurprd07.prod.outlook.com (2603:10a6:102:d5::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4995.8; Sat, 19 Feb 2022 12:23:32 +0000
Received: from VI1PR07MB6704.eurprd07.prod.outlook.com ([fe80::5d5a:a0db:c614:596f]) by VI1PR07MB6704.eurprd07.prod.outlook.com ([fe80::5d5a:a0db:c614:596f%6]) with mapi id 15.20.5017.015; Sat, 19 Feb 2022 12:23:32 +0000
To: Don Fedyk <dfedyk@labn.net>
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> <MN2PR14MB4030082721A510DA7A466FA6BB379@MN2PR14MB4030.namprd14.prod.outlook.com>
Cc: "detnet@ietf.org" <detnet@ietf.org>, "draft-ietf-detnet-yang.all@ietf.org" <draft-ietf-detnet-yang.all@ietf.org>
From: tom petch <daedulus@btconnect.com>
Message-ID: <6210E141.4020607@btconnect.com>
Date: Sat, 19 Feb 2022 12:23:29 +0000
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:38.0) Gecko/20100101 Thunderbird/38.5.0
In-Reply-To: <MN2PR14MB4030082721A510DA7A466FA6BB379@MN2PR14MB4030.namprd14.prod.outlook.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
X-ClientProxiedBy: LO3P265CA0005.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:bb::10) To VI1PR07MB6704.eurprd07.prod.outlook.com (2603:10a6:800:18b::8)
MIME-Version: 1.0
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 1d5110a4-6729-4249-33e5-08d9f3a2a4ba
X-MS-TrafficTypeDiagnostic: PA4PR07MB7229:EE_
X-Microsoft-Antispam-PRVS: <PA4PR07MB72295D287FE898C193A173B6C6389@PA4PR07MB7229.eurprd07.prod.outlook.com>
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: 7U1MvzEsxqFMiU/UCe+gHLsvqT/2CIUoUnsBs/1N0Qu/7Y3eBTbWK1Tyvx9UnAAM6V36nKZtBpfYBnVWOEWw1UI4CFJxc2AA0439WUEOCgIGL0oiCphHJVpJPx/PiIAh8yVIQklPNiPKMVPk15gwoDhaxBXyBcgQF49oDJ6DISS9HIfDiOj5VY+noETwtPre4JAAZGvg4bYhZ1jgjQHKPFh1pR8o4Xox4fYTlqX6crl1BaIgDCPi1YrE/VPhXamW6cVLFklRBDVblpzlS5FpXyJh69cpxCjtTWnq2RK6c4/v1OOCnlzELpoA8V8TjrsI94Q5IvTnivCEDCTyqFpgutVLTpPz5sVHCe8YZav10B7HGojVcqIsCoO0vORx+sR4NQltInb4D2yrkZVsTHqPsYgjRd/In5RieTe2GwWWsQq3Qx2lKLfDvSdgjTa297Xp9YfM9BeUprWL5mtQDduzjlHnXU8dNZocH7EeHxoY1OzOohce6UmsZBprE9xIc77Q59WZRhpDwsVMmI7n5Ep4/lJS74dDmjQVDHEmivNFhuNWjGZseK9DadJ66WEIF3UiTyGx46MJHVkBppP4z3+JQI5wErHMb94uT5QmvIrsZ0Wl2EbbEOxr7hD4y5cgN4NurUgUFQqB+sZf3pIjzOk07APUF/ly30rNNgcXHQV++y18bS/E7vx+JJrDDX6QI4qrgXsvF41z285gR/fxMNmHMIIhqhiiVHAg/rlcHpwj7MS9NcgW1NPTE8xJQFObbYKi4hlPU7pIrljTHm0pY/d05A==
X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR07MB6704.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(366004)(86362001)(6506007)(966005)(38350700002)(316002)(38100700002)(83380400001)(52116002)(53546011)(26005)(6666004)(54906003)(15974865002)(186003)(6512007)(2906002)(2616005)(6916009)(508600001)(87266011)(36756003)(4326008)(5660300002)(8936002)(66556008)(66476007)(66946007)(6486002)(82960400001)(33656002)(8676002); DIR:OUT; SFP:1102;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?M1E2TlNWclBlQXRiZFpvOWp5eGhNa0FyVGRQbndLWkFEZDh2MTcvR1JwWTlE?= =?utf-8?B?d295aDRiSUxrZkFCbytZZU1rVTRqaDF6YU9FMlpCNDF2MXRvcWtkei9rMmNw?= =?utf-8?B?cHVWMTc0WGpDaWhzOFZtZEdudndhRXJ6ck1NQ2Y2SVE3SjRwSmNvL0NkNUV3?= =?utf-8?B?NkpiTkp0RnBNbDdqN3Y1cHZrNFZQUEp4QnhwYVNxZEsrcFozdkp5Y0ZDSWg2?= =?utf-8?B?bU52aldJamhiOTZmSnMwc1l1cWlaUk5YTVpiTjJUa25BZjBUY0hLSCtuYXdv?= =?utf-8?B?d2ZCNWZhUkFCelVTM1ZKeE5vWm1xdWJsSXFCblV5VWJCbm1kRzVYRitTSTh5?= =?utf-8?B?d2V5YnkxT3M0ZlE0NGk3REdKVzQvNkphYmsxQkV6T3dnRVRDZUVhMy80UWxa?= =?utf-8?B?eTBUS1hBb0MvN0IvMXB1WFU0c0cvVTAyeVlha2RNZXpvTjlmQjc3QzcvcnQ5?= =?utf-8?B?RzB0blFFTUw4L25DUkppalorMldjbGtmZjY0bk1MNmROdVhpTXFCUjhhbVBt?= =?utf-8?B?L3dWVEZuTTFLaE51dWhmdURjVjY1WTA0UHBsTFJXWWNYVXN0eWpNL1RHWWR2?= =?utf-8?B?NDQrRzFobTlYQ1NKcklmOStoZG15a1JYQ2grSHUraGhCdDdPTUJ2TzVWanhu?= =?utf-8?B?dFlPQTg5cnFzNGpGbXoxNmNNZUlNcUM4cVhwNWgvaXdBWXg3bk40M0ptQXU0?= =?utf-8?B?b3BZVmVobW5rcTZCUG54dzZyTm5uSnhwZ3RxS2lxVW5KNFFoU1pjK2JOMHJP?= =?utf-8?B?Y003N2ZqUEZyWmQrWCtUNS81SDY1Ri9RRmZaR0RBTjlmU292R0l0WVNQVzJ0?= =?utf-8?B?ejY1Qm9oWkRRSnd3bXEzMlV1VEx1NU9CNC80MzBGRC8rK1d6YUxaVStYUGR2?= =?utf-8?B?amdkbWhucWluRExySmxZMHZmbWNlVHRYUHpHbEU3Wnl6TjlVV2syMUY5aUJv?= =?utf-8?B?WGFGTUR3VkRkNGx1UHhqQjBFOHB4VjlxMFpJbWRQbG1ycGs4Y1dzMVVDSTEz?= =?utf-8?B?MVlCUU8wZ3R6d2diaW5RR0JWRCs5bEpkOTBmbmtQSnZYbVNrMUJCcVdRWjdy?= =?utf-8?B?a2w1Q0xIckJURVVuVVRjRVlxRjlPYzZOOW9DZTJ4LzUxMkhMSUJ1MGR2bjY4?= =?utf-8?B?eFRlM1F6M1BCdmlXN2NjME9nY1pqekFpRi85QUtLK0tLQ0c2bWxqTHQyNE9s?= =?utf-8?B?WkFDQlNxbGhaOGJaUFlGc3orU1NFanJCMktnOUFJYVljcGt4cnVGbUt5RS92?= =?utf-8?B?cERxTStyNnRFNWZnN0RRMHNaV0llZVE1WmRucmZIdG1vSjhLSlUrYW02eXVz?= =?utf-8?B?dXhna2pDNkxMNEp4UXRUTWNPZ0VoTGYzcEN0a09YTjk3NzB1NW1DL3BkZEVU?= =?utf-8?B?eUI0azFaYUFyNjhsdnJOckk4d1JYemdYOU5OWmtxS2x0L2RhRDAxSjlKTFRo?= =?utf-8?B?cUFVWEdkUGRxVzBaSTk5SHJwM0NOQThOMzFUYXdpMXJJZ3dHMjl0bGFwY3E5?= =?utf-8?B?dTdHUEp5UkRDbG5pRVpiVm1aZlRidVFMa1pGQjE0YnE0V3hLNGdBQjVmUjVT?= =?utf-8?B?dEZud0tKRzdQU0VjRi9MdHRiOGdaY3doZXU0VTkrODdUczRHdUlFamxXWUNM?= =?utf-8?B?OXBBZTJJOW1Kak9GYzhwRlVCQmlVeG4ycW41MDRGb0k3V2dEUjlEdE5GOVZu?= =?utf-8?B?ODdaZjF5U3pFWW5vVWJpRXA1b0QyV3REcW9NQmQ5c2ZMVGlFelcva2lQNmZ2?= =?utf-8?B?a0tEUlk5TE1xWldvbjIyMlU0VWhDb3hqZnJsUFh4Z1M4VVpaMU9yOTdpck5m?= =?utf-8?B?czFmMzM5Z1kzcWJNWkZyMHROb2Qyd1AyUElveHZHRUFta1FwSmVCSDhJUkdS?= =?utf-8?B?cVFnSWVIb2Z3MTBHdFdTSS9IcG1nUEJmMmtSRGprbVdVMUNCaEtaOWMybXp2?= =?utf-8?B?OEdnZ0hmaXZHV09rclNtQ1IyYXo4NjdzRjVrNnowQnN1SVBKU1B2YVdydjVw?= =?utf-8?B?V2kvZ2hGL3h1ZU9jM2VoeHpYd2ZJdkRwd3RISzRSRW83TlJoaXVlSFJZVDMy?= =?utf-8?B?NmpPb011TGlPZWpxTmphdzVGeW03SzFab2pDdzU1YlJ6bHdCQis4dVJyeWFw?= =?utf-8?B?THpFZUNtcGpKamM5UkVnMVhPcmdBMFlVSlE1eTE3ZEJLWWg0Tm83eGQ2NEFY?= =?utf-8?Q?Q3u2N1xhI3lqVDzdX4wX9Sw=3D?=
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1d5110a4-6729-4249-33e5-08d9f3a2a4ba
X-MS-Exchange-CrossTenant-AuthSource: VI1PR07MB6704.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Feb 2022 12:23:32.5428 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wXTDDiWQ71ms2L68FJHbH9xYDE7G76ngQz2FQXAO4bzqxpFLNOujnxWjLVSGfDffwc/QNJyxXq3Rezc4s9Qkqw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR07MB7229
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/Ljf0fux04nYQISe3h1Txdg-nyq0>
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: Sat, 19 Feb 2022 12:23:44 -0000

On 18/02/2022 15:57, Don Fedyk wrote:
> 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.

Two thoughts.  First, the IETF advice is strongly that when an IETF 
module is referenced, then the prefix used should be that that module 
itself declares.  Since this is the common practice, then using any 
other is likely to cause confusion.  Yes, tools have to cope with 
deviations such as 'xmlns:ia=...' but that does not make them right.

Second, iana-if-type is an IANA maintained module so RFC7224 is no 
longer relevant.  Any reference should be to 
www.iana.org/assignments/iana-if-type/ .... (where the prefix is 'ianaift').

Since I see 'xmlns: ...' in the text of the RFC-to-be, in the examples, 
then my take is that understanding the RFC will to some extent depend on 
being familiar with iana-if-type.  I think you need a statement at the 
start of the examples that the examples include references to YANG 
module 'iana-if-type ' [Informative reference to IANA website]


Tom Petch
>
> 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
<snip>