Re: [Gen-art] Genart review of draft-ietf-hip-reload-instance-08

Ari Keränen <ari.keranen@ericsson.com> Thu, 26 September 2013 09:44 UTC

Return-Path: <ari.keranen@ericsson.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 A684E11E8187 for <gen-art@ietfa.amsl.com>; Thu, 26 Sep 2013 02:44:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.949
X-Spam-Level:
X-Spam-Status: No, score=-5.949 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HELO_EQ_SE=0.35, MIME_8BIT_HEADER=0.3, 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 kMoSUksCCXUo for <gen-art@ietfa.amsl.com>; Thu, 26 Sep 2013 02:43:56 -0700 (PDT)
Received: from mailgw1.ericsson.se (mailgw1.ericsson.se [193.180.251.45]) by ietfa.amsl.com (Postfix) with ESMTP id CB9CB11E8180 for <gen-art@ietf.org>; Thu, 26 Sep 2013 02:43:55 -0700 (PDT)
X-AuditID: c1b4fb2d-b7f738e000003ee3-f5-524401da9476
Received: from ESESSHC019.ericsson.se (Unknown_Domain [153.88.253.125]) by mailgw1.ericsson.se (Symantec Mail Security) with SMTP id 56.98.16099.AD104425; Thu, 26 Sep 2013 11:43:54 +0200 (CEST)
Received: from mail.lmf.ericsson.se (153.88.183.149) by smtp.internal.ericsson.com (153.88.183.77) with Microsoft SMTP Server id 14.2.328.9; Thu, 26 Sep 2013 11:43:53 +0200
Received: from nomadiclab.lmf.ericsson.se (nomadiclab.lmf.ericsson.se [131.160.33.3]) by mail.lmf.ericsson.se (Postfix) with ESMTP id E9E04110455; Thu, 26 Sep 2013 12:43:52 +0300 (EEST)
Received: from nomadiclab.lmf.ericsson.se (localhost [127.0.0.1]) by nomadiclab.lmf.ericsson.se (Postfix) with ESMTP id 89FEA5606A; Thu, 26 Sep 2013 12:43:45 +0300 (EEST)
Received: from tri60.nomadiclab.com (localhost [127.0.0.1]) by nomadiclab.lmf.ericsson.se (Postfix) with ESMTP id 33045556BD; Thu, 26 Sep 2013 12:43:45 +0300 (EEST)
Message-ID: <524401D7.8040800@ericsson.com>
Date: Thu, 26 Sep 2013 12:43:51 +0300
From: Ari Keränen <ari.keranen@ericsson.com>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20130801 Thunderbird/17.0.8
MIME-Version: 1.0
To: Christer Holmberg <christer.holmberg@ericsson.com>
References: <7594FB04B1934943A5C02806D1A2204B1C4AC625@ESESSMB209.ericsson.se>
In-Reply-To: <7594FB04B1934943A5C02806D1A2204B1C4AC625@ESESSMB209.ericsson.se>
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-Transfer-Encoding: 8bit
X-Virus-Scanned: ClamAV using ClamSMTP
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrKLMWRmVeSWpSXmKPExsUyM+Jvre4tRpcgg033hCxevZjNaHH11WcW ByaPJUt+Mnl8ufyZLYApissmJTUnsyy1SN8ugSvj/mnPgj+CFWtaG9kbGI/ydjFyckgImEj8 vXOHFcIWk7hwbz1bFyMXh5DAYUaJTVeb2EASQgIbGCXaX9RCJPYySvy9NocRwlnHKPF9/36o lhWMEp83bWYHaeEV0JbY/3kBI4jNIqAq0fd4CdgoNgFHidsPX4LtExVIlti79DQzRL2gxMmZ T1hAbBEBM4nrn3uZQIYyC0xjlDh5pR+sSFjAXmLPm0vMEDf5Suy99BrM5hTwk1iweC+QzQHU YC/xYGsZSJhZQF6ieetsZojf1CSuntsE1aoqcfXfK8YJjKKzkKyehdA9C0n3AkbmVYzsuYmZ OenlhpsYgUF/cMtv3R2Mp86JHGKU5mBREuf98NY5SEggPbEkNTs1tSC1KL6oNCe1+BAjEwen VAOjfmOprf4TJs83G6drba58fMhX8VG1647ti6QmWj1wfnQ70yu8Pu+XosPF6X2MaXkubKnm 8xtX/ftXujRqouyBpo0HNslzl3x+KsLg4bnV6+P712z/tH/vkp7LFDmr1mqS3LyDPkKfBOOu 3WouCN30fcdPUY9lHTebTzxb7354btlGI0u1FWXTlFiKMxINtZiLihMBhRa50kgCAAA=
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, "draft-ietf-hip-reload-instance.all@tools.ietf.org" <draft-ietf-hip-reload-instance.all@tools.ietf.org>
Subject: Re: [Gen-art] Genart review of draft-ietf-hip-reload-instance-08
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: Thu, 26 Sep 2013 09:44:01 -0000

Thanks for the review and comments Christer! We'll take these into 
account for the next revision.


Cheers,
Ari

On 9/26/13 10:25 AM, Christer Holmberg wrote:
> 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-hip-reload-instance-08
>
> Reviewer:                           Christer Holmberg
>
> Review Date:                     26 September 2013
>
> IETF LC End Date:             30 September 2013
>
> IETF Telechat Date:         N/A
>
> Summary:  The document is well written, and ready for publication, with
> some minor editorial nits that I ask the authors to consider fixing.
>
> Major Issues: None
>
> Minor Issues: None
>
> Nits/editorial comments:
>
> Abstract:
>
> ------------
>
> Q_A_1:
>
> I would suggest to modify “is the” to “specifies the”.
>
> Section 1:
>
> -------------
>
> Q_1_1:
>
> In the abstract the text says “Host Identity Protocol-Based Overlay
> Networking Environment (HIP BONE)”, and in Section 1 the text says “HIP
> BONE (Host Identify Protocol-Based Overlay Networking Environment)”. I
> suggest to be consistent.
>
> Q_1_2:
>
> Throughout the document, you talk about “RELOAD HIP BONE”. For that
> reason, I suggest to expand:
>
> “This document addresses all the previous items and provides additional
> details needed to built RELOAD-based HIP BONEs.”
>
> …to:
>
> “This document addresses all the previous items and provides additional
> details needed to built RELOAD-based HIP BONEs, referred to as RELOAD
> HIP BONEs.”
>
> Section 3:
>
> -------------
>
> Q_3_1:
>
> I suggest to modify:
>
> “The peer protocol to be used is RELOAD, which is specified in
> [I-D.ietf-p2psip-base].”
>
> …to:
>
> “The peer protocol to be used is RELOAD [I-D.ietf-p2psip-base].”
>
> Q_3_2:
>
> Please expand RELOAD on first occurrence (outside the abstract).
>
> Section 4:
>
> -------------
>
> Q_4_1:
>
> Please expand ORCHID on first occurrence.
>
> Section 5:
>
> -------------
>
> Q_5_1:
>
> I suggest to remove “defined in Section 6.5 of [I-D.ietf-p2psip-base]”,
> as you already state this in Section 3.
>