Re: [Gen-art] Gen-ART Review of draft-ietf-forces-interfelfb-04

Jari Arkko <jari.arkko@piuha.net> Wed, 01 June 2016 07:24 UTC

Return-Path: <jari.arkko@piuha.net>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B6FB612D0D8; Wed, 1 Jun 2016 00:24:00 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.326
X-Spam-Level:
X-Spam-Status: No, score=-3.326 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-1.426] autolearn=ham autolearn_force=no
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 lv48Ye3y8LcI; Wed, 1 Jun 2016 00:23:58 -0700 (PDT)
Received: from p130.piuha.net (p130.piuha.net [IPv6:2a00:1d50:2::130]) by ietfa.amsl.com (Postfix) with ESMTP id C3263128E19; Wed, 1 Jun 2016 00:23:57 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by p130.piuha.net (Postfix) with ESMTP id D0BA42CEF0; Wed, 1 Jun 2016 10:23:56 +0300 (EEST) (envelope-from jari.arkko@piuha.net)
X-Virus-Scanned: amavisd-new at piuha.net
Received: from p130.piuha.net ([127.0.0.1]) by localhost (p130.piuha.net [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id uEG3hl0uNnxV; Wed, 1 Jun 2016 10:23:56 +0300 (EEST)
Received: from [127.0.0.1] (p130.piuha.net [IPv6:2a00:1d50:2::130]) by p130.piuha.net (Postfix) with ESMTP id 374802CC64; Wed, 1 Jun 2016 10:23:56 +0300 (EEST) (envelope-from jari.arkko@piuha.net)
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\))
Content-Type: multipart/signed; boundary="Apple-Mail=_FE9659BC-F8DD-43CA-B100-62847A02B1FE"; protocol="application/pgp-signature"; micalg="pgp-sha512"
X-Pgp-Agent: GPGMail 2.5.2
From: Jari Arkko <jari.arkko@piuha.net>
In-Reply-To: <B94EDFE2-F234-417B-B1C6-5A611D52B96E@vigilsec.com>
Date: Wed, 01 Jun 2016 10:23:47 +0300
Message-Id: <450A63DC-DD3B-4EAB-BD9C-0272E60A3FA4@piuha.net>
References: <B94EDFE2-F234-417B-B1C6-5A611D52B96E@vigilsec.com>
To: Russ Housley <housley@vigilsec.com>
X-Mailer: Apple Mail (2.1878.6)
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/a03LtwWCi6WNLrWDjcKKa0afOzM>
Cc: draft-ietf-forces-interfelfb.all@ietf.org, IETF Gen-ART <gen-art@ietf.org>
Subject: Re: [Gen-art] Gen-ART Review of draft-ietf-forces-interfelfb-04
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
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: <https://mailarchive.ietf.org/arch/browse/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: Wed, 01 Jun 2016 07:24:01 -0000

Thanks for your review, Russ. Again!

Authors, any comments?

Jari

On 31 May 2016, at 21:54, Russ Housley <housley@vigilsec.com> wrote:

> 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-forces-interfelfb-04
> Reviewer: Russ Housley
> Review Date: 2016-05-31
> IETF LC End Date: 2016-06-22
> IESG Telechat date: unknown
> 
> Summary:  Almost Ready
> 
> 
> Major Concerns:
> 
> In Section 5.2, the document says:
> 
>   o  The Ethernet type is used to identify the frame as inter-FE LFB
>      type.  Ethertype 0xFEFE is to be used (XXX: Note to editor, likely
>      we wont get that value - update when available).
> 
> There is a process for getting an ethertype assigned.  It is handled by
> the IESG.  Please see: http://www.ietf.org/iesg/statement/ethertypes.html
> 
> 
> Minor Concerns:
> 
> In Section 1.2: s/document reiterates the terminology defined/
>                 /document depends on the terminology defined/
> 
> In Section 2: s/The current ForCES model/The ForCES model/
> 
> 
> Nits:
> 
> Many places: s/IPV4/IPv4/
> 
> In Section 2: s/architecture calls out for/architecture calls for/
> 
> In Section 3.1, why are the paragraphs in bullets?  They see to work
> just fine as normal paragraphs.
> 
> In Section 5.2: s/header illustrated in Figure 6)/
>                 /header (illustrated in Figure 6)
> 
>