Gen-ART LC review of draft-ietf-pim-rpf-vector-07

Ben Campbell <ben@estacado.net> Tue, 13 January 2009 21:19 UTC

Return-Path: <ietf-bounces@ietf.org>
X-Original-To: ietf-archive@megatron.ietf.org
Delivered-To: ietfarch-ietf-archive@core3.amsl.com
Received: from [127.0.0.1] (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 743B828C145; Tue, 13 Jan 2009 13:19:03 -0800 (PST)
X-Original-To: ietf@core3.amsl.com
Delivered-To: ietf@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id E401828C144; Tue, 13 Jan 2009 13:19:01 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qjkGr+QCIuki; Tue, 13 Jan 2009 13:19:01 -0800 (PST)
Received: from estacado.net (estacado-pt.tunnel.tserv2.fmt.ipv6.he.net [IPv6:2001:470:1f03:266::2]) by core3.amsl.com (Postfix) with ESMTP id 9A4C93A6B32; Tue, 13 Jan 2009 13:19:00 -0800 (PST)
Received: from dn3-213.estacado.net (dn3-213.estacado.net [172.16.3.213]) (authenticated bits=0) by estacado.net (8.14.2/8.14.2) with ESMTP id n0DLIbxS003382 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Tue, 13 Jan 2009 15:18:37 -0600 (CST) (envelope-from ben@estacado.net)
Message-Id: <6716E9B6-0633-4FC0-8CE1-C85BAF128ECD@estacado.net>
From: Ben Campbell <ben@estacado.net>
To: erosen@cisco.com, aboers@cisco.com, ice@cisco.com, General Area Review Team <gen-art@ietf.org>
Mime-Version: 1.0 (Apple Message framework v930.3)
Subject: Gen-ART LC review of draft-ietf-pim-rpf-vector-07
Date: Tue, 13 Jan 2009 15:18:37 -0600
X-Mailer: Apple Mail (2.930.3)
Cc: mmcbride@cisco.com, venaas@uninett.no, ietf@ietf.org, dward@cisco.com
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
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>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="us-ascii"; Format="flowed"; DelSp="yes"
Sender: ietf-bounces@ietf.org
Errors-To: ietf-bounces@ietf.org

I have been selected as the General Area Review Team (Gen-ART)
reviewer for this draft (for background on Gen-ART, please see
http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html).

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-pim-rpf-vector-07
Reviewer: Ben Campbell
Review Date: 2009-01-13
IETF LC End Date: 2009-01-19
IESG Telechat date: (if known)

Summary: This draft is very close to ready for publication as a  
proposed standard. There is a minor issue that should be addressed  
prior to publication, and a couple of editorial nits.

Note 1: The Gen-ART assignment was for version 06, but version 07 has  
been published. This review is for 07.

Note 2: I previously reviewed 06 for publication as an informational  
RFC. This review is incremental to that one. I consider any comment  
from that review not mentioned here to be resolved.



Major issues: None



Minor issues:

The following issue from my previous review has not been addressed.  
Email with an author indicated that they intended to fix this, but it  
doesn't appear to have made it into the draft.

> -- IDNITS reports that there is no RFC 2119 reference or  
> boilerplate, but there is at least one use of normative language  
> (2.3.4).

Furthermore, Section 2 is liberally sprinkled with occurrences of  
lower-case "must", "should" and "may" that should be examined to  
determine if they warrant normative language. I did not consider this  
an issue for an informational RFC, but do consider it to so for a  
proposed standard.



Nits/editorial comments:

-- IANA Considerations, last sentence (new comment)

s/propose/proposed

The following editorial comments from my previous review has not been  
addressed. Email with an author indicated that they intended to fix  
most of these, but the fixes don't appear to have made it into the  
draft.

> -- There are a number of acronyms that should be expanded on first  
> use. I would not worry about expanding acronyms that are well known  
> to the entire IETF community (e.g. TCP), but acronyms that are not  
> widely known outside the BGP community probably should be.
>
> -- Section 2, first sentence:
>
> Who is the "we" in this context? A edge router? (This is not a  
> complaint about 2nd person language in general so much as a concern  
> about the actor being obscured.)  The pattern of saying "we" or  
> "our" referring to a network element taking some particular action  
> occurs a few more times in the document. It would be better to  
> simply name the element.
>
> -- Section 2.3.4, first paragraph:
>
> s/depending/dependent



_______________________________________________
Ietf mailing list
Ietf@ietf.org
https://www.ietf.org/mailman/listinfo/ietf