Re: [mpls] working group last call for draft-ietf-mpls-lsp-ping-reply-mode-simple-01

[Qin Wu <[e-mail]>]

Hi, Authors of draft-ietf-mpls-lsp-ping-reply-mode-simple-01:
I have read draft-ietf-mpls-lsp-ping-reply-mode-simple-01 and think it is ready to go for publication.
Here are few editorial comments on this draft.
1.section 3.2 said:
"
4.  If a responder LSR understands the Reply Mode Order TLV and the
       TLV is valid, then the responder LSR MUST consider Reply Mode
       values described in the TLV and MUST NOT use the value described
       in the Reply Mode field of received MPLS echo request.
"
I think the value carried in the Reply mode Order TLV overrides the value included in the Reply Mode field of MPLS echo request message header, would it be good to make this more clear and explicit.

2. section 3.2 also said:
"
5.  If a responder LSR understands the Reply Mode Order TLV but the
       TLV is not valid (due to conditions  listed below), then the
       responder LSR MUST only use the value described in the Reply Mode
       field of received MPLS echo request.
"
Not sure where the conditions are specified, suggest to provide a link to the conditions specified in this document

3.section 4.1 said:
"
[RFC7110] has defined that the "Reply Path TLV" can include Sub-TLVs
   describing multiple FECs, from which the responder LSR can chose  the
   FEC to send the MPLS echo reply message on .  
"
s/chose/choose
send the MPLS echo reply message on?
What does "on" mean here? Remove "on" from this sentence?

4. section 4.1.2 said:
"
   Then the MPLS echo request message is to carry:

   o  The Reply Mode Order TLV carrying Reply Modes  {5, 2, 5}
"
Reply mode 5 is not defined in RFC4379.Where reply mode 5 is defined?

5. section 4.2 said:
"
The mechanism defined in this document will work with Proxy LSP Ping
   defined by [I-D.ietf-mpls-proxy-lsp-ping].  MPLS proxy ping request
   can carry a Reply Mode value and  the Reply Mode Order TLV with list
   of Reply Mode values.
"

Suggest To distinct reply mode value carried In the message header and reply mode value carried in the Reply mode order TLV here

6. section 4.2 said:
"
With these procedures, Reply Mode used by the MPLS echo
   reply sender is propagated in the Reply Mode field to the sender of
   MPLS proxy ping request.
"
I am totally confused by this last sentence.
what does " sender" in"the MPLS echo reply" means? Are you saying reply mode value in the echo rely is propagated to proxy ping request? Or propagated to proxy ping reply?
How the sender of MPLS proxy ping request is related to initiator LSR or responder LSR or responding node used in the document?
It will be nice to have a consistence terminologies use.

-Qin
-------- Forwarded Message --------
Subject: 	working group last call for
draft-ietf-mpls-lsp-ping-reply-mode-simple-01
Date: 	Fri, 20 Mar 2015 14:03:52 +0000
From: 	Ross Callon <[e-mail]>
To: 	[e-mail] <[e-mail]>
CC: 	[e-mail] <[e-mail]>, George
Swallow (swallow) <[e-mail]>, Carlos Pignataro (cpignata) <[e-mail]>, Loa Andersson <[e-mail]>, Mach Chen <[e-mail]>, [e-mail] <[e-mail]>



Working Group,
This is to initiate a working group last call on draft-ietf-mpls-lsp-ping-reply-mode-simple-01.
Because this WGLC will span the IETF in Dallas, it will be extended to three weeks.
Please send your comments to the mpls wg mailing list ([e-mail]_ <mailto:[e-mail]>).
There are no IPR disclosures against this document. All the authors have stated that they are not aware of any IPR that relates to this draft.
This working group last call ends Friday  April 10, 2015.
Ross
for the MPLS WG chairs