Re: [Gen-art] [Netconf] Genart last call review of draft-ietf-netconf-rfc7895bis-06

Alissa Cooper <alissa@cooperw.in> Wed, 10 October 2018 15:53 UTC

Return-Path: <alissa@cooperw.in>
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 86601130DD9; Wed, 10 Oct 2018 08:53:53 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=cooperw.in header.b=B/HTe7Ip; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=aghcqX87
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 yqPyIevrx1Jf; Wed, 10 Oct 2018 08:53:50 -0700 (PDT)
Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D762A130F59; Wed, 10 Oct 2018 08:53:49 -0700 (PDT)
Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id 1C79021FA2; Wed, 10 Oct 2018 11:53:48 -0400 (EDT)
Received: from mailfrontend1 ([10.202.2.162]) by compute7.internal (MEProxy); Wed, 10 Oct 2018 11:53:48 -0400
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cooperw.in; h= from:message-id:content-type:mime-version:subject:date :in-reply-to:cc:to:references; s=fm1; bh=fAlUScotIYK/xbTGEB1yR4I zeZbyJZU0ekpv/iz8JuQ=; b=B/HTe7IpSv6X/aZb2RtGxD2LpSwVnKHgcdS/+/6 oe5MPUcb8LTj9LXOSvemyRJ2WU6h4VPA0inJp6HmNXXfT1geFXnobpl8DutbzH6U a5+fpKSV/PRqxkiq+9I7UwyiNk4co12JapchsxOQKO3UDJusgUEqHc/bJ0QvBcva 7AQTe81ttnENG78idQfqCN97Vuis7xmbx3PIYtB74bnSejI6gyqvinUs64l6MPTq aQVjDISndBmsEDPwEm3M6hwQouN5/WuJFz0Sa442BUqpiRGAYWlWRCN3vw/p6IPE gkm+Q2dYaZJ7n1PBRRFh/jYzL/XDuYLVUhOfWjmXoQENz5g==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=fAlUSc otIYK/xbTGEB1yR4IzeZbyJZU0ekpv/iz8JuQ=; b=aghcqX87tpEgO99+kQZCrT 6scKsLBPqUBHnyjN/ZgIx/9+EhKV5cDZtf5gizv6Nj3VBuWPdBy4Pnv6lBLXNe4d 9a4/E73gYPWlHYGr1MKhtnCjCWkFGnx/RuBqJFfNJJAdGmYjXCeX+Kyl4rbxmzs7 6AdAY0cv1jRh+YfxNKFrzfeNVvMAx/WX2D8pypzq2sG91Etgroz6VdfCZPUyN8xM q9aXlIVqyLIEeXwN8SYpzRYCZpX6r+swdJkNVe2O9b4JKV5gkBATEp+u4G8MNIsw 4cSqqers3JxXR6WSnvcU5ScwghlAhD9eCZwuPXKd2bGIU9HipWoxH8k9GJkm80dQ ==
X-ME-Sender: <xms:iyC-W0dYaIu7qoInq8V4z5YvcmSr8IZeB7LnhwGh8Gv1Px16GhjQbw>
X-ME-Proxy: <xmx:iyC-W71k8ceMNmpTkYdOTloKlDCkCwrCzs_nUdyioqevj1asH-N0fQ> <xmx:iyC-W8vxeloqvdyjCA8Tk97SiorRoBqXMH22V00NzYyZUO47HH45Ng> <xmx:iyC-W7YD00ZbxowsV3tyW_jGf68R9RIiUmxAZVpc4EJy7zeelc2bsw> <xmx:iyC-WyIMfKz7jBc79Ohre4qK0nkDg6HXFjcpDt7KrJtXrPOJyBUXug> <xmx:iyC-W8jcgXLPntUxyecaXTRRcHb6lijcmC_7y_crqck0P5HXa672eA> <xmx:jCC-W8ZElM18JK-cSLLrMdH2Ydzy4xAtLCJyJTJulobst3_YcnbAeA>
Received: from rtp-alcoop-nitro5.cisco.com (nccm-cmcs-client.cisco.com [173.38.117.70]) by mail.messagingengine.com (Postfix) with ESMTPA id D7403E46BA; Wed, 10 Oct 2018 11:53:46 -0400 (EDT)
From: Alissa Cooper <alissa@cooperw.in>
Message-Id: <C97F6421-B33B-4983-8929-054661231F45@cooperw.in>
Content-Type: multipart/alternative; boundary="Apple-Mail=_734B9BE9-534A-41F3-B6B8-D6F532E02153"
Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\))
Date: Wed, 10 Oct 2018 11:53:45 -0400
In-Reply-To: <20181001.090836.195350082384169070.mbj@tail-f.com>
Cc: General Area Review Team <gen-art@ietf.org>, draft-ietf-netconf-rfc7895bis.all@ietf.org, netconf@ietf.org
To: Martin Bjorklund <mbj@tail-f.com>, Erik Kline <ek=40google.com@dmarc.ietf.org>
References: <153034650620.26529.10990280643529917972@ietfa.amsl.com> <20181001.090836.195350082384169070.mbj@tail-f.com>
X-Mailer: Apple Mail (2.3445.9.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/uA1OuofMQhCZDd0QSu6VRANtrA8>
Subject: Re: [Gen-art] [Netconf] Genart last call review of draft-ietf-netconf-rfc7895bis-06
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 10 Oct 2018 15:53:54 -0000

Erik, thanks for your review. Martin, thanks for your response. I agree that “checksum” is a misnomer here and don’t really understand why it was picked, but at least the text explains that it isn’t a checksum. I entered a No Objection ballot.

Alissa

> On Oct 1, 2018, at 3:08 AM, Martin Bjorklund <mbj@tail-f.com> wrote:
> 
> Hi,
> 
> Thank you for your review!  Comments inline.
> 
> Erik Kline <ek=40google.com@dmarc.ietf.org <mailto:ek=40google.com@dmarc.ietf.org>> wrote:
>> Reviewer: Erik Kline
>> Review result: Ready
>> 
>> I am the assigned Gen-ART reviewer for this draft. The General Area
>> Review Team (Gen-ART) reviews all IETF documents being processed
>> by the IESG for the IETF Chair.  Please treat these comments just
>> like any other last call comments.
>> 
>> For more information, please see the FAQ at
>> 
>> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
>> 
>> Document: draft-ietf-netconf-rfc7895bis-06
>> Reviewer: Erik Kline
>> Review Date: 2018-06-30
>> IETF LC End Date: 2018-06-28
>> IESG Telechat date: Not scheduled for a telechat
>> 
>> Summary: ready
>> 
>> Major issues: none
>> 
>> Minor issues: none
>> 
>> Nits/editorial comments: 
>> 
>> Conceptually, the "checksum" isn't a checksum so much as just a unique
>> identifier. The text in Section 3 text generally seems to acknowledge
>> this (even using checksum in quotation marks), and so I'm left wondering
>> whether "checksum" is really the best name.
>> 
>> I've no strong opinion, just this observation, and nothing that should
>> impede this document (I assume bike-shedding may have already occurred).
> 
> Yes; this used to be called just an "identifier", but people found it
> confusing and it was always referred to as the checksum...
> 
>> Section 2, item 4 "more than one datastores" -> "more than one datastore".
> 
> Fixed.
> 
>> Section 4, I believe ietf-datastores reference can be updated to RFC 8342,
>> if I understand things correctly.
> 
> Fixed.
> 
>> Section 4, Author list entry for Rob Wilton lacks "mailto:" before the
>> email address.
> 
> Fixed.
> 
> 
> 
> /martin
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org <mailto:Gen-art@ietf.org>
> https://www.ietf.org/mailman/listinfo/gen-art <https://www.ietf.org/mailman/listinfo/gen-art>