Telechat Review of draft-ietf-detnet-ip-over-tsn-05
review-ietf-detnet-ip-over-tsn-05-intdir-telechat-chown-2021-02-14-00

Request Review of draft-ietf-detnet-ip-over-tsn
Requested revision No specific revision (document currently at 07)
Type Telechat Review
Team Internet Area Directorate (intdir)
Deadline 2021-02-14
Requested 2021-02-08
Requested by Éric Vyncke
Authors Balazs Varga , János Farkas , Andrew G. Malis , Stewart Bryant
I-D last updated 2021-02-14
Completed reviews Rtgdir Last Call review of -05 by Russ White (diff)
Opsdir Last Call review of -05 by Dan Romascanu (diff)
Genart Last Call review of -05 by Paul Kyzivat (diff)
Intdir Telechat review of -05 by Tim Chown (diff)
Iotdir Telechat review of -06 by Niklas Widell (diff)
Comments 
Sorry for the short-term request (but this document has been placed very recently on the IESG telechat) but it is about 10 pages long.
It is part of a pair of Time Sensitive Networking documents, a common reviewer would probably be more efficient:

1) draft-ietf-detnet-ip-over-tsn

2) draft-ietf-detnet-mpls-over-tsn

Thank you

-éric
Assignment Reviewer Tim Chown
State Completed
Request Telechat review on draft-ietf-detnet-ip-over-tsn by Internet Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/int-dir/fKUYTizlwdO8oX1QDoSytKVEcHQ
Reviewed revision 05 (document currently at 07)
Result Ready w/nits
Completed 2021-02-14
review-ietf-detnet-ip-over-tsn-05-intdir-telechat-chown-2021-02-14-00
Hi,

Re: INT-DIR review of draft-ietf-detnet-ip-over-tsn-05

This draft details how a DetNet IP data plane can run over a TSN sub-network.

I have reviewed this document and believe it is Ready for publication, with
Nits.  Note I am not overly familiar with specifics of the DetNet architecture.
There are some hopefully simple questions that might be addressed and some
minor editorial nits as detailed below.

Nits:

p.2
Expand TSN on first use, rather than on page 4.

p.3
Is “6-tuple” well-defined?  Perhaps expand for clarity.

p.4
Is there a specific reason to show a multi-homed TSN node?
Are there implications if it is multi homed in TSN but not IP?

p.5
Not being familiar with DetNet in detail it might be useful to expand on the
rationale for mapping to multicast given the statement that the flow is
“directed”.

p.6
covers required -> covers the required
I note it says there are no requirements stated for TSN-unaware nodes in the
document, but is it worth adding something about requirements to ensure the TSN
Relay can be reached?

p.7
Implementations must -> implementation must (twice)
FRER function -> The FRER function

p.8
of the document -> of this document
challanges -> challenges
Are member -=> are members

p.9
In some case -> in some cases

--
Tim