Re: [core] Genart last call review of draft-ietf-core-sid-15

Carsten Bormann <cabo@tzi.org> Tue, 16 March 2021 17:59 UTC

Return-Path: <cabo@tzi.org>
X-Original-To: core@ietfa.amsl.com
Delivered-To: core@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E77353A08A7; Tue, 16 Mar 2021 10:59:12 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.919
X-Spam-Level:
X-Spam-Status: No, score=-1.919 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 QJIY7vYEhnGQ; Tue, 16 Mar 2021 10:59:10 -0700 (PDT)
Received: from gabriel-vm-2.zfn.uni-bremen.de (gabriel-vm-2.zfn.uni-bremen.de [134.102.50.17]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2EB123A08A6; Tue, 16 Mar 2021 10:59:10 -0700 (PDT)
Received: from [192.168.217.118] (p5089a828.dip0.t-ipconnect.de [80.137.168.40]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by gabriel-vm-2.zfn.uni-bremen.de (Postfix) with ESMTPSA id 4F0LdS5jD5zyhk; Tue, 16 Mar 2021 18:59:08 +0100 (CET)
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.4\))
From: Carsten Bormann <cabo@tzi.org>
In-Reply-To: <161591604910.2184.1526890406711267425@ietfa.amsl.com>
Date: Tue, 16 Mar 2021 18:59:08 +0100
Cc: gen-art@ietf.org, last-call@ietf.org, draft-ietf-core-sid.all@ietf.org, core@ietf.org
X-Mao-Original-Outgoing-Id: 637610348.285911-b3962ab0ade0c8ecd2947fddcf81c725
Content-Transfer-Encoding: quoted-printable
Message-Id: <3FDAB132-A4CA-4E70-9567-125B51AF4CDD@tzi.org>
References: <161591604910.2184.1526890406711267425@ietfa.amsl.com>
To: Linda Dunbar <linda.dunbar@futurewei.com>
X-Mailer: Apple Mail (2.3608.120.23.2.4)
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/bGQHD9db5vyfF-3Aj9Ix0xldZdc>
Subject: Re: [core] Genart last call review of draft-ietf-core-sid-15
X-BeenThere: core@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Constrained RESTful Environments \(CoRE\) Working Group list" <core.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/core>, <mailto:core-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/core/>
List-Post: <mailto:core@ietf.org>
List-Help: <mailto:core-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/core>, <mailto:core-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 16 Mar 2021 17:59:13 -0000

Hi Linda,

> On 2021-03-16, at 18:34, Linda Dunbar via Datatracker <noreply@ietf.org> wrote:
> 
> Reviewer: Linda Dunbar
> Review result: Not Ready
> 
> I am the assigned Gen-ART reviewer for this draft. The General Area
> Review Team (Gen-ART) reviews all IETF documents being processed
> by the IESG for the IETF Chair.  Please treat these comments just
> like any other last call comments.
> 
> For more information, please see the FAQ at
> 
> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
> 
> Document: draft-ietf-core-sid-15
> Reviewer: Linda Dunbar
> Review Date: 2021-03-16
> IETF LC End Date: 2021-03-17
> IESG Telechat date: Not scheduled for a telechat
> 
> Summary:
> This document introduces the globally unique 63-bits integers to identify YANG
> items which include data nodes, RPCs, their associated inputs/outputs,
> notifications, YANG modules, submodules, and features.
> 
> Major issues:
> - One of the benefits of YANG is its explicit naming and human-understandable
> notations. It will be a nightmare if all the YANG items are represented by
> integers. YANG items being represented by integers will be worse than the TYPE
> values in the TLVs. At least, the TLV types are in the context of the protocols
> and their messages. - It will be a tremendous amount of work to map all YANG
> items to globally unique integers. - YANG has been widely deployed without the
> numbering system. Which environment will need the integer representations? Who
> will validate the numbers used? How to validate the YANG modules represented by
> those "integers"?

These are all questions that were valid before this document was written.
The document now shows that this can be done, without any of the disadvantages you cite.

There never will be a need to map *all* YANG identifiers to YANG SIDs, as YANG-CBOR is designed to allow string identifiers as well (you can use YANG-CBOR entirely without using YANG SIDs).  But where encoding and lookup efficiency is important, YANG SIDs are the way to go, and the process for allocating and using them that we arrived at after years of discussion (including several rounds with IANA) makes them very much workable.

> - SID can be mistaken as Segment Identifiers (SID). Suggest using a different
> name.

Indeed, that’s why they are called “YANG SIDs”.

Grüße, Carsten