[Gen-art] Gen-ART Review of draft-ietf-rtgwg-mofrr-06

Russ Housley <housley@vigilsec.com> Fri, 17 April 2015 16:25 UTC

Return-Path: <housley@vigilsec.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DBC6A1A1B9E; Fri, 17 Apr 2015 09:25:51 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -101.9
X-Spam-Level:
X-Spam-Status: No, score=-101.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, USER_IN_WHITELIST=-100] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id TzvHBiomy1EN; Fri, 17 Apr 2015 09:25:50 -0700 (PDT)
Received: from odin.smetech.net (x-bolt-wan.smeinc.net [209.135.219.146]) by ietfa.amsl.com (Postfix) with ESMTP id 2A48B1A1EB7; Fri, 17 Apr 2015 09:25:41 -0700 (PDT)
Received: from localhost (unknown [209.135.209.5]) by odin.smetech.net (Postfix) with ESMTP id 4E218F240EF; Fri, 17 Apr 2015 12:25:30 -0400 (EDT)
X-Virus-Scanned: amavisd-new at smetech.net
Received: from odin.smetech.net ([209.135.209.4]) by localhost (ronin.smeinc.net [209.135.209.5]) (amavisd-new, port 10024) with ESMTP id oMke6f2zcLnP; Fri, 17 Apr 2015 12:25:29 -0400 (EDT)
Received: from [192.168.2.100] (pool-96-255-133-185.washdc.fios.verizon.net [96.255.133.185]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by odin.smetech.net (Postfix) with ESMTP id 94D1BF240E9; Fri, 17 Apr 2015 12:25:29 -0400 (EDT)
From: Russ Housley <housley@vigilsec.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Date: Fri, 17 Apr 2015 12:25:18 -0400
Message-Id: <94FB12C5-C701-4763-AFC0-3996E3C1F8E8@vigilsec.com>
To: draft-ietf-rtgwg-mofrr.all@ietf.org
Mime-Version: 1.0 (Apple Message framework v1085)
X-Mailer: Apple Mail (2.1085)
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/wG04Z1-ZpriuSAFhco7WpO4_j-s>
Cc: IETF Gen-ART <gen-art@ietf.org>, IETF <ietf@ietf.org>
Subject: [Gen-art] Gen-ART Review of draft-ietf-rtgwg-mofrr-06
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 17 Apr 2015 16:25:52 -0000

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>.

This review is in response to a request for early Gen-ART review.

Document: draft-ietf-rtgwg-mofrr-06
Reviewer: Russ Housley
Review Date: 2015-04-17
IETF LC End Date: 2015-04-30
IESG Telechat date: unknown

Summary: Almost Ready


Major Concerns:

None


Minor Concerns:

Please add POP to the list of terms in Section 1.2.

The introduction to Section 6 really only talks about Section 6.1.
Perhaps Sections 6 and 6.1 should be merged.  If they are merged, it
could look like this: 

   6.  MoFRR Applicability to Dual-Plane Topology

   MoFRR applicability is topology dependent.  The applicability is the
   same as LFA FRR which is discussed in [RFC6571].

   MoFRR works best in dual-planes topologies as illustrated in the
   figures below.  MoFRR may be enabled on any router in the network.
   In the figures below, MoFRR is shown enabled on the Provider Edge
   (PE) routers to illustrate one way in which the technology may be
   deployed.

   . . . 

If this suggestion is taken, Sections 6.2, 6.3, and 6.4 could become
top-level sections of their own.


Other Comments:

The figures are referenced in many ways: FIG3, Fig3, and Fig 3.
Please pick one style and use it throughout the document.

Section 1, 2nd paragraph: add a period at the end of the paragraph.

Section 1.2, LFA definition: add a period after the first sentence.

Section 4, intro paragraph: Delete "  See below;" from the end.

Section 5, 1st paragraph: s/section explore some/section explores some/