Re: [Gen-art] Gen-ART review of draft-ietf-pwe3-redundancy-bit-07.txt

"Aissaoui, Mustapha (Mustapha)" <mustapha.aissaoui@alcatel-lucent.com> Wed, 27 June 2012 20:22 UTC

Return-Path: <mustapha.aissaoui@alcatel-lucent.com>
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 273B221F86A8 for <gen-art@ietfa.amsl.com>; Wed, 27 Jun 2012 13:22:05 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id lCY1nSqFfVCr for <gen-art@ietfa.amsl.com>; Wed, 27 Jun 2012 13:22:04 -0700 (PDT)
Received: from ihemail4.lucent.com (ihemail4.lucent.com [135.245.0.39]) by ietfa.amsl.com (Postfix) with ESMTP id EB89121F8665 for <gen-art@ietf.org>; Wed, 27 Jun 2012 13:22:03 -0700 (PDT)
Received: from usnavsmail3.ndc.alcatel-lucent.com (usnavsmail3.ndc.alcatel-lucent.com [135.3.39.11]) by ihemail4.lucent.com (8.13.8/IER-o) with ESMTP id q5RKLutW000436 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Wed, 27 Jun 2012 15:21:56 -0500 (CDT)
Received: from USNAVSXCHHUB01.ndc.alcatel-lucent.com (usnavsxchhub01.ndc.alcatel-lucent.com [135.3.39.110]) by usnavsmail3.ndc.alcatel-lucent.com (8.14.3/8.14.3/GMO) with ESMTP id q5RKLuxB020745 (version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=NOT); Wed, 27 Jun 2012 15:21:56 -0500
Received: from USNAVSXCHMBSC2.ndc.alcatel-lucent.com ([135.3.39.145]) by USNAVSXCHHUB01.ndc.alcatel-lucent.com ([135.3.39.110]) with mapi; Wed, 27 Jun 2012 15:21:56 -0500
From: "Aissaoui, Mustapha (Mustapha)" <mustapha.aissaoui@alcatel-lucent.com>
To: "Miguel A. Garcia" <Miguel.A.Garcia@ericsson.com>
Date: Wed, 27 Jun 2012 15:21:55 -0500
Thread-Topic: Gen-ART review of draft-ietf-pwe3-redundancy-bit-07.txt
Thread-Index: Ac04tNVtXAkzBuQLRKy+X09n0lIVbwby0yMA
Message-ID: <5DF53972F7E9134782DCE51624466FE50BD74A84F3@USNAVSXCHMBSC2.ndc.alcatel-lucent.com>
References: <4FBC9029.1000305@ericsson.com>
In-Reply-To: <4FBC9029.1000305@ericsson.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Scanned-By: MIMEDefang 2.57 on 135.245.2.39
X-Scanned-By: MIMEDefang 2.64 on 135.3.39.11
Cc: "draft-ietf-pwe3-redundancy-bit.authors@tools.ietf.org" <draft-ietf-pwe3-redundancy-bit.authors@tools.ietf.org>, General Area Review Team <gen-art@ietf.org>, Andy Malis <amalis@gmail.com>, "pwe3-chairs@tools.ietf.org" <pwe3-chairs@tools.ietf.org>, Stewart Bryant <stbryant@cisco.com>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-pwe3-redundancy-bit-07.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
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: <http://www.ietf.org/mail-archive/web/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, 27 Jun 2012 20:22:05 -0000

Hi Miguel,
Thank you again for the second review of this draft. All your comments are accepted and will be included in the next version.

Regards,
Mustapha. 

-----Original Message-----
From: Miguel A. Garcia [mailto:Miguel.A.Garcia@ericsson.com] 
Sent: Wednesday, May 23, 2012 3:22 AM
To: draft-ietf-pwe3-redundancy-bit.authors@tools.ietf.org; Andy Malis; pwe3-chairs@tools.ietf.org; Stewart Bryant
Cc: General Area Review Team
Subject: Gen-ART review of draft-ietf-pwe3-redundancy-bit-07.txt

I have been selected as the General Area Review Team (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>

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

Document: draft-ietf-pwe3-redundancy-bit-07.txt
Reviewer: Miguel Garcia <Miguel.A.Garcia@ericsson.com> Review Date: 23-May-2012 IESG Telechat date: 24-May-2012

Summary: The document is almost ready for publication as a standards track RFC, but has some minor issues that should be fixed.

I reviewed version -06 of this document. At that time I had a few comments. Most of them have been successfully addressed. In this review I am addressing the leftovers and new issues.

Major issues: none

Minor issues:

- In my previous review, I highlighted that many lowercase 2119-reserved words (like "must", "may", etc.) should actually be written in uppercase to be really normative. Rather than analyzing each case on a one by one basis, the authors have systematically written in uppercase all 2119-reserved words. The problem is that now some uppercase 2119-reserved words don't make sense, and should be reverted to lowercase. Allow me some examples:

   * Sections 1 (Introduction) and 2 (Motivation and Scope). Since these are descriptive sections in nature, normative text should not be written here. If there is a need to write normative text, it should be written later in any of the procedures sections.

   * Section 5.2, 1st paragraph, the "MUST" does not really have a normative intention as it is currently written:

    One endpoint node of the redundant set of PWs is designated the
    Master and is responsible for selecting which PW both endpoints MUST
    use to forward user traffic.

   * Section 15.2. I understand that this section describes scenarios or examples of architectures. Therefore, it does not make sense to write normative statements in here. If you need one, it should be written in any of the procedures sections. There is a "SHOULD" on the 4th paragraph of Section 15.2. The same applies to the last paragraph in Section 15.4 and last paragraph in Section 15.6.


Nits/editorial comments:

- The abstract has added a new sentence at the end. Unfortunately there is a new reference "in RFC 5542 [9]". Abstracts shouldn't include references. Just write "... in RFC 5542".

- There is an extra dot at the end of the second paragraph in Section 6.1. The same happen in the last paragraph of Section 15.3. And yet the same in the third paragraph of Section 15.4, and the second paragraph in 15.6.

- Section 15.4, second paragraph, the term "Figure" is repeated.

- Section 15.5, the last paragraph starts with some strange indentation.

/Miguel
--
Miguel A. Garcia
+34-91-339-3608
Ericsson Spain