Skip to main content

Last Call Review of draft-ietf-detnet-tsn-vpn-over-mpls-05
review-ietf-detnet-tsn-vpn-over-mpls-05-secdir-lc-nystrom-2021-02-11-00

Request Review of draft-ietf-detnet-tsn-vpn-over-mpls
Requested revision No specific revision (document currently at 07)
Type Last Call Review
Team Security Area Directorate (secdir)
Deadline 2021-02-05
Requested 2021-01-22
Authors Balazs Varga , János Farkas , Andrew G. Malis , Stewart Bryant , Don Fedyk
I-D last updated 2021-02-11
Completed reviews Secdir Last Call review of -05 by Magnus Nyström (diff)
Tsvart Last Call review of -05 by Joerg Ott (diff)
Genart Last Call review of -05 by Vijay K. Gurbani (diff)
Assignment Reviewer Magnus Nyström
State Completed
Request Last Call review on draft-ietf-detnet-tsn-vpn-over-mpls by Security Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/secdir/FPOfz_AUaPgx1dX9Rxc5ohBzONw
Reviewed revision 05 (document currently at 07)
Result Has nits
Completed 2021-02-08
review-ietf-detnet-tsn-vpn-over-mpls-05-secdir-lc-nystrom-2021-02-11-00
 I have reviewed this document as part of the security directorate's
ongoing effort to review all IETF documents being processed by the IESG.
These comments were written primarily for the benefit of the security area
directors.  Document editors and WG chairs should treat these comments just
like any other last call comments.

This document describes a deterministic network data plane when
time-sensitive networks are interconnected.

The security considerations section seems adequate though the integration
of a protocol (TSN) on top of another protocol (DetNet/MPLS) sometimes can
yield new results so it is just a question to the authors if  no new
security consideration results from the application of TSN over DetNet/MPLS?

Editorial:

   - "TSN" is used as an acronym in the abstract as is DetNet (and MPLS,
   although that is perhaps more common knowledge). Would be goot to spell out
   these acronyms directly in the abstract.
   - Section 6, "challanges" -> "challenges"
   - Section 6, "are member" -> "are members"

Thanks,
-- Magnus