[Gen-art] Gen-ART review of draft-ietf-bess-mvpn-extranet-04

Christer Holmberg <christer.holmberg@ericsson.com> Tue, 01 December 2015 12:26 UTC

Return-Path: <christer.holmberg@ericsson.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 7A47D1B2BB5 for <gen-art@ietfa.amsl.com>; Tue, 1 Dec 2015 04:26:53 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] 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 yOos3am5_2xz for <gen-art@ietfa.amsl.com>; Tue, 1 Dec 2015 04:26:51 -0800 (PST)
Received: from sesbmg23.ericsson.net (sesbmg23.ericsson.net [193.180.251.37]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B7E3E1B2BB4 for <gen-art@ietf.org>; Tue, 1 Dec 2015 04:26:50 -0800 (PST)
X-AuditID: c1b4fb25-f79876d0000011ee-fc-565d92082ebe
Received: from ESESSHC008.ericsson.se (Unknown_Domain [153.88.183.42]) by sesbmg23.ericsson.net (Symantec Mail Security) with SMTP id 0B.3F.04590.8029D565; Tue, 1 Dec 2015 13:26:48 +0100 (CET)
Received: from ESESSMB209.ericsson.se ([169.254.9.142]) by ESESSHC008.ericsson.se ([153.88.183.42]) with mapi id 14.03.0248.002; Tue, 1 Dec 2015 13:26:48 +0100
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART review of draft-ietf-bess-mvpn-extranet-04
Thread-Index: AdEjv6Z/GO6WVYNwTyC0uPKp1ol0Rg==
Date: Tue, 1 Dec 2015 12:26:47 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B37C762A9@ESESSMB209.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.18]
Content-Type: multipart/alternative; boundary="_000_7594FB04B1934943A5C02806D1A2204B37C762A9ESESSMB209erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupgkeLIzCtJLcpLzFFi42KZGbFdS5djUmyYwbPDmhZXP/9js7j66jOL A5PHkiU/mTy+XP7MFsAUxWWTkpqTWZZapG+XwJXxc/YP1oLXQRX3D51iamBs9Oxi5OSQEDCR aHvZzAZhi0lcuLceyObiEBI4zCix9vxiRghnMaPElDX9LF2MHBxsAhYS3f+0QRpEBDQl5q54 ywRiMwvkSjw+384MYgsLWEtc3PeSFaLGQeLY3kMsELaexKc3jYwgNouAisTrRR1gNq+Ar8TU 3X/BehmBjvh+ag3UTHGJW0/mM0EcJyCxZM95ZghbVOLl43+sELaixMdX+xgh6vMlmrsamCBm CkqcnPmEZQKj8Cwko2YhKZuFpAwiriOxYPcnNghbW2LZwtfMMPaZA4+ZkMUXMLKvYhQtTi1O yk03MtZLLcpMLi7Oz9PLSy3ZxAiMn4NbfqvuYLz8xvEQowAHoxIP74crMWFCrIllxZW5hxgl OJiVRHilq2PDhHhTEiurUovy44tKc1KLDzFKc7AoifM2Mz0IFRJITyxJzU5NLUgtgskycXBK NTDySt9m2VKYeLhhwwmhmEn3btyY1FJ1uerH59PeDgseJbbq97SlTUreYbZZuECsmGN+/fvL ev+u6Ues+f/CvXnGXMl/5q2uU9X3XNc4bL49Kobts2D1I66zbeb+Tcu+F2myPWtJ3HeIcyZz uanwZvsdE0N2iZbYh036Znx3yWOnQJ/KLXfyNVKUWIozEg21mIuKEwFu4oIimwIAAA==
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/ltscvDMleXd2WVEz3lWONFVYkxw>
Cc: "'draft-ietf-bess-mvpn-extranet.all@tools.ietf.org'" <draft-ietf-bess-mvpn-extranet.all@tools.ietf.org>
Subject: [Gen-art] Gen-ART review of draft-ietf-bess-mvpn-extranet-04
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: <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: Tue, 01 Dec 2015 12:26:53 -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>
Document:                                                         draft-ietf-bess-mvpn-extranet-04.txt
Reviewer:                                                           Christer Holmberg
Review Date:                                                     1 December 2015
IETF LC End Date:                                             4 December 2015
IETF Telechat Date:                                         17 December 2015
Summary:           The document is well written, and almost ready for publication. However, I have some general editorial comments that I'd like the authors to address.
Major Issues: None
Minor Issues: None
Editorial Issues:

General:

QG_1:

Throughout the document, there are places where the text say "We follow", "We say", etc. I suggest to talk about the "document" instead of "We".

I also wonder whether all the "We says are needed.


QG_2:

The Abstract says that the document updates RFCs 6513, 6514, and 6625. However, there is no dedicated section(s) which defines the updates.

I also think the Introduction should contain some general overview text on what is updated.

If possible, it would also be good to have dedicated "Updates to RFC XXXX" chapters, so that people can easily find what exactly has been updated.


QG_3:

The Abstract and Introduction say "Previous RFCs". I suggest to list the relevant RFCs instead.

Something like:

                "RFC 6513 and RFC6514 specify the procedures..."