Skip to main content

Early Review of draft-ietf-opsawg-ipfix-tcpo-v6eh-05
review-ietf-opsawg-ipfix-tcpo-v6eh-05-opsdir-early-qu-2024-01-04-00

Request Review of draft-ietf-opsawg-ipfix-tcpo-v6eh
Requested revision No specific revision (document currently at 18)
Type Early Review
Team Ops Directorate (opsdir)
Deadline 2024-01-08
Requested 2023-12-18
Requested by Joe Clarke
Authors Mohamed Boucadair , Benoît Claise
I-D last updated 2024-01-04
Completed reviews Genart Last Call review of -11 by Joel M. Halpern (diff)
Secdir Last Call review of -11 by Tero Kivinen (diff)
Tsvart Last Call review of -11 by Wesley Eddy (diff)
Intdir Last Call review of -11 by Dirk Von Hugo (diff)
Secdir Last Call review of -15 by Tero Kivinen (diff)
Tsvart Early review of -05 by Wesley Eddy (diff)
Opsdir Early review of -05 by Yingzhen Qu (diff)
Intdir Early review of -05 by Dirk Von Hugo (diff)
Assignment Reviewer Yingzhen Qu
State Completed
Request Early review on draft-ietf-opsawg-ipfix-tcpo-v6eh by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/1NXDLE-pXgQ4t1PstzrJLuDV0ss
Reviewed revision 05 (document currently at 18)
Result Has nits
Completed 2024-01-04
review-ietf-opsawg-ipfix-tcpo-v6eh-05-opsdir-early-qu-2024-01-04-00
I have reviewed this document as part of the Ops area directorate's ongoing
effort to review all IETF documents being processed by the IESG.  These
comments were written primarily for the benefit of the Ops area directors.
Document editors and WG chairs should treat these comments just like any other
last-call comments.

This document is well written and I believe it is ready for publication.
However I have a few comments on the latest version v-05.


The line numbers are generated using idnits.

93	   This document specifies new IP Flow Information Export (IPFIX)
94	   Information Elements (IEs) to solve a set of issues encountered with

nits: reference to RFC7011 and RFC7012 should be added.

170	   The definition of the ipv6ExtensionHeaders IE is updated in
171	   [I-D.ietf-opsawg-ipfix-fixes] to address some of the issues listed in
172	   Section 1.1.  Because some of these limitations can't be addressed by
173	   simple updates to ipv6ExtensionHeaders, this section specifies a set
174	   of new IEs to address all the ipv6ExtensionHeaders IE limitations.

Question: I'm confused here. Do the new IEs proposed in this document address
all the issues? If so, why are the updates to ipv6ExtensionHeaders needed? Some
clarifications will be helpful.

187	      packet of this Flow contained the respective IPv6 extension

nits: s/contained/contains