Skip to main content

Last Call Review of draft-ietf-detnet-mpls-over-ip-preof-08
review-ietf-detnet-mpls-over-ip-preof-08-tsvart-lc-bonaventure-2023-12-22-00

Request Review of draft-ietf-detnet-mpls-over-ip-preof
Requested revision No specific revision (document currently at 11)
Type Last Call Review
Team Transport Area Review Team (tsvart)
Deadline 2023-12-22
Requested 2023-12-08
Authors Balazs Varga , János Farkas , Andrew G. Malis
I-D last updated 2023-12-22
Completed reviews Intdir Telechat review of -09 by Tatuya Jinmei (diff)
Tsvart Last Call review of -08 by Olivier Bonaventure (diff)
Genart Last Call review of -08 by Joel M. Halpern (diff)
Secdir Last Call review of -08 by Catherine Meadows (diff)
Opsdir Last Call review of -08 by Carlos Pignataro (diff)
Rtgdir Early review of -02 by Bruno Decraene (diff)
Assignment Reviewer Olivier Bonaventure
State Completed
Request Last Call review on draft-ietf-detnet-mpls-over-ip-preof by Transport Area Review Team Assigned
Posted at https://mailarchive.ietf.org/arch/msg/tsv-art/fyBm4hz1dZgMYrxbMMohGk-SiWk
Reviewed revision 08 (document currently at 11)
Result Ready w/issues
Completed 2023-12-22
review-ietf-detnet-mpls-over-ip-preof-08-tsvart-lc-bonaventure-2023-12-22-00
This document has been reviewed as part of the transport area review team's
ongoing effort to review key IETF documents. These comments were written
primarily for the transport area directors, but are copied to the document's
authors and WG to allow them to address any issues raised and also to the IETF
discussion list for information.

When done at the time of IETF Last Call, the authors should consider this
review as part of the last-call comments they receive. Please always CC
tsv-art@ietf.org if you reply to or forward this review.

I reviewed the document from the transport viewpoint without being an expert in
DetNet. As Bruno Decraene mentions in his review, the document seems to be a
small modification to RFC9025 that already defines the encapsulation. Is it
intended to informational while RFC9025 is standards track.

In Section 5, my main concern is the handling of the FlowLabel field in the
IPv6 header when several Detnet flows are aggregated together. Will these flows
use packets with the same IPv6 flowLabel or different flowLabels (one per
Detnet flow) ? This would have an impact on ECMP hash and thus influence the
packet that different Detnet flows follow. If several Detnet flows are
aggregated in a single UDP tunnel, do they all need to follow the same path in
the network or not ?

The handling of this FlowLabel must be clarified in a revision of this document.

Details

In Section 1
  However, the DetNet IP
   data plane described in [RFC8939] does not specify how sequencing
   information can be encoded in the IP header.

the end of the sentence (IP header) is misleading. The reader could think that
you will change the IP header with an extension, which is clearly not the case.

In 4.4, at the end of

   In the first case, the different DetNet PWs use the same UDP tunnel,
   so they are treated as a single (aggregated) flow at the forwarding
   sub-layer.  At the service sub-layer, each flow uses a different
   Service ID.

I would suggest to provide a reference to Figure 3 that describes this
encapsulation.