Last Call Review of draft-ietf-opsawg-tsvwg-udp-ipfix-10
review-ietf-opsawg-tsvwg-udp-ipfix-10-opsdir-lc-korhonen-2024-05-21-00

Request Review of draft-ietf-opsawg-tsvwg-udp-ipfix-07
Requested revision 07 (document currently at 14)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2024-04-26
Requested 2024-04-08
Requested by Mahesh Jethanandani
Authors Mohamed Boucadair , Tirumaleswar Reddy.K
I-D last updated 2024-05-21
Completed reviews Genart Last Call review of -08 by Robert Sparks (diff)
Secdir Last Call review of -08 by Watson Ladd (diff)
Tsvart Last Call review of -07 by Tommy Pauly (diff)
Intdir Last Call review of -08 by Dr. Joseph D. Touch (diff)
Opsdir Last Call review of -10 by Jouni Korhonen (diff)
Tsvart Early review of -03 by Tommy Pauly (diff)
Intdir Early review of -03 by Dr. Joseph D. Touch (diff)
Assignment Reviewer Jouni Korhonen
State Completed
Request Last Call review on draft-ietf-opsawg-tsvwg-udp-ipfix by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/2uoHQcZtFtbX9BXhdHckpSFvd1Y
Reviewed revision 10 (document currently at 14)
Result Has nits
Completed 2024-05-21
review-ietf-opsawg-tsvwg-udp-ipfix-10-opsdir-lc-korhonen-2024-05-21-00
I am the assigned OPS-DIR reviewer for this draft. Apologies for a late review.

Summary: The document is ready with nits.

I found the document solid and ready. One minor comment below:

In section 4.1:
      options.  For example, if only option Kinds <= 32 are observed,
      then the value of the udpSafeOptions IE can be encoded as
      unsigned32, or if only option Kinds <= 63 are observed, then the

Shouldn't it say:
      options.  For example, if only option Kinds <= 31 are observed,