Skip to main content

Last Call Review of draft-ietf-opsawg-vpn-common-09
review-ietf-opsawg-vpn-common-09-rtgdir-lc-pritchard-2021-08-24-00

Request Review of draft-ietf-opsawg-vpn-common
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team Routing Area Directorate (rtgdir)
Deadline 2021-08-06
Requested 2021-07-16
Requested by Alvaro Retana
Authors samier barguil , Oscar Gonzalez de Dios , Mohamed Boucadair , Qin Wu
Draft last updated 2021-08-24
Completed reviews Yangdoctors Early review of -02 by Radek Krejčí (diff)
Rtgdir Last Call review of -06 by Ron Bonica (diff)
Tsvart Last Call review of -06 by Wesley Eddy (diff)
Yangdoctors Last Call review of -06 by Radek Krejčí (diff)
Rtgdir Last Call review of -09 by Victoria Pritchard (diff)
Opsdir Last Call review of -09 by Tim Wicinski (diff)
Intdir Last Call review of -09 by Suresh Krishnan (diff)
Tsvart Last Call review of -09 by Wesley Eddy (diff)
Genart Last Call review of -09 by Joel M. Halpern (diff)
Assignment Reviewer Victoria Pritchard
State Completed
Review review-ietf-opsawg-vpn-common-09-rtgdir-lc-pritchard-2021-08-24
Posted at https://mailarchive.ietf.org/arch/msg/rtg-dir/CobX0WeHV5U5N6kIuZuYDjbDjg4
Reviewed revision 09 (document currently at 12)
Result Has Nits
Completed 2021-08-24
review-ietf-opsawg-vpn-common-09-rtgdir-lc-pritchard-2021-08-24-00
Hello,

I have been selected as the Routing Directorate reviewer for this draft. The
Routing Directorate seeks to review all routing or routing-related drafts as
they pass through IETF last call and IESG review, and sometimes on special
request. The purpose of the review is to provide assistance to the Routing ADs.
For more information about the Routing Directorate, please see
​http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Document: draft-ietf-opsawg-vpn-common-09
Reviewer: Victoria Pritchard
Review Date: 24th August 2021
Intended Status: Standards Track

Summary:
This document is basically ready for publication but has nits that should be
considered prior to publication.

Comments:
This is a concise document, indicating clearly where it fits, what it defines,
and how it should be used. Seems a good idea to have the common things in a
re-usable module.

Major Issues:
No major issues found.

Minor issues:
No minor issues found.

Nits:
In Section 3, I think it might be easier to read if each grouping was shown
individually with the accompanying explanation next to the diagram. In Section
4, I noticed some typos: In the inbound-bw feature, defines -> defined
"identity igmpv3" description states IGMPv2 adminisitrative -> administrative
(seen in a few places) "list forwarding-profile-identifier": Forwrding ->
Forwarding