[Gen-art] Gen-ART review of draft-ietf-pwe3-aii-aggregate-02

<Pasi.Eronen@nokia.com> Sun, 18 March 2007 12:35 UTC

Return-path: <gen-art-bounces@ietf.org>
Received: from [127.0.0.1] (helo=stiedprmman1.va.neustar.com) by megatron.ietf.org with esmtp (Exim 4.43) id 1HSubu-0002b3-T1; Sun, 18 Mar 2007 08:35:43 -0400
Received: from [10.91.34.44] (helo=ietf-mx.ietf.org) by megatron.ietf.org with esmtp (Exim 4.43) id 1HSubt-0002at-Fy for gen-art@ietf.org; Sun, 18 Mar 2007 08:35:41 -0400
Received: from smtp.nokia.com ([131.228.20.171] helo=mgw-ext12.nokia.com) by ietf-mx.ietf.org with esmtp (Exim 4.43) id 1HSubs-0003R6-1O for gen-art@ietf.org; Sun, 18 Mar 2007 08:35:41 -0400
Received: from esebh105.NOE.Nokia.com (esebh105.ntc.nokia.com [172.21.138.211]) by mgw-ext12.nokia.com (Switch-3.2.5/Switch-3.2.5) with ESMTP id l2ICZAee027962; Sun, 18 Mar 2007 14:35:32 +0200
Received: from esebh104.NOE.Nokia.com ([172.21.143.34]) by esebh105.NOE.Nokia.com with Microsoft SMTPSVC(6.0.3790.1830); Sun, 18 Mar 2007 14:35:24 +0200
Received: from esebe105.NOE.Nokia.com ([172.21.143.53]) by esebh104.NOE.Nokia.com with Microsoft SMTPSVC(6.0.3790.1830); Sun, 18 Mar 2007 14:35:23 +0200
x-mimeole: Produced By Microsoft Exchange V6.5
Content-class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: quoted-printable
Date: Sun, 18 Mar 2007 14:35:21 +0200
Message-ID: <B356D8F434D20B40A8CEDAEC305A1F2403E566CB@esebe105.NOE.Nokia.com>
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
Thread-Topic: Gen-ART review of draft-ietf-pwe3-aii-aggregate-02
Thread-Index: AcdpWeTnZZQPnTpgTCiuHvWtzeKBiw==
From: Pasi.Eronen@nokia.com
To: gen-art@ietf.org, townsley@cisco.com, lmartini@cisco.com, chmetz@cisco.com, balus@nortel.com, sugimoto@nortel.com, stbryant@cisco.com, danny@arbor.net
X-OriginalArrivalTime: 18 Mar 2007 12:35:23.0862 (UTC) FILETIME=[E6357F60:01C76959]
X-Nokia-AV: Clean
X-Spam-Score: 0.2 (/)
X-Scan-Signature: 69a74e02bbee44ab4f8eafdbcedd94a1
Cc:
Subject: [Gen-art] Gen-ART review of draft-ietf-pwe3-aii-aggregate-02
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www1.ietf.org/pipermail/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
Errors-To: gen-art-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-pwe3-aii-aggregate-02
Reviewer: Pasi Eronen
Review Date: 2007-03-17
IETF LC End Date: 2007-03-30
IESG Telechat date: (not known yet)

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

Comments:

1) As the PROTO write-up already pointed out, the current version
has a normative downward reference to draft-ietf-l3vpn-bgpvpn-auto.
To me it seems this might be changed to informative reference,
but probably someone more familiar with the protocol(s) should
also confirm this.

2) In Section 3.2 the document uses the phrases like "the AC ID is
NULL" to mean that that "AC ID" field is not included in the message
at all. I found this a bit confusing; probably something like "the AC
ID field is omitted" would be clearer?

3) Section 3.2, "If the global ID is derived from a 2-octet AS 
number, then the high-order 4 octets of this 4 octet field MUST 
be set to zero." Should this be "...the high-order 2 octets..."?

4) The document uses RFC 2119 keywords, but does not include
RFC 2119 in the references list.

Best regards,
Pasi

_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www1.ietf.org/mailman/listinfo/gen-art