Skip to main content

Last Call Review of draft-ietf-mpls-lsp-ping-relay-reply-10

Request Review of draft-ietf-mpls-lsp-ping-relay-reply
Requested revision No specific revision (document currently at 11)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2015-09-25
Requested 2015-09-17
Authors Luo Jian , Lizhong Jin , Thomas Nadeau , George Swallow
I-D last updated 2015-09-18
Completed reviews Genart Last Call review of -04 by Joel M. Halpern (diff)
Genart Last Call review of -10 by Joel M. Halpern (diff)
Secdir Last Call review of -04 by Sean Turner (diff)
Opsdir Last Call review of -04 by Carlos Pignataro (diff)
Opsdir Last Call review of -10 by Carlos Pignataro (diff)
Rtgdir Early review of -10 by Andrew G. Malis (diff)
Assignment Reviewer Joel M. Halpern
State Completed
Request Last Call review on draft-ietf-mpls-lsp-ping-relay-reply by General Area Review Team (Gen-ART) Assigned
Reviewed revision 10 (document currently at 11)
Result Ready w/issues
Completed 2015-09-18
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at


Document: draft-ietf-mpls-lsp-ping-relay-reply-10
    Relayed Echo Reply mechanism for LSP Ping
Reviewer: Joel M. Halpern
Review Date: 17-September-2015
IETF LC End Date: 25-Sept-2015
IESG Telechat date: N/A

Summary: This document is ready for publication as a Proposed Standard

Major issues: none

Minor issues:

    In this document, the address stack grows downward.  While 

reasonable, I think that in the absence of warning of this, readers are 

likely to be confused when they read the detailed procedures.  I think 

the document would be helped by adding the sentence:

    "This stack grows downward, with addresses further along the LSP 

appearing lower down in the stack."

   This could appear right after:
    "-  Stack of Relayed Addresses: a list of relay node addresses."
in section 3.2.

Nits/editorial comments: