Last Call Review of draft-ietf-ccamp-mw-topo-yang-10
review-ietf-ccamp-mw-topo-yang-10-genart-lc-hares-2024-02-02-00
Request | Review of | draft-ietf-ccamp-mw-topo-yang |
---|---|---|
Requested revision | No specific revision (document currently at 12) | |
Type | Last Call Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2024-02-05 | |
Requested | 2024-01-22 | |
Authors | Scott Mansfield , Jonas Ahlberg , Min Ye , Xi Li , Daniela Spreafico | |
I-D last updated | 2024-02-02 | |
Completed reviews |
Yangdoctors Last Call review of -07
by Michal Vaško
(diff)
Rtgdir Last Call review of -07 by Stig Venaas (diff) Genart Last Call review of -10 by Susan Hares (diff) Secdir Last Call review of -10 by Joseph A. Salowey (diff) |
|
Assignment | Reviewer | Susan Hares |
State | Completed | |
Request | Last Call review on draft-ietf-ccamp-mw-topo-yang by General Area Review Team (Gen-ART) Assigned | |
Posted at | https://mailarchive.ietf.org/arch/msg/gen-art/d-O38Vc6jdSYTfwN1FULw68j2yI | |
Reviewed revision | 10 (document currently at 12) | |
Result | Ready w/nits | |
Completed | 2024-02-02 |
review-ietf-ccamp-mw-topo-yang-10-genart-lc-hares-2024-02-02-00
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://wiki.ietf.org/en/group/gen/GenArtFAQ>. Document: draft-ietf-ccamp-mw-topo-yang-?? Reviewer: Susan Hares Review Date: 2024-02-02 IETF LC End Date: 2024-02-05 IESG Telechat date: Not scheduled for a telechat Summary: Excellent work that reuses earlier work (TE + network model (I2RS), and CCAMP documents. It has the clear flow I have come to expect from the authors (Scott, Jonas, Min Ye, Xi Li, and Daniela) Thank you for your excellent work. Major issues: none Minor issues: none Nits/editorial comments: 6 nits #1 - Editorial nit, Appendix A, format issues Appendix A has format issues in pdf in the following variables +rw tet:name? string +ro mwt:mw-bandwidth? unit64 +rw mwt:num-bonded-carriers unit32 +rw mwt:numb-protecting-carriers unit32 You might want to check to see if it fits the switch to the next line. [yang diagrams are tough in line wrap] #2 - Appendix A.1 example - editorial nit, line termination { } + [ ] in seem off, but I cannot judge this by eye. Did you generate this example from a real-world example? If so, you can ignore it. Suggestion: Some people put comments at the end of lines at key points. It makes the example easier to read. [You do not have to adopt suggestions. It is just my experience that it helps the person reading the example). #3 - Same Suggestion of adding line comments for readability for A.2 #4 - Appendix B Please look at the line wrap on the following lines: +rw tet:te-tip-id? +rw te:name string #5 - figures 1 and 2 are very small in pdf, but OK in the txt. #6 - Examples in B.1 Same comments on adding comments if long strings of } ... } ... } ] ...} A short comment on what it ends, really helps.