Hi, I do not have time to do another full review of this draft. These are high-level comments.
## document scope is too large The draft should be split into 2 or 3 drafts. 1) SID File Definition 2a) SID Management Procedures 2b) Initial SID File Definitions ## document scope is not CORE-specific Several WGs and SDOs want to use SID files. The 13 modules listed in sec 6.4.4 are a small fraction of the IETF YANG modules that should have SID files. A separate document can better address the IETF-wide requirements. ## SID files are normative - The algorithm is not stable enough yet (both specification and implementation). - Manual editing is allowed. - There is no indication in the SID file that it is automatically generated or manually edited. - Only the authoritative SID file can be trusted to have the correct assignments ## Initial SID files are missing The normative SID files for all YANG modules listed in sec 6.4.4 are missing. The assignment range is insufficient for interoperability. Andy On Mon, Sep 4, 2023 at 1:22 PM Marco Tiloca <marco.tiloca= 40ri...@dmarc.ietf.org> wrote: > Dear all, > > This email starts a Working Group Last Call for the document: > > https://datatracker.ietf.org/doc/html/draft-ietf-core-sid-21 > > (YANG Schema Item iDentifier (YANG SID)) > > > The document status can be found at: > > https://datatracker.ietf.org/doc/draft-ietf-core-sid/ > > > Please provide your comments and feedback by Friday, 2023-09-15. > > This WGLC is also copied to the NETMOD WG mailing list. > > Best, > /Marco > > -- > Marco Tiloca > Ph.D., Senior Researcher > > Phone: +46 (0)70 60 46 501 > > RISE Research Institutes of Sweden AB > Box 1263 > 164 29 Kista (Sweden) > > Division: Digital Systems > Department: Computer Science > Unit: Cybersecurity > https://www.ri.se > > _______________________________________________ > netmod mailing list > netmod@ietf.org > https://www.ietf.org/mailman/listinfo/netmod >
_______________________________________________ netmod mailing list netmod@ietf.org https://www.ietf.org/mailman/listinfo/netmod
