[Gen-art] Genart telechat review of draft-ietf-bier-mpls-encapsulation-09

Peter Yee <peter@akayla.com> Mon, 16 October 2017 21:22 UTC

Return-Path: <peter@akayla.com>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 66CB8132F2C; Mon, 16 Oct 2017 14:22:06 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Peter Yee <peter@akayla.com>
To: gen-art@ietf.org
Cc: draft-ietf-bier-mpls-encapsulation.all@ietf.org, bier@ietf.org, ietf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.63.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150818892635.438.2743077513006159051@ietfa.amsl.com>
Date: Mon, 16 Oct 2017 14:22:06 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/cAfGmv5klvZL40_XpfiMFObpvhg>
Subject: [Gen-art] Genart telechat review of draft-ietf-bier-mpls-encapsulation-09
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
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: Mon, 16 Oct 2017 21:22:06 -0000

Reviewer: Peter Yee
Review result: Ready with Issues

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

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-bier-mpls-encapsulation-09
Reviewer: Peter Yee
Review Date: 2017-10-16
IETF LC End Date: 2017-10-16
IESG Telechat date: 2017-10-26

Summary:  This well-written draft is an Experimental specification for how
encapsulation is performed in the BIER architecture in order to optimally
forward a packet through a multicast domain while not being burdensome to
intermediate routers.

This draft is ready with issues.

Major issues: None.

Minor issues: This Experimental draft specifies that entries to its BIER Next
Protocol Identifiers registry are to be done according to the "Standards
Action" policy of RFC 8126.  It's not entirely clear to me whether RFC 8126's
requirement that such entries be from Standards Track or BCP documents applies
to the initial set of values.  If it does, then there's an issue here between
RFC 8126 and this draft's desired status.

Nits/editorial comments:

General:

Replace "ethernet" with "Ethernet" throughout.

Replace "ethertype" with "Ethertype" or "EtherType" throughout.  (IEEE usage is
varies between the two, unfortunately.)

Specific:

Page 3, second bullet item: change "is" to "are".

Page 4, last paragraph: this paragraph is redundant to one it follows fairly
closely in Section 1.  Consider striking it, although it's not harmful.

Page 5, Section 2.1.1.1, 1st paragraph, 3rd sentence: change "ISIS" to "IS-IS".

Page 9, OAM definition, 1st paragraph, second sentence: delete a duplicated
"the" in the sentence.

Page 10, Section 2.1.3, 1st paragraph, 2nd sentence: delete a duplicated "in"
from the sentence.