Skip to main content

Last Call Review of draft-ietf-netmod-schema-mount-10
review-ietf-netmod-schema-mount-10-opsdir-lc-ersue-2018-06-29-00

Request Review of draft-ietf-netmod-schema-mount
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2018-06-29
Requested 2018-06-15
Authors Martin Björklund , Ladislav Lhotka
Draft last updated 2018-06-29
Completed reviews Secdir Last Call review of -10 by Shawn M Emery (diff)
Opsdir Last Call review of -10 by Mehmet Ersue (diff)
Rtgdir Telechat review of -10 by Matthew Bocci (diff)
Genart Last Call review of -10 by Joel M. Halpern (diff)
Genart Telechat review of -11 by Joel M. Halpern (diff)
Assignment Reviewer Mehmet Ersue
State Completed
Review review-ietf-netmod-schema-mount-10-opsdir-lc-ersue-2018-06-29
Reviewed revision 10 (document currently at 12)
Result Has Nits
Completed 2018-06-29
review-ietf-netmod-schema-mount-10-opsdir-lc-ersue-2018-06-29-00
I have reviewed this document as part of the Operational directorate's ongoing
effort to review all IETF documents being processed by the IESG.  These
comments were written with the intent of improving the operational aspects of
the IETF drafts. Comments that are not addressed in last call may be included
in AD reviews during the IESG review.  Document editors and WG chairs should
treat these comments just like any other last call comments.

Intended status: Standards Track
Current IESG state: Waiting for Writeup
IANA review state: Not OK / Expert review needed (see
https://datatracker.ietf.org/doc/draft-ietf-netmod-schema-mount/history/)

Summary:
The document defines a mechanism to add the schema trees defined by a set of
YANG modules onto a mount point defined in the schema tree in some YANG modules.

I think the document is well-written and can be published after addressing last
issues indicated by different reviews.

A few nits below:

- In 2.  Terminology and Notation
for both "system-controlled interface" and "YANG library checksum":
s/are not/is not/

- In 2.1.  Glossary of New Terms
As there are indeed terms which are not new I would suggest to change the
section title to: "Glossary of Used Terms". One can even merge section 2 and
section 2.1.

- As one of the commenters indicated, the words "schema mount" are often used
casually and without an article. Though what the author means is "the schema
mount mechanism" which is specifically defined in this document.

Regards,
Mehmet