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

Martin Bjorklund <mbj@tail-f.com> Mon, 01 October 2018 07:08 UTC

Return-Path: <mbj@tail-f.com>
X-Original-To: netconf@ietfa.amsl.com
Delivered-To: netconf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 540B6130934; Mon, 1 Oct 2018 00:08:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=unavailable autolearn_force=no
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 O7tuEegCoB6e; Mon, 1 Oct 2018 00:08:42 -0700 (PDT)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id C5613130DC5; Mon, 1 Oct 2018 00:08:38 -0700 (PDT)
Received: from localhost (unknown [173.38.220.61]) by mail.tail-f.com (Postfix) with ESMTPSA id 6EF7D1AE03DD; Mon, 1 Oct 2018 09:08:37 +0200 (CEST)
Date: Mon, 01 Oct 2018 09:08:36 +0200 (CEST)
Message-Id: <20181001.090836.195350082384169070.mbj@tail-f.com>
To: ek=40google.com@dmarc.ietf.org
Cc: gen-art@ietf.org, draft-ietf-netconf-rfc7895bis.all@ietf.org, ietf@ietf.org, netconf@ietf.org
From: Martin Bjorklund <mbj@tail-f.com>
In-Reply-To: <153034650620.26529.10990280643529917972@ietfa.amsl.com>
References: <153034650620.26529.10990280643529917972@ietfa.amsl.com>
X-Mailer: Mew version 6.7 on Emacs 24.5 / Mule 6.0 (HANACHIRUSATO)
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/Xi7tnAazkEQismdSf2gJ-G9ypyc>
Subject: Re: [Netconf] Genart last call review of draft-ietf-netconf-rfc7895bis-06
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Network Configuration WG mailing list <netconf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netconf>, <mailto:netconf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netconf/>
List-Post: <mailto:netconf@ietf.org>
List-Help: <mailto:netconf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netconf>, <mailto:netconf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 01 Oct 2018 07:08:44 -0000

Hi,

Thank you for your review!  Comments inline.

Erik Kline <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