Re: [Teas] [Lsr] WG Last Call for draft-ietf-lsr-isis-rfc5316bis

"Les Ginsberg (ginsberg)" <ginsberg@cisco.com> Wed, 03 March 2021 20:47 UTC

Return-Path: <ginsberg@cisco.com>
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 A2DFC3A1A9E; Wed, 3 Mar 2021 12:47:07 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.601
X-Spam-Level:
X-Spam-Status: No, score=-9.601 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=h4ZSlPpk; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=zGO1WoQF
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 NDVJnVhqquRB; Wed, 3 Mar 2021 12:47:04 -0800 (PST)
Received: from alln-iport-1.cisco.com (alln-iport-1.cisco.com [173.37.142.88]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 826F83A1A9C; Wed, 3 Mar 2021 12:47:04 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6120; q=dns/txt; s=iport; t=1614804424; x=1616014024; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=J4s5sBoSf8ji7cOS3HLQDY0bdo5QDsPoisPvOn/iNWo=; b=h4ZSlPpk0NKa20rD8eG05JtCpgaNDSVQgGnaf1/J/9DDKdszfVDmrbHh vcn7ePGfwuv5lxYUGlG0M8UmFSKKcEH5c9Cff/5sRrGFVIaY3NFZnzFeF qbrI4qRVi5v9R+JjXPYYspyqIqBB7qEJ+7ofRPzKKXtYj+hMDZFBu75TT M=;
X-IPAS-Result: =?us-ascii?q?A0A7AAA99D9gmIwNJK1fAxoBAQEBAQEBAQEBAwEBAQESA?= =?us-ascii?q?QEBAQICAQEBAUCBT4FTIy59WjYxiAkDhTmIVgOBBYkfjn2CUwNUCwEBAQ0BA?= =?us-ascii?q?R0LCgIEAQGBEwGDOQKBegIlOBMCAwEBAQMCAwEBAQEFAQEBAgEGBBQBAQEBA?= =?us-ascii?q?QEBAYY2DYZEAQEBAwEBARsdBgEBLAsBBAcEAgEIDgMEAQEBHhAhBgsdCAIEA?= =?us-ascii?q?Q0FCIJoAYJVAw4hAQ6jLAKKJXSBNIMEAQEGgUdBgxwNC4ISAwaBOIJ2g3eCY?= =?us-ascii?q?4NzJhyBSUKBEUOBWUk1PoIaQgEBAwGBXh8mgwOCK4FPGlsGPiYBA1MiOSoTN?= =?us-ascii?q?DAIDh+TWaVNWwqCfIk/jTWFToM3ik+Fe49VlFWLPYMCjxgOhDgCAgICBAUCD?= =?us-ascii?q?gEBBoFrIYFZcBU7gmlQFwINjh8Zg1aFFIVFczUDAgYKAQEDCXyLFwEB?=
IronPort-PHdr: =?us-ascii?q?9a23=3AsbdeJhC0+/XoAZ/s22G4UyQJPHJ1sqjoPgMT9p?= =?us-ascii?q?ssgq5PdaLm5Zn5IUjD/qw00A3GWIza77RPjO+F+6zjWGlV55GHvThCdZFXTB?= =?us-ascii?q?YKhI0QmBBoG8+KD0D3bZuIJyw3FchPThlpqne8N0UGF8P3ZlmUqXq3vnYeHx?= =?us-ascii?q?zlPl9zIeL4UofZk8Ww0bW0/JveKwVFjTawe/V8NhKz+A7QrcIRx4BlL/U8?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-AV: E=Sophos;i="5.81,220,1610409600"; d="scan'208";a="656423362"
Received: from alln-core-7.cisco.com ([173.36.13.140]) by alln-iport-1.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 03 Mar 2021 20:47:03 +0000
Received: from mail.cisco.com (xbe-aln-003.cisco.com [173.36.7.18]) by alln-core-7.cisco.com (8.15.2/8.15.2) with ESMTPS id 123Kl3BR011267 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=OK); Wed, 3 Mar 2021 20:47:03 GMT
Received: from xfe-rcd-002.cisco.com (173.37.227.250) by xbe-aln-003.cisco.com (173.36.7.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.792.3; Wed, 3 Mar 2021 14:47:03 -0600
Received: from xhs-rtp-002.cisco.com (64.101.210.229) by xfe-rcd-002.cisco.com (173.37.227.250) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.2.792.3; Wed, 3 Mar 2021 14:47:02 -0600
Received: from NAM11-BN8-obe.outbound.protection.outlook.com (64.101.32.56) by xhs-rtp-002.cisco.com (64.101.210.229) with Microsoft SMTP Server (TLS) id 15.0.1497.2 via Frontend Transport; Wed, 3 Mar 2021 15:47:02 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GKaM1toH7Gz21BsMeUhx/C2yVvgCf8osmXqkVl+00w3AV+w86XB3WdIj/HtAH8NW2AzFJWUNcZ7dZhNsCKrVrk/2621pVZAvBV7maSxAVkTCt8qVENhVf5SPQztXpswnOi6Nmx2+1f0lD+ZRTfPpUi7VuMJRZhCxo9HPn9/hnl+KXLuW4hvTGCfSiuHzOppdlp/8XyPYqprI/v3kDOetT8gMsz9J9mLK0zv9oDLvmktyP9HNKxAmciyFCUJblnbjEl9omHtd2/knt5qhVdPeF2uKeuPOFe6Lt5c3/2laSzxNA0qkZG3lKU2pmXklOVWeiX8olimxSWHeVa4tXjlv5w==
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-SenderADCheck; bh=Ki33AfQoFkZwiG3FnNZ74IOUSKjrwA0ZlarllgjK6zQ=; b=IYkbdpXu59R/NjiMN9LQmy3WFlTPURSmpDmZotPkPOcpVfCbc98//pFGrBLMPqgOeEaTnK+fUpZYrh0jI2W+n0J0Y47eP6C3GEZscYtzWVjybmc+Lhr8Ydx4YAcGnkMUIwA1Mqwle07WE06WdaNJu/7gLcxegevaVaYHW7p4p6qTxi+5YdgTJgefeI6EUxQLCgaVnjH/84ci6PBxvuKUa1qeZqgLHzDyfDS4go8bkwV+L2l5sT0IZK+uvmrCKQ0AWuy87uVMuj5NZIu25Mcnv5Bj/5s6ZxaEEjXtuBM+IQiTQkM5CjIbpwjvgertu/WDhYanyUNbKFT4vhqai+1Ubw==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Ki33AfQoFkZwiG3FnNZ74IOUSKjrwA0ZlarllgjK6zQ=; b=zGO1WoQFU6+zTjWd7kDn/+VdRXHBH/NsxEvP33WAt8N10myySrnVizkDQL9fncrlqIcj7Hv4VcZiT059idlXhnbSucmows/BTq2F1hhj4MOYLfFhF/gj116+2YfKRmObIAMHoKdzE0DM7cHfPUUZWYSLNO5QHSK+Fo2yXC+HF+k=
Received: from BY5PR11MB4337.namprd11.prod.outlook.com (2603:10b6:a03:1c1::14) by SJ0PR11MB5166.namprd11.prod.outlook.com (2603:10b6:a03:2d8::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3912.17; Wed, 3 Mar 2021 20:47:01 +0000
Received: from BY5PR11MB4337.namprd11.prod.outlook.com ([fe80::19ab:fe94:ddeb:1ccf]) by BY5PR11MB4337.namprd11.prod.outlook.com ([fe80::19ab:fe94:ddeb:1ccf%6]) with mapi id 15.20.3890.030; Wed, 3 Mar 2021 20:47:01 +0000
From: "Les Ginsberg (ginsberg)" <ginsberg@cisco.com>
To: Donald Eastlake <d3e3e3@gmail.com>, Christian Hopps <chopps@chopps.org>
CC: "teas-chairs@ietf.org" <teas-chairs@ietf.org>, "teas-ads@ietf.org" <teas-ads@ietf.org>, "teas@ietf.org" <teas@ietf.org>, "lsr-chairs@ietf.org" <lsr-chairs@ietf.org>, "lsr@ietf.org" <lsr@ietf.org>, "lsr-ads@ietf.org" <lsr-ads@ietf.org>
Thread-Topic: [Lsr] WG Last Call for draft-ietf-lsr-isis-rfc5316bis
Thread-Index: AQHXBUHqugdPY9/1MUCePpVar7tizKpyrJmAgAAXvSA=
Date: Wed, 3 Mar 2021 20:47:01 +0000
Message-ID: <BY5PR11MB4337C87749153805EC6B7AA1C1989@BY5PR11MB4337.namprd11.prod.outlook.com>
References: <A31F6308-B1A4-4CD7-AC71-BB6722CAC1A7@chopps.org> <CAF4+nEHc=ZsLMAijh94PFg8C23_BSYP2VtJNPmFLAxst8K9P-g@mail.gmail.com>
In-Reply-To: <CAF4+nEHc=ZsLMAijh94PFg8C23_BSYP2VtJNPmFLAxst8K9P-g@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=cisco.com;
x-originating-ip: [2602:306:36ca:6640:2837:c6c9:caaa:f699]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 52b33ef0-13ec-4781-56dc-08d8de857ef0
x-ms-traffictypediagnostic: SJ0PR11MB5166:
x-microsoft-antispam-prvs: <SJ0PR11MB5166E7210FE3AEC885B965B4C1989@SJ0PR11MB5166.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: rrcBwARpCjKo3KoCUH6He0sbZBSb4GNpsx2k4xxFLAxpjCavK1Bto6oQlX3+lubhO2J/+AzrdFaJ92c1tp66u4dXDcOe23sxspjgih8Qq2n1Dvanf/fCWgYeM2ZGf0hXlRft2PtKhctBJkR50e/j8GvjqB7+vOWClXTqJQFsevnnky/kp9Vgx7nu28k5CxLg9Y7ZDf0AHykl/jPPD+Q38kipu+JN4LToQR4Bh8wer6OlxDgIinbLpUka08O/e1BbjYjOZsrv4ht1YKGt4PPzEr0BCBylCJOWQV2gUulUTIoK5TSsanodBdnthVrJM+zk3/LphieiXpwIdwLg/8ciNc2MhqKI+kJMYipmZBXPQ++zQN0/HkDia9en6o8DCNmW7Xq4HSSNE0HEPKt0XqGhJQpHYmW7MwpGTGYO8bhicQ51j90HLx3gtU1/jBtSKZjBPyFAIOoLRmpcGtlmJZ/zNk2XNyYMUIIMnxhjoR0byEnab2pTDqiDBFFbJOEqgbInSNP+2fOYgYcEbLN7bSR8Z2oPDWRuJu30msUfc9zK7XrLdEGSUpslCByf4mC36aMNGO1G8eKYjvwmKlPBd/NwaKURWUuAktHfEAMh28FPmX0=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BY5PR11MB4337.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(376002)(39860400002)(346002)(396003)(136003)(110136005)(33656002)(76116006)(478600001)(54906003)(8676002)(66556008)(66476007)(66946007)(64756008)(7696005)(52536014)(8936002)(966005)(66446008)(2906002)(71200400001)(9686003)(6506007)(316002)(55016002)(5660300002)(4326008)(53546011)(83380400001)(86362001)(66574015)(186003); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: =?us-ascii?Q?jMG5q0UB5FHsEaFgo8mIQqpIDx8yW0cqkhbpCS2QukCbC7irdwsZcgoOYVGo?= =?us-ascii?Q?pDoqs9nyQt5tr203Jn5tQ8etKVFZ3DJlLKoQ2XdSKhltsM+73aUPbjNrj5Cw?= =?us-ascii?Q?QHQFwVslsBKQ+VPcrgoLGAWNJ6sbP85Bt/RZxDQMXLx+uFX4kwg9pfEhRo1v?= =?us-ascii?Q?B8MymgaEstMhhiyCJ+bQFKK6kf9xWxrLI3eCda3+yBjkXht3S2mP1MUi1oOP?= =?us-ascii?Q?WjkpwYhXl22xQRmlUY3jHEz70AuzNd2kx72WKv6QFyPxmbTgMAYSAjM1xFSE?= =?us-ascii?Q?yy13tvddKbEU1MPGXHStSJo349IXiEszU3k39Bgm5l3pS3bI7tOzefymqejz?= =?us-ascii?Q?wNsuc/A4bSuda26kVNG4Z5ZIzhE3X2TxOINtXuj8Q36WoMEZRih5wTgdMttK?= =?us-ascii?Q?8u0bHVPWYbvZTyj5T2POCe35hJjEpVzqem7Lg2FU+5dfgMG7Z2vVuCWP4oPQ?= =?us-ascii?Q?X/044DXtbSa6j9WjTto3RNsH9ypcF2IFFTG2G+CS6wd4dWPlYtV83GtpiCfn?= =?us-ascii?Q?UaTae7U91Be0OlRvInxv92v0KTPThxZ0LLfNpS/xI+3JAX+SMhrIJxs1AXOh?= =?us-ascii?Q?FLakgrD0OSxRHa9RHBTTTStmq2PWxQ8B6nezGnlS5E6NsPN3epizzJ679R7H?= =?us-ascii?Q?ugNHJZQR5BlIvyUxjtnX0ZYprgE2CMDIY/XE9G9t2s/7bOYmeaFGNGlRmUf/?= =?us-ascii?Q?N+t6vC8JaoE9+rS2ZooouS2xklcLw2hIt70Wby1DMHQTTI2ZY4JSV8ptiW3x?= =?us-ascii?Q?l0+iAhZJANrkqAX5GL4TrAbDsJKQZj4lW8uaZNWz8q9hBL/6jQsbwZim66u9?= =?us-ascii?Q?886IBc752LeNI7OgtC/fY9t+HhnbaOnkOSv7ojmrD5kHnMSSBD01ZZSAk0qR?= =?us-ascii?Q?XL9vyTSnl7Z/WORB2lKcqEyj7tAaWXCdSOi++P9k+QnDP10PBHUr/A89WYh3?= =?us-ascii?Q?m8wlIgnfBX7qFjTKn7XmPESCjPQ/F5vmxuJ0rTSgubWHGxzeXxIrjeh+Ow2p?= =?us-ascii?Q?VHYipH9PFLCNd7vvdxxkMSF11ghED7qzd57jCUEFx3RnF8e4Leio5KRRQ5kL?= =?us-ascii?Q?UmiVtUI5XKpE4b3qxmK5D8C0E4avB6YmRGcSwFWNhnIeKV2GZdw++tmqAhZO?= =?us-ascii?Q?YoO594t3jk2lsfzZM57do8CuJjnDRXBa0OKa++RS887ESRbewSvlt05U1tRY?= =?us-ascii?Q?9ZxRUFkjTpZuh2XgC9WtbzY/pvPtOpN53371mCv0aLmZdyd8qRe53MKvq1xm?= =?us-ascii?Q?kls8qocvyDSyoTXS+Ma76CkSR7rTNNXPrwf76uTjdTx18dbdTEQPF/71DFGE?= =?us-ascii?Q?N52ii5Hhm6R/UWj4o04EGNZsKjbTb2myKUgroFwLYj+WfmhfhvSfU6zlq8M8?= =?us-ascii?Q?GYtXAshzHT7B7JJsP99rYF70nyzK?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BY5PR11MB4337.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 52b33ef0-13ec-4781-56dc-08d8de857ef0
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Mar 2021 20:47:01.3222 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: RM3nqRgu2AQ7Ei/3y1760rTbFqSgtb+VIgD6y6mIue3406X3hdeFRNMzpKdnmv5dQU9BJgU287d2Lxy8HYYdJA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5166
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.18, xbe-aln-003.cisco.com
X-Outbound-Node: alln-core-7.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/PrGC9bcvkbEOja7-8GpQV7MYhB4>
Subject: Re: [Teas] [Lsr] WG Last Call for draft-ietf-lsr-isis-rfc5316bis
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: Wed, 03 Mar 2021 20:47:08 -0000

Donald -

Thanx for your careful review and your support of the draft.
Replies inline.

> -----Original Message-----
> From: Lsr <lsr-bounces@ietf.org> On Behalf Of Donald Eastlake
> Sent: Wednesday, March 03, 2021 10:32 AM
> To: Christian Hopps <chopps@chopps.org>
> Cc: teas-chairs@ietf.org; teas-ads@ietf.org; teas@ietf.org; lsr-
> chairs@ietf.org; lsr@ietf.org; lsr-ads@ietf.org
> Subject: Re: [Lsr] WG Last Call for draft-ietf-lsr-isis-rfc5316bis
> 
> Hi,
> 
> I have a few comments. Sorry to send these so late in the process. I
> support publication of this draft regardless of whether any action is
> taken on my comments.
> 
> 1. Since there are non-allocation actions, I suggest that the first
> sentence of Section 6 be more like "IANA is requested to take the
> following actions."
> 

[Les:] I understand your point.
However, in this case we are inheriting allocations made by RFC5316 AND adding a new code point for the new IPv6 local ASBR identifier sub-TLV.
Being 100% accurate requires identifying what has been done already vs what is new.
But once the RFC is published the text will change to " IANA has made..." (as it is in RFC 5316) for all the code points (new and old).
Having worked w IANA folks many times, I have great confidence that they will get things right even with the current less than 100% strictly accurate text - so I prefer not to invest time here.
Hope that is OK with you.


> 2. It should be called out as an explicit IANA action to replace all
> References to "[RFC5316]" on the IANA IS-IS TLV Codepoints web page
> with References to "[this document]".
> 

[Les:] OK

> 3. Use of "new" throughout the document for codepoints that were
> assigned for RFC 5316 more than a decade ago should be eliminated.
> 

[Les:] Well, this document replaces RFC 5316. Which means future readers need not ever look at RFC 5316. In which case the distinction between what was "new" in 5316 and what is "new" in 5316bis becomes moot. 
So while I agree that strictly speaking you are correct I am not convinced that doing as you suggest aids clarity. 

> 4. I generally think it is better for implementation requirements to
> be in the main text rather than the IANA Considerations, so I suggest
> moving "Note that all four sub-TLVs SHOULD NOT appear in TLVs 22, 23,
> 25, 222, or 223 and MUST be ignored if they are included in any of
> these TLVs." up to near the end of Section 3.1.

[Les:] I have looked at other documents with similar cases (i.e., a sub-TLV that is permitted in only a subset of the TLVs in the combined registry) and they do not have such a statement at all. The "N" indication in the registry columns is deemed sufficient.
I am therefore inclined to remove the Note altogether.

> 
> 2. I like diagrams and enjoy doing ASCII art, so I suggest replacing
> the prose table at the beginning of 3.1 with the following. In any
> case note that the usual IETF admonition regarding the reserved bits,
> that they MUST be sent as zero and ignored on receipt, seems to be
> missing in the document.
> 
>     0                   1                   2                   3
>     0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
>    +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>    |   Router ID                                     (4 octets)    |
>    +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>    |   default metric                              | (3 octets)
>    +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>    |S|D| Rsvd      |                                 (1 octet)
>    +-+-+-+-+-+-+-+-+
>    |sub-TLVs length|                                 (1 octet)
>    +-+-+-+-+-+-+-+-+-+-+-+-
>    | sub-TLVs ...                                    (0-246 octets)
>    +-+-+-+-+-+-+-+-+-+-+-+-
> 
>      - S, D: Flooding-scope and up/down information discussed below.
>      - Rsvd: 6 reserved bits that MUST be sent as zero and ignored
>              on receipt.
>      - sub-TLVs length: gives the total number of octets of sub-TLVs,
>              which is variable from zero to 246 octets, as an unsigned
>              integer. sub-TLVs are structured as shown below. sub-TLVs
>              with an unknown type MUST be ignored. If the value of the
>              sub-TLVs length field is larger than 246, or the last
>              sub-TLV extends beyond the sub-TLVs length, the TLV is
>              malformed and MUST be ignored.
> 
>    +-+-+-+-+-+-+-+-+
>    | sub-type      |                                 (1 octet)
>    +-+-+-+-+-+-+-+-+
>    | sub-TLV length|                                 (1 octet)
>    +-+-+-+-+-+-+-+-+-+-+-+-
>    | sub-TLV value ...                               (variable)
>    +-+-+-+-+-+-+-+-+-+-+-+-

[Les:] OK
It was not done this way in RFC 5316. When writing bis documents I am biased to NOT changing existing presentation if there is no actual change in functionality to that section.
But I agree diagrams are easier to read and it would be more consistent with other sections of the document.

   Les

> 
> 
> Thanks,
> Donald
> ===============================
>  Donald E. Eastlake 3rd   +1-508-333-2270 (cell)
>  2386 Panoramic Circle, Apopka, FL 32703 USA
>  d3e3e3@gmail.com
> 
> On Wed, Feb 17, 2021 at 10:30 AM Christian Hopps <chopps@chopps.org>
> wrote:
> >
> > Hi LSR and TEAS,
> >
> > This begins a joint WG last call for:
> >
> >   https://datatracker.ietf.org/doc/draft-ietf-lsr-isis-rfc5316bis/
> >
> > Please discuss any issues on the LSR mailing list. The WGLC will end March
> 3, 2021.
> >
> > Authors, please indicate wether you are aware of any IPR related to this
> document to the list.
> >
> > Thanks,
> > Chris, Acee, (Lou and Pavan).
> 
> _______________________________________________
> Lsr mailing list
> Lsr@ietf.org
> https://www.ietf.org/mailman/listinfo/lsr