Skip to main content

Early Review of draft-ietf-spring-srv6-srh-compression-17
review-ietf-spring-srv6-srh-compression-17-secdir-early-smith-2024-07-03-00

Request Review of draft-ietf-spring-srv6-srh-compression
Requested revision No specific revision (document currently at 23)
Type Early Review
Team Security Area Directorate (secdir)
Deadline 2024-07-12
Requested 2024-03-21
Requested by Joel M. Halpern
Authors Weiqiang Cheng , Clarence Filsfils , Zhenbin Li , Bruno Decraene , Francois Clad
I-D last updated 2024-07-03
Completed reviews Intdir Early review of -17 by Benson Muite (diff)
Rtgdir Early review of -16 by Nicolai Leymann (diff)
Opsdir Early review of -16 by Gyan Mishra (diff)
Secdir Early review of -17 by Ned Smith (diff)
Genart Last Call review of -19 by Stewart Bryant (diff)
Secdir Last Call review of -19 by Ned Smith (diff)
Comments
This draft is in working group last call, with a small number of issues still being discussed.  The content is alrgely stable, so review at this stage would be helpful.
Assignment Reviewer Ned Smith
State Completed
Request Early review on draft-ietf-spring-srv6-srh-compression by Security Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/secdir/8dR12_kv-PGF5MxWOVUubOvw5k8
Reviewed revision 17 (document currently at 23)
Result Ready
Completed 2024-07-03
review-ietf-spring-srv6-srh-compression-17-secdir-early-smith-2024-07-03-00
The I-D is ready but the authors may consider making the following changes:
Section 4, 5th paragraph: It would be helpful to describe which operational
issues / challenges are made easier and why to give guidance on when it may/may
not be appropriate to take the recommendation.

Section 5.3, second paragraph: S/Therefore/Therefore,/
third paragraph: S/is able to/can/ and S/Therefore/Therefore,/
fourth paragraph: S/GIB,/GIB;/

Section 6.2, first paragraph: S/in order to/to/

Section 6.3, second paragraph: S/instead/instead,/

Section 9.2, seventh paragraph: S/is able to/can/

Section 9.3, third paragraph: S/in particular is/is/

Section 10, third paragraph: S/Draft, and/Draft and/

Section 10.13, first paragraph: S/open source/open-source/

Section 10.13, second paragraph: S/open source/open-source/

I did not attempt to verify the content of the appendix A was correct / sound.
I assume the reference implementations are there to achieve this goal despite
the stated goal of the reference implementations was for interoperability.
Possibly, the authors could comment on how the I-D describes a correct and
scalable, as well as interoperable solution.