Telechat Review of draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08
review-ietf-pals-mpls-tp-pw-over-bidir-lsp-08-genart-telechat-even-2016-06-30-00

Request Review of draft-ietf-pals-mpls-tp-pw-over-bidir-lsp
Requested revision No specific revision (document currently at 09)
Type Telechat Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2016-07-05
Requested 2016-06-20
Authors Mach Chen , Wei Cao , Attila Takacs , Ping Pan
I-D last updated 2016-06-30
Completed reviews Genart Telechat review of -08 by Roni Even (diff)
Secdir Last Call review of -08 by Christian Huitema (diff)
Opsdir Last Call review of -06 by Sarah Banks (diff)
Rtgdir Early review of -06 by Daniele Ceccarelli (diff)
Assignment Reviewer Roni Even
State Completed
Request Telechat review on draft-ietf-pals-mpls-tp-pw-over-bidir-lsp by General Area Review Team (Gen-ART) Assigned
Reviewed revision 08 (document currently at 09)
Result Ready w/issues
Completed 2016-06-30
review-ietf-pals-mpls-tp-pw-over-bidir-lsp-08-genart-telechat-even-2016-06-30-00
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART,
please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Please resolve these comments along with any other Last Call comments you may
receive.

Document:   draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08



Reviewer: Roni Even

Review Date:2016–6-30

IETF LC End Date: 2016–7-4

IESG Telechat date: 2016-7-7



Summary: This draft is ready for publication as standard track  RFC.





Major issues:



Minor issues:

1.



In section 2.1 “

C (Co-routed path) bit: This informs the remote T-PE/S-PEs about the properties
of the underlying LSPs.  When set, the remote T-PE/S-PEs need to select
co-routed LSP (as the forwarding tunnel) as the reverse PSN tunnel.  If there
is no such tunnel available, it may trigger the remote T-PE/S-PEs to establish
a new LSP.” Why are you using non normative language here “need to” instead of
MUST or SHOULD while for the S bit normative language is used?





Nits/editorial comments:

In the abstract “TE” is not expanded, only later in the document