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

"Aissaoui, Mustapha (Mustapha)" <mustapha.aissaoui@alcatel-lucent.com> Tue, 20 March 2012 13:48 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 61D8921F8691 for <gen-art@ietfa.amsl.com>; Tue, 20 Mar 2012 06:48:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.282
X-Spam-Level:
X-Spam-Status: No, score=-7.282 tagged_above=-999 required=5 tests=[AWL=-0.683, 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 QpTW3GptMhVt for <gen-art@ietfa.amsl.com>; Tue, 20 Mar 2012 06:48:38 -0700 (PDT)
Received: from ihemail4.lucent.com (ihemail4.lucent.com [135.245.0.39]) by ietfa.amsl.com (Postfix) with ESMTP id AE65121F867A for <gen-art@ietf.org>; Tue, 20 Mar 2012 06:48:38 -0700 (PDT)
Received: from usnavsmail1.ndc.alcatel-lucent.com (usnavsmail1.ndc.alcatel-lucent.com [135.3.39.9]) by ihemail4.lucent.com (8.13.8/IER-o) with ESMTP id q2KDmVae009367 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Tue, 20 Mar 2012 08:48:32 -0500 (CDT)
Received: from USNAVSXCHHUB03.ndc.alcatel-lucent.com (usnavsxchhub03.ndc.alcatel-lucent.com [135.3.39.112]) by usnavsmail1.ndc.alcatel-lucent.com (8.14.3/8.14.3/GMO) with ESMTP id q2KDXlqQ023747 (version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=NOT); Tue, 20 Mar 2012 08:48:30 -0500
Received: from USNAVSXCHMBSC2.ndc.alcatel-lucent.com ([135.3.39.147]) by USNAVSXCHHUB03.ndc.alcatel-lucent.com ([135.3.39.112]) with mapi; Tue, 20 Mar 2012 08:48:27 -0500
From: "Aissaoui, Mustapha (Mustapha)" <mustapha.aissaoui@alcatel-lucent.com>
To: "Miguel A. Garcia" <Miguel.A.Garcia@ericsson.com>
Date: Tue, 20 Mar 2012 08:48:24 -0500
Thread-Topic: Gen-ART review of draft-ietf-pwe3-redundancy-bit-06.txt
Thread-Index: Ac0Gnyspk/CE9zvaQe6VyEa9FA1d1wAAK2hw
Message-ID: <5DF53972F7E9134782DCE51624466FE50AD5960180@USNAVSXCHMBSC2.ndc.alcatel-lucent.com>
References: <4F688906.5060207@ericsson.com>
In-Reply-To: <4F688906.5060207@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.9
X-Mailman-Approved-At: Tue, 20 Mar 2012 06:54:10 -0700
Cc: "Muley, Praveen V \(Praveen\)" <praveen.muley@alcatel-lucent.com>, General Area Review Team <gen-art@ietf.org>, "\"Andrew G. Malis\" " <amalis@gmail.com>, Stewart Bryant <stbryant@cisco.com>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-pwe3-redundancy-bit-06.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: Tue, 20 Mar 2012 13:48:39 -0000

Thanks Miguel. I will make these updates along with other ones I may receive as part of the last call.

Regards,
Mustapha. 

-----Original Message-----
From: Miguel A. Garcia [mailto:Miguel.A.Garcia@ericsson.com] 
Sent: Tuesday, March 20, 2012 9:41 AM
To: Muley, Praveen V (Praveen); Aissaoui, Mustapha (Mustapha); "Andrew G. Malis" ; Stewart Bryant
Cc: General Area Review Team
Subject: Gen-ART review of draft-ietf-pwe3-redundancy-bit-06.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-06.txt
Reviewer: Miguel Garcia <miguel.a.garcia@ericsson.com> Review Date: 2012-03-20 IETF LC End Date: 2012-03-21

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

Major issues: none

Minor issues:

- I noticed that many RFC-2119 alike reserved words are written in lower case (thus, they are not normative), where I think they should be normative, in order to achieve interoperability. Allow me to highlight a few examples:

   + Section 5.1, bullet point 2: Both Note 1 and Note 2 include "it is recommended", where I think it should be "it is RECOMMENDED". As a collateral effect of this, if you agree to write uppercase words here, you need to remove the terms "Note 1" and "Note 2", because notes should be informative by nature; they should not contain normative text.

  + Section 5.1, last paragraph on page 10. "should be generated". A bit later on the same paragraph, but now on page 11: "a PE may use ..."

  + Section 5.1, page 11. The paragraph under the bullet point "Active state" contains "the PE nodes must implement". And the paragraph under the bullet point "Standby State" contains "The endpoints of the PW may also allow..."

  + Section 5.2, 4th paragraph on page 12: There are three instances of "must" that should be "MUST".

  + Section 6.2, second paragraph: "the PW MUST also not be used" should include the NOT in uppercase, and probably remove the "also" (remember that "MUST not" does not equal "MUST NOT" or "MUST also NOT".



Nits/editorial comments:

- As an external reviewer, not familiar with the PW technology, I would appreciate if the draft adds a section (perhaps inside Section 3) where all the acronyms are expanded, and perhaps briefly described. Otherwise, it is hard to read the draft, and try to find where is the first occurrence of the acronym, where it was expanded.

- Table 1: In the STANDBY state, the last action is "No action", whereas the rest of the actions in the table, where there is no action, the text is set to "None". So, probably this one should also be set to "None".

- Section 15.5, first paragraph, there is extra spacing and extra dots at the end of this first paragraph.



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