Last Call Review of draft-ietf-bess-nsh-bgp-control-plane-13
review-ietf-bess-nsh-bgp-control-plane-13-tsvart-lc-bonaventure-2019-12-17-00

Request Review of draft-ietf-bess-nsh-bgp-control-plane
Requested rev. no specific revision (document currently at 18)
Type Last Call Review
Team Transport Area Review Team (tsvart)
Deadline 2019-12-13
Requested 2019-11-29
Authors Adrian Farrel, John Drake, Eric Rosen, Jim Uttaro, Luay Jalil
Draft last updated 2019-12-17
Completed reviews Rtgdir Last Call review of -13 by Ravi Singh (diff)
Tsvart Last Call review of -13 by Olivier Bonaventure (diff)
Opsdir Last Call review of -13 by Sheng Jiang (diff)
Secdir Last Call review of -13 by Scott Kelly (diff)
Genart Last Call review of -12 by Brian Carpenter (diff)
Assignment Reviewer Olivier Bonaventure 
State Completed
Review review-ietf-bess-nsh-bgp-control-plane-13-tsvart-lc-bonaventure-2019-12-17
Posted at https://mailarchive.ietf.org/arch/msg/tsv-art/M_sUBRzilWqmxNrsULaxSFqZCUo
Reviewed rev. 13 (document currently at 18)
Review result Ready
Review completed: 2019-12-17

Review
review-ietf-bess-nsh-bgp-control-plane-13-tsvart-lc-bonaventure-2019-12-17

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 have found the document to be clear and readable. I could not find any transport specific issue that inside the document that would need to be discuss. From a transport viewpoint, the classical MTU concerns are valid, but they mainly concern RFC8300 and not specifically this document. 

Concerning ECMP, a point of attention when deploying stateful functions in a network where ECMP is used would be to ensure that transport flows would be mapped to the same instance of a stateful function. This is probably already discussed in other documents related to service function chaining.