Re: [Gen-art] review of draft-ietf-forces-ceha-08.txt

Jari Arkko <jari.arkko@piuha.net> Tue, 03 December 2013 06:04 UTC

Return-Path: <jari.arkko@piuha.net>
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 1DCA01AE055 for <gen-art@ietfa.amsl.com>; Mon, 2 Dec 2013 22:04:21 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-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 R8z1nAsFb-qf for <gen-art@ietfa.amsl.com>; Mon, 2 Dec 2013 22:04:19 -0800 (PST)
Received: from p130.piuha.net (p130.piuha.net [193.234.218.130]) by ietfa.amsl.com (Postfix) with ESMTP id A1EC61AE04C for <gen-art@ietf.org>; Mon, 2 Dec 2013 22:04:18 -0800 (PST)
Received: from localhost (localhost [127.0.0.1]) by p130.piuha.net (Postfix) with ESMTP id 8F8662CC61; Tue, 3 Dec 2013 08:04:15 +0200 (EET)
X-Virus-Scanned: amavisd-new at piuha.net
Received: from p130.piuha.net ([127.0.0.1]) by localhost (p130.piuha.net [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ScPwO5P46nf7; Tue, 3 Dec 2013 08:04:13 +0200 (EET)
Received: from [127.0.0.1] (p130.piuha.net [IPv6:2a00:1d50:2::130]) by p130.piuha.net (Postfix) with ESMTP id 6C5BB2CC48; Tue, 3 Dec 2013 08:04:13 +0200 (EET)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 6.6 \(1510\))
From: Jari Arkko <jari.arkko@piuha.net>
In-Reply-To: <005201cee4b9$fec5c660$fc515320$@upatras.gr>
Date: Tue, 03 Dec 2013 08:04:13 +0200
Content-Transfer-Encoding: quoted-printable
Message-Id: <FB20B7F4-9458-48DD-9F69-C65F5B9A14AB@piuha.net>
References: <201311071759.rA7Hxv3t067420@givry.fdupont.fr> <005201cee4b9$fec5c660$fc515320$@upatras.gr>
To: Haleplidis Evangelos <ehalep@ece.upatras.gr>
X-Mailer: Apple Mail (2.1510)
Cc: gen-art@ietf.org, draft-ietf-forces-ceha.all@tools.ietf.org
Subject: Re: [Gen-art] review of draft-ietf-forces-ceha-08.txt
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: <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, 03 Dec 2013 06:04:21 -0000

Thank you Francis for your careful review (as always) and Evangelos for taking the comments into account in the new draft.

I have placed a no-obj position for this document in this week's IESG telechat.

Jari

On Nov 19, 2013, at 1:57 AM, Haleplidis Evangelos <ehalep@ece.upatras.gr> wrote:

> Greetings Francis,
> 
> Thank you for your comments. They are currently all being addressed and will
> soon have the revised draft.
> 
> Regarding the ordering of the definitions, someone should indeed have read
> at least the ForCES protocol RFC prior to arriving to this one. But your
> comment make sense, therefore I reversed the order having the definitions of
> the FE/CE first.
> 
> Regards,
> Evangelos Haleplidis.
> 
>> -----Original Message-----
>> From: Francis.Dupont@fdupont.fr [mailto:Francis.Dupont@fdupont.fr]
>> Sent: Thursday, November 07, 2013 8:00 PM
>> To: gen-art@ietf.org
>> Cc: draft-ietf-forces-ceha.all@tools.ietf.org
>> Subject: review of draft-ietf-forces-ceha-08.txt
>> 
>> 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>.
>> 
>> Please resolve these comments along with any other Last Call comments
>> you may receive.
>> 
>> Document: draft-ietf-forces-ceha-08.txt
>> Reviewer: Francis Dupont
>> Review Date: 20131028
>> IETF LC End Date: 20131106
>> IESG Telechat date: unknown
>> 
>> Summary: Ready
>> 
>> Major issues: None
>> 
>> Minor issues: None
>> 
>> Nits/editorial comments:
>> - 1 pages 2 and 3: I have a concern with the order of definitions.
>>  IMHO there are 3 solutions:
>>   * keep the document order arguing definitions are repeated for
>>    convenience so it doesn't matter there are backward references
>>    (i.e., someone new in the domain should first read referenced RFCs,
>>     and at the opposite someone not new in the domain already knows
>>     the used acronyms)
>> 
>>   * introduce each acronym at its first use
>> 
>>   * same + reorder the definition list to minimize out-of-order
>>    internal references
>> 
>>  Note the best choice depends on the intended public so you have a
>> better
>>  idea than me about this...
>> 
>> - 2.2 second 1. page 5: IMHO the interface is Fp, not Fr.
>> 
>> - 3.1 figure 2 page 7: Estbalishment -> Establishment
>> 
>> - 3.1.1 page 7: parametrization -> parameterization
>> 
>> - 4.1 page 11 (twice): i.e. -> i.e.,
>> 
>> - 4.1 2. page 11: the and in "+ and 2" should be moved to the end
>>   of the previous item, i.e., I suggest to change:
>> 
>>           +  1 (HA Mode - Cold Standby) represents that the FE is in
>> HA
>>              mode cold Standby
>> 
>>           +  and 2 (HA Mode - Hot Standby) represents that the FE is
>> in
>>              HA mode hot Standby
>> 
>> into
>> 
>>           +  1 (HA Mode - Cold Standby) represents that the FE is in
>> HA
>>              mode cold Standby, and
>> 
>>           +  2 (HA Mode - Hot Standby) represents that the FE is in
>>              HA mode hot Standby
>> 
>>    Note if you want to put something at the end of each items the
>> correct
>>    character is ";", and "." for the last item.
>> 
>> - 4.2 page 13: practise -> practice
>> 
>> - 4.2 pages 13 and 14: figure 4 should be on one page (this is
>>  something to leave to the RFC Editor anyway).
>> 
>> - 4.2 figure 5 page 14 (3!): Estbalishment -> Establishment
>> 
>> - Appendix A page 20: some indent problems with "The FE should
>>  stop | continue" (same remark: we can expect the RFC Editor will
>>  use a XML pretty-printer for the final editing).
>> 
>> Regards
>> 
>> Francis.Dupont@fdupont.fr
>> 
>> PS: I am at the IETF meeting.
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art