Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-29
Lou Berger <lberger@labn.net> Thu, 14 April 2022 11:52 UTC
Return-Path: <lberger@labn.net>
X-Original-To: teas@ietfa.amsl.com
Delivered-To: teas@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1DDAA3A0FCA; Thu, 14 Apr 2022 04:52:09 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.911
X-Spam-Level:
X-Spam-Status: No, score=-1.911 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, NICE_REPLY_A=-0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, 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 9rR8C-DPFtkK; Thu, 14 Apr 2022 04:52:07 -0700 (PDT)
Received: from NAM12-DM6-obe.outbound.protection.outlook.com (mail-dm6nam12on20723.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe59::723]) (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 BDC413A0FC7; Thu, 14 Apr 2022 04:52:06 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=TAamCNY2MgQfFkzW43+EiZp9xqPlKZepCJEBY7bPUCFZho1hVmk2mRWgDCBjxF7FEqO7hj/mZ/o/C1Asf0wG2eg0ogS8wW7SPp8MFF5BpwkyxTls6Q3t2VmXQ3DeerTQtoyRFEF+i1i6RvsDPBM0r+8JX71IqfNnTN1fiNROQyp+8tx6+E9HqvGhoqBbYex4OfaPETfOJPKoOHz6uq5YnpFQsC4d8U+1KyRDIcI2N4FKGZUwkvkAIUxCTgk7aNrp7PpAo19oCnZmCe0tASfyrwa3TkuqTq00juA/RYHr4pTJCxqYX2wjiWPbFBayRwMYcKe2W5COENVEVQZel3+5fw==
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=AX5QN5u2F7E4xK8gC80JeMAw1+1zAVjRFfnXM7HGGdo=; b=EyhvFTPubDIrbpGybPUEBeXNYHk2pHk7X5KE36Mw1ydK3E/V3NHkcSutd4FXonfvbWimjpF4ebUsg6wzVjy1G9NCtk7yi9UmPI5tYOZcl+4gSjosw81REwBOa8zZ97p9ogHFJnPD0pob1QhRcZ/7t8y3GWxblygZb/mgAdFWg1aMGf39jhikoZiBBhk8vStMNLe5E3bxi1k4VE8KUCrKSgssq0hIAH6faftHuR7Iwb1nslyUxse42/Ghfr8YTO3ggC6fw0m/m58iX9ZdpsyCrZufET9HJVuH10F68sm2k5xdgUefmTKH1e2DzaRW5iAFJkWuZ+w/na0NZ8RQhgVhEQ==
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=AX5QN5u2F7E4xK8gC80JeMAw1+1zAVjRFfnXM7HGGdo=; b=j4dyq5i12f5eWiU1PbM33grZ+YwOm8aOPlErmzET0XhkoqefMxzSOi2lk1an3LHc9r49huSEo6oguaxK0KR27maHQxt1d3yIvbSRP0lSiM7FjOZb0c0brG7FRB5XVaBwnmQyDsLlTWsUfA9rtO7A9vCIeFEAT1ZzgGu7Jw4YC1k=
Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=labn.net;
Received: from SJ0PR14MB4792.namprd14.prod.outlook.com (2603:10b6:a03:379::24) by DM5PR1401MB1945.namprd14.prod.outlook.com (2603:10b6:4:58::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5144.28; Thu, 14 Apr 2022 11:52:00 +0000
Received: from SJ0PR14MB4792.namprd14.prod.outlook.com ([fe80::95a5:5cf5:ae92:70ac]) by SJ0PR14MB4792.namprd14.prod.outlook.com ([fe80::95a5:5cf5:ae92:70ac%7]) with mapi id 15.20.5164.020; Thu, 14 Apr 2022 11:52:00 +0000
Message-ID: <329c9caa-53f4-bfb2-f028-894ee8edfde8@labn.net>
Date: Thu, 14 Apr 2022 07:51:52 -0400
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.8.0
Content-Language: en-US
To: t petch <ietfa@btconnect.com>, TEAS WG <teas@ietf.org>
Cc: TEAS WG Chairs <teas-chairs@ietf.org>
References: <409bd958-008c-76df-1692-221ab8dfbab0@labn.net> <62540E64.3060403@btconnect.com> <6258023E.9010708@btconnect.com>
From: Lou Berger <lberger@labn.net>
In-Reply-To: <6258023E.9010708@btconnect.com>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
X-ClientProxiedBy: BL0PR02CA0115.namprd02.prod.outlook.com (2603:10b6:208:35::20) To SJ0PR14MB4792.namprd14.prod.outlook.com (2603:10b6:a03:379::24)
MIME-Version: 1.0
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: d97befca-6cf8-4652-5850-08da1e0d2f23
X-MS-TrafficTypeDiagnostic: DM5PR1401MB1945:EE_
X-Microsoft-Antispam-PRVS: <DM5PR1401MB194548F7855A11B8AA804A39C3EF9@DM5PR1401MB1945.namprd14.prod.outlook.com>
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: yqhxrVNQs7FRxf5AF0fs9LHZZXK5t48fGfXi7Td8py4Bp1gGygDwcHoAaXBZpQwDZYwpn8e87tFD9C0dV8liNAfQDZ7wPbyN3GipVEMHiCJdh6cP5my0sEoZz46R0DF4uHeztdmXnyWdXg89zyLABFk5hlqDun+gRXqoqbyVuVvlhzuLZIY9gE1LgZb4U3AycnAPt8UKepNPOuCV/C2jYdRHE8LZFuCHGiuvCnautL/dVRkk5k3NB7k0UonpuyQ+YTAu0pUmdPSnUbg7VvrlYlBs9Zv0gzfk5pe6hoM2dThnN+1uSw4R7D4+kHYPuErqMTkmoz/7zr9bulYswC6yOBsYCU7PyRPPiteFtFQC1EX4Kz7uNYdl1U4N7xl0VE0TdKYaKx0hWHZlblyI8UVKM6NNqsxlho1evWxGNjIUELAL/WB2bS3rS9bOO9pOV77S86cblwUw0pBgvnwJZIny11GONGrj+7WZqQeAFHAnB4ej90M3N0r1pthPljERDPqSiPxCLOFdxNWSQycF93va0tFxpBm1iXlJMn1VGLfoFfeCh178YFZ02OTuL3scT6ZVfqU4tcqTlibyoJmZvJT9PTnLgMoqTxzvLoUVoJ8LLFvcsvJVaYoK3abJ0gjuz0PiPIZql6RszRZ0TcNnhOb36C9/T9nwES0Za7rTRlyQtTSmI87BGSOLFGvuaGpsy4KqF+41LL/fe6CwARG/sGbm5WJM03JYOKsUDi8TudGSpUs1M0Ftpmiuw5m4rPSdViKAJn9KYUn3G4yV3dxlsLpEgeufTteVGmb3ie2qRRTHEp1QID9rWtO3EIui40PBUFTrr8mO5xEatvb/e1t2ifWDjRgg6r7iw1cigmOv/lv7WRymJlCXpFl3jZw6v6i2a/sS
X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR14MB4792.namprd14.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(39830400003)(376002)(396003)(136003)(346002)(366004)(5660300002)(2906002)(36756003)(83380400001)(6666004)(186003)(26005)(296002)(316002)(110136005)(6512007)(31686004)(31696002)(66556008)(66476007)(8676002)(53546011)(38100700002)(6506007)(38350700002)(66946007)(52116002)(4326008)(508600001)(6486002)(966005)(86362001)(8936002)(2616005)(45980500001)(43740500002); DIR:OUT; SFP:1102;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: tyCwdCcDW/TXmIoTRriocBBJnz/E6s7j9spqCDZ4lNLLT3AbXZ84vrOGZYkgW4vZYyXhUFAD8yZt42O/bW13TLxyik5GIakwj7joeXf86l6cvL2futQzqLYzpylfdaErfXHVNGxLE/d4drV0+PXvjPzeKYpqAeIaUCWsUzyDS6P/uz4AuHbzF+8XU/1+dB7Unc0m6jqDc2Z3L4tX/zV7Xs3iEKEQGKTyNoPnlmJ9G3jRhVjIXPJXBYncDKdjgn7UVh0+8SAM2ZV+bONzbbHjfj4tHR33LJKuL1Q4nzS1YvlQbKTU8ckKxCvQpIioXzWXn1AAPsyZ0NINVoUn2pXjZxU1RJRBlGdpVOpa0vpZlLe7zcv1xnIBTZjSzJU2X8HehmUkzBHHSRjSbTbqvn5YqHlahl31JdFYUOCSpQCC8HOueE5mEGr7fZcnF/i7h8o4CfhJVpIqick/Zf3IwKKPIwq6xv6S3OWHvmyzfdcJQW/Qx8aNybN8o9aVhaN7vPeysHrRvwUtHTWkniC8CL2ZS51JJxHu6t1YdOYuMY+Pl3lTbk7ue6CTPrShzxJf3k8o0achy036kLnzoEOJ23GxjEBeULJvEHmNi8rkyCgyU5fwo42EviaGeLlD7sXN+c5bnSQXe3q3QqEk5jL7a1H+vkOh7g4T8qX/L+jwpSMA6fSG+Q16tJVJQK29IgLqbmk6XVos5F/RYYM9E3i4juXqYacc7Bn59UY8KhXBgyzNvHRLf/PMVdqIdfmBo4Xo68weM9nZw+4sE/YklSTpLLOskSQr/nJUV1RshdQ/f73Z7R1wlPT/zKKX9XYRpN+nnKk95YenqSly6vmkd71piPy5FdOt1MO87gQD1r84jn7fUyn8PNVtvO9YUyOwTuS2FJMEZrQzx/F3l0JbK03aVdc6H+qFWL9aglfJ4TQjRrOiAHdlgvuQjLGutI9QO2DknCi/H4EgtOTfcpKOvDEmnUfW1yh76fOXOpCaYNhfoP8GU4I5+dHx6YmXVHueEUZmC9HUhiIfeVZxJ5xywQzL4g5h4tRiXigKhV9K/jhHA96X95poVamwf8FSpZ7ww9N3BkvmSTX/gl+QQNBJQCV9Eg6f6YBQzbjmpKBBKk1+o4svFXbF4EoYAEW3U7huCVEtWoqA2BdQ+OXSWJRkEPm5NxshbAXxSqNSjD5t4N3iWi60uepyw7QYwLdYJkCqnnaY/PjISPiw3qYnNUYNAqIuM22MfUFjrRZJRV9nHqDPHtXwZ2sUf+YOi9Jet+HY/LwqpIHUPcoIfzd1/BWGZT/gk+PZnSbExjSeCREXdTD1RO/00AuUP8pZgvSx8j/1YrynTMMwmURKmN+bLSmp55hUT47nL5WcGAeeT5QStw27CyHn3O3MgDRLoN1XzEdMcO5LNle1dZ95XQ+KX2Jh5rk1qebIUeRj7EFtSmHzp2snfFuTRlWSB6FQzbziU7q4DeKbQZRz3w57KJbviSkdSp+LHRJSE5d4FZm5++I4ZSK0uQ6RvAmskbCVMsuuNy0NIYOtwZv8zJyY8Fw+J5NMhiF6Za1K9AlaVeWAzYHQk8bD/ZrcPgO3Wm/ZgV11No2bJbqaM+XZphqbTX5uFwp66MA+3EJ0PyZjdOmpkZN7psbtyW9cNtXkaTjx0ySdzXDep7bCoXiPXa08ZYlk7AHqYASrPgSN2wGi3dyvtbrwpzE4xIu0DjEzndEXWcRnlfZg/2fQhZSMylMjiXD05mUKoEkGc31ZPA==
X-OriginatorOrg: labn.net
X-MS-Exchange-CrossTenant-Network-Message-Id: d97befca-6cf8-4652-5850-08da1e0d2f23
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR14MB4792.namprd14.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Apr 2022 11:52:00.2474 (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: /+B5nWWhSPmu+0HoEjmeYqNpYO1HyghPPN/New1JY5ODveh0ReeqAleS1Vtwyp4valxC+hQX5bh/xeG27IwFiA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR1401MB1945
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/_yR5yQWMKlmiKlrn61OTPEeNuis>
Subject: Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-29
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/teas>, <mailto:teas-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/teas/>
List-Post: <mailto:teas@ietf.org>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/teas>, <mailto:teas-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 14 Apr 2022 11:52:09 -0000
Tom, Thank you for this (well at some level ;-) Authors, Can you work with Tom, perhaps even schedule some time to meet - assuming he's willing/available - and come up with an approach to address his comments. Thanks, Lou PS I still like my page numbers so use the htmlized link on datatracker, e.g., https://datatracker.ietf.org/doc/html/draft-ietf-teas-yang-te - the .txt version also has them... On 4/14/2022 7:15 AM, t petch wrote: > On 11/04/2022 12:17, t petch wrote: >> On 21/03/2022 13:28, Lou Berger wrote: >>> All, >>> >>> This starts working group last call on >>> https://datatracker.ietf.org/doc/draft-ietf-teas-yang-te/ >>> >>> Given the size of the document and this week's meeting, this will be an >>> extended LC. The working group last call ends on April 13th. >>> Please send your comments to the working group mailing list. >>> >>> Positive comments, e.g., "I've reviewed this document >>> and believe it is ready for publication", are welcome! >>> This is useful and important, even from authors. > I have not reviewed the document - it is too complex, largely due to the > way in is constructed - but have seen enough to think, 'Not ready for > publication'. > > Security has the concept of number used once, soomething which makes it > harder for anyone else to know what is going on. The equivalent in YANG > is 'grouping used once' which I often see YANG Doctors point out and ask > to be removed since they increase complexity, reduce ease of use. This > I-D does just that and goes further, nesting groupings extensively which > makes it even harder, verging on impossible, to understand. > > I start at a high level, as in Section 5, intending to 'drill down' to > the lower levels, leaf and its type and description, to see if they make > sense, if they do what is wanted, and the highest level is good; the > English needs some tweaking but nothing I am unclear about. > > With s.5.1.1, I get uncertain - where is the description of path > in-segment and path out-segment that feature in the tree diagram? > > With s.5.1.2 I know that something is wrong. The YANG snippets are not > those of this I-D; they would help if they did but they relate to a > different I-D. Thus the YANG has, buried in a grouping used once, > leaf description > leaf admin-state > leaf operational-state > s.5.1.2 has > description > operational-state > encoding > a related, but different, module. > > Likewise, the YANG has bidirectional after dst-tunnel-tp-id in another > grouping used once, not in this snippet. At which point, I cry 'Hold, > enough'; I cannot trust what I am reading. > > Another problem, besides difficulty of understanding, with grouping used > once, is the difficulty of relating the different parts of the I-D, > description, tree diagram, YANG and so on. The obvious solution is > comment lines showing where globals ends and tunnel starts, in the YANG, > in the tree diagram, in the introductory tex. That is pointless here > since in the YANG, they are adjacent 'uses' and at a more detailed > level, globals, tunnels and so on are all jumbled up. > > The IETF has abolished the page number so you will have 40 > undifferentiated pages of tree diagram followed by 40 undifferentiated > pages of YANG, but with objects in a different order, courtesy of > grouping and grouping used once. > > Tom Petch > >> It has been a few years since I reviewed this so the >> Copyright 2019 >> seems about right. >> >> However, reading Adrian's comments brought back memories and indeed, my >> notes say 'three e-mail of comments' which suggests to me that they were >> never acted on:-) >> >> So >> - there are ten references in the YANG to RFC that do not appear in the >> I-D references (used to be four) >> >> - abbreviations need expanding, I suggest in s.2 with a separate heading >> to the current s.2 >> >> - http: needs to be https: >> >> - the TLP are out of date (that's new) >> >> - the format of the tree diagram is clumsy - identifiers are split in >> the middle, across lines >> >> - revision should be 'Initial ..' not 'latest' >> >> - /constraitnt/constraint/ >> >> I will plough on - my target is by Easter, ie COB 14th April >> >> Tom Petch >> >>> Thank you, >>> Lou (Co-Chair & doc Shepherd) >>> >>> >>> >>> >>> _______________________________________________ >>> Teas mailing list >>> Teas@ietf.org >>> https://www.ietf.org/mailman/listinfo/teas >>> . >>>
- [Teas] WG Last Call: draft-ietf-teas-yang-te-29 Lou Berger
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Adrian Farrel
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… t petch
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Rakesh Gandhi
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Italo Busi
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… t petch
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Lou Berger
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Tarek Saad
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Tarek Saad
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Adrian Farrel
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… julien.meuric
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Tarek Saad
- Re: [Teas] WG Last Call: draft-ietf-teas-yang-te-… Tarek Saad