Skip to main content

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