Review of draft-ietf-teas-gmpls-resource-sharing-proc-07

Dale Worley <worley@ariadne.com> Mon, 23 January 2017 21:09 UTC

Return-Path: <worley@ariadne.com>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 4DFF91298A5; Mon, 23 Jan 2017 13:09:17 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Dale Worley <worley@ariadne.com>
To: <gen-art@ietf.org>
Subject: Review of draft-ietf-teas-gmpls-resource-sharing-proc-07
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.4
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148520575728.29440.1145810018959016669.idtracker@ietfa.amsl.com>
Date: Mon, 23 Jan 2017 13:09:17 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/J15cW8jb_9L5nu9bdYaW3nwN-xE>
Cc: draft-ietf-teas-gmpls-resource-sharing-proc.all@ietf.org, ietf@ietf.org, teas@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 23 Jan 2017 21:09:17 -0000

Reviewer: Dale Worley
Review result: Ready with Nits

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 wait for direction from your
document shepherd or AD before posting a new version of the draft.

For more information, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document:  draft-ietf-teas-gmpls-resource-sharing-proc-07
Reviewer:  Dale R. Worley
Review Date:  23 Jan 2017
IETF LC End Date:  17 Jan 2017
IESG Telechat date:  2 Feb 2017

Summary:

       This draft is basically ready for publication, but has nits
       that should be fixed before publication.

There remain a few editorial items:

2. Conventions Used in This Document	
		
    The reader is assumed to be familiar with the terminology in	
    [RFC3209], [RFC3473], [RFC4872], [RFC4873] and [RFC4427].	
		
That is a significant help to the reader, but it's also rather
intimidating!  Is there a way to point out that 4427 is specific to
recovery?

3.1.1.  1+R Restoration

   Unlike a protecting LSP which is set up before the failure, a
   restoration LSP is set up per need basis, after the failure.

Probably better to change "per need basis" to "when needed".

3.2. Resource Sharing By Restoration LSP	
			
"By" generally should not be capitalized in titles, as it is a
preposition.

                               +-----+      +-----+
                               |  F  +------+  G  +--------+
                               +--+--+      +-----+        |
                                  |                        |
                                  |                        |
        +-----+    +-----+     +--+--+      +-----+     +--+--+
        |  A  +----+  B  +-----+  C  +--X---+  D  +-----+  E  |
        +-----+    +-----+     +-----+      +-----+     +-----+

          Figure 3: Resource Sharing in 1+R Recovery Scheme


   [...]  Nodes A and B
   reconfigure the resources to set up the restoration LSP by sending
   cross-connection command to the data plane.

   In the recovery scheme employing revertive behavior, after the
   failure is repaired, the resources on nodes A and B need to be
   reconfigured to set up the working LSP.  The traffic is then
reverted
   back to the original working LSP.  

It's not clear to me why nodes A and B are reconfigured and/or do the
reconfiguring.  Any "global" reconfiguring would be driven by the
head-end A alone, I think.  Any "local" reconfiguring would be done
by
C and possibly E.  Though perhaps there is reconfiguring that must be
done along the entire path, but that would be attributed to A, B, C,
F, G, and E together.  I suspect there is a trivial editorial error
here...

[END]