Skip to main content

Last Call Review of draft-ietf-netmod-nmda-diff-09
review-ietf-netmod-nmda-diff-09-opsdir-lc-bhandari-2021-07-02-00

Request Review of draft-ietf-netmod-nmda-diff
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2021-07-02
Requested 2021-06-18
Authors Alexander Clemm , Yingzhen Qu , Jeff Tantsura , Andy Bierman
I-D last updated 2021-07-02
Completed reviews Yangdoctors Last Call review of -06 by Reshad Rahman (diff)
Genart Last Call review of -10 by Matt Joras (diff)
Opsdir Last Call review of -09 by Shwetha Bhandari (diff)
Secdir Last Call review of -09 by Alexey Melnikov (diff)
Assignment Reviewer Shwetha Bhandari
State Completed
Request Last Call review on draft-ietf-netmod-nmda-diff by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/D44giwZmpy9Fmh1HGmF3VuZ0Q54
Reviewed revision 09 (document currently at 12)
Result Has nits
Completed 2021-07-02
review-ietf-netmod-nmda-diff-09-opsdir-lc-bhandari-2021-07-02-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.

 Summary:
This is a Standards Track document that defines an RPC operation to compare
management datastores and returns diffs between the datastores as a yang-patch.

While  most access management of the RPC, ensuring availability of the server
by rate limiting are considered I have an open question to authors: where/how
will operational metrics such as rate of requests received, errors, rate
limiting if applied, server resources consumed to process the request etc,
about this new RPC be defined and reported? This is useful information for
server operation where this RPC is enabled.

Nits:
The RESTCONF example content-type is json but it is set to application/yang-d
that is not present in the registry - should it be application/yang-patch+json?