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

From: Lars Eggert <[email-masked]>
Date: Mon, 12 Jul 2021 17:05:38 +0300

Linda, thank you for your review. I have entered a No Objection ballot for this document.

Lars

> On 2021-3-16, at 19:34, Linda Dunbar via Datatracker <[email-masked]> 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"?
> 
> - SID can be mistaken as Segment Identifiers (SID). Suggest using a different
> name.
> 
> Minor issues:
> 
> Nits/editorial comments:
> 
> Best Regards,
> Linda Dunbar
> 
> 
> --
> last-call mailing list
> [email-masked]
> https://www.ietf.org/mailman/listinfo/last-call