Last Call Review of draft-ietf-sfc-ioam-nsh-11
review-ietf-sfc-ioam-nsh-11-opsdir-lc-hares-2023-04-19-00
| Request | Review of | draft-ietf-sfc-ioam-nsh |
|---|---|---|
| Requested revision | No specific revision (document currently at 13) | |
| Type | IETF Last Call Review | |
| Team | Ops Directorate (opsdir) | |
| Deadline | 2023-04-18 | |
| Requested | 2023-04-04 | |
| Authors | Frank Brockners , Shwetha Bhandari | |
| I-D last updated | 2023-08-23 (Latest revision 2023-05-05) | |
| Completed reviews |
Genart IETF Last Call review of -11
by Christer Holmberg
(diff)
Tsvart IETF Last Call review of -11 by Mirja Kühlewind (diff) Opsdir IETF Last Call review of -11 by Susan Hares (diff) Rtgdir IETF Last Call review of -10 by Mach Chen (diff) |
|
| Assignment | Reviewer | Susan Hares |
| State | Completed | |
| Request | IETF Last Call review on draft-ietf-sfc-ioam-nsh by Ops Directorate Assigned | |
| Posted at | https://mailarchive.ietf.org/arch/msg/ops-dir/4K1oB6_hMJ0HU4Ky8f_EYGFl69E | |
| Reviewed revision | 11 (document currently at 13) | |
| Result | Ready | |
| Completed | 2023-04-19 |
review-ietf-sfc-ioam-nsh-11-opsdir-lc-hares-2023-04-19-00
document: draft-ietf-sfc-ioam-nsh-11.txt reviewer: Susan Hares reviewer expertise: Routing, general awareness of SFC and NSH, not an IOAM expert. review focus: OPS-DIR - operational AFIL, the document is consistent within the text and with documents referenced. I have read https://datatracker.ietf.org/doc/html/draft-ietf-ippm-ioam-deployment-05, but I am not an expert in the deployment considerations. I recommend that someone review this text who is a expert in deployment considerations. Only two sentences had editorial issues that the authors might want to review: Section 3. sentence 2. The phrase "the different IOAM-Data-Fields" made the logic difficult to follow. However, it may be the best phrase for the technology. Appendix A, paragraph 4 (starts Total length), the second sentence has a grammatical roughness. Here's the roughness Current text: If for example [text], then text. normal mechanisms use: For example, if -----, then ---- I have not flagged these issues as a nit, but the authors might want to review check these before getting IESG reviews.