Last Call Review of draft-ietf-opsawg-service-assurance-yang-09
review-ietf-opsawg-service-assurance-yang-09-opsdir-lc-wu-2022-11-21-00
Request | Review of | draft-ietf-opsawg-service-assurance-yang |
---|---|---|
Requested revision | No specific revision (document currently at 11) | |
Type | Last Call Review | |
Team | Ops Directorate (opsdir) | |
Deadline | 2022-11-22 | |
Requested | 2022-11-08 | |
Authors | Benoît Claise , Jean Quilbeuf , Paolo Lucente , Dr. Paolo Fasano , Thangam Arumugam | |
I-D last updated | 2022-11-21 | |
Completed reviews |
Yangdoctors Early review of -07
by Michal Vaško
(diff)
Opsdir Last Call review of -09 by Bo Wu (diff) Genart Last Call review of -09 by Dan Romascanu (diff) Intdir Telechat review of -11 by Tommy Pauly Opsdir Telechat review of -10 by Bo Wu (diff) |
|
Assignment | Reviewer | Bo Wu |
State | Completed | |
Request | Last Call review on draft-ietf-opsawg-service-assurance-yang by Ops Directorate Assigned | |
Posted at | https://mailarchive.ietf.org/arch/msg/ops-dir/JddbDLjiSeGGYn6VkBfOQHEZf5M | |
Reviewed revision | 09 (document currently at 11) | |
Result | Has nits | |
Completed | 2022-11-21 |
review-ietf-opsawg-service-assurance-yang-09-opsdir-lc-wu-2022-11-21-00
Reviewer: Bo Wu Review result: Has Issues I am the assigned Ops reviewer for this draft. Please treat these comments just like any other last call comments. Document: draft-ietf-opsawg-service-assurance-yang-09 Summary: This document (with intended status Standards Track) defines three YANG modules, a basic service and two common subservices of device and interface, for representing service assurance graphs. This is a well written document with specific decomposed examples of a tunnel service. Major issues: None Minor issues: None Nits/editorial comments: 1.It would be better to add some text to explain why the agent list is defined. Introduction says the module should be supported by an agent. 2. In section 3.2 and 3.4 The "subservice" list contains all the subservice instances currently configured on the server. "server" is not defined in the document. Is it referring "SAIN agent"? 3. Appendix C Example of YANG instances Paragraph 1 s/examples/example since there is only one example. Thanks, Bo