Robert Wilton has entered the following ballot position for draft-ietf-alto-oam-yang-15: Yes
When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ for more information about how to handle DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-alto-oam-yang/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Hi, Thank you for this well written document and YANG module. I have a few minor, non-blocking comments, that the authors can address as they wish. Minor level comments: (1) p 8, sec 5.1. Overview of ALTO O&M Data Model module: ietf-alto +--rw alto! Making alto a top level presence container isn't a problem, but given the clients are a list anyway, I would have probably just have made alto-server a presence container instead of the top level container. (2) p 62, sec Appendix A. Examples of Extending the ALTO O&M Data Model The case peeringdb allows the ALTO server to update the server URI to the org object of the organization record in PeeringDB. Perhaps include an informative reference to PeeringDB, or briefly explain what it is. Nit level comments: (3) p 3, sec 1. Introduction The basic structure of this YANG data model is guided by Section 16 of [RFC7285] and [RFC7971]. Although the scope of the YANG data model in this document mainly focuses on the support of the base ALTO protocol [RFC7285] and the existing ALTO standard extensions: [RFC8189], [RFC8895], [RFC8896], [RFC9240], [RFC9241], [RFC9275], and [RFC9439]. I'm not sure the second sentence quite scans, perhaps drop "Although"? Regards, Rob _______________________________________________ alto mailing list [email protected] https://www.ietf.org/mailman/listinfo/alto
