Last Call Review of draft-ietf-netconf-rfc7895bis-06
review-ietf-netconf-rfc7895bis-06-genart-lc-kline-2018-06-30-00

Request Review of draft-ietf-netconf-rfc7895bis
Requested rev. no specific revision (document currently at 07)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2018-06-28
Requested 2018-06-14
Draft last updated 2018-06-30
Completed reviews Yangdoctors Last Call review of -04 by Reshad Rahman (diff)
Genart Last Call review of -06 by Erik Kline (diff)
Opsdir Last Call review of -06 by Linda Dunbar (diff)
Assignment Reviewer Erik Kline
State Completed
Review review-ietf-netconf-rfc7895bis-06-genart-lc-kline-2018-06-30
Reviewed rev. 06 (document currently at 07)
Review result Ready
Review completed: 2018-06-30

Review
review-ietf-netconf-rfc7895bis-06-genart-lc-kline-2018-06-30

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).

Section 2, item 4 "more than one datastores" -> "more than one datastore".

Section 4, I believe ietf-datastores reference can be updated to RFC 8342,
if I understand things correctly.

Section 4, Author list entry for Rob Wilton lacks "mailto:" before the
email address.