[netmod] Murray Kucherawy's No Objection on draft-ietf-netmod-factory-default-14: (with COMMENT)

[Murray Kucherawy via Datatracker <[e-mail]>]

Murray Kucherawy has entered the following ballot position for
draft-ietf-netmod-factory-default-14: No Objection

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-netmod-factory-default/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Section 2:
* "All security sensitive data (i.e., private keys, passwords, etc.)  SHOULD be
overwritten ..." presents a choice.  Why would an implementer not do this? *
"Implementors SHOULD reboot the device or otherwise restart processes needed to
bootstrap it." leads me to the same question.

Nits:
* "Upon receiving the RPC" is followed by a list, so please add a colon
* "datastores(e.g.," -- add a space after "datastores"

Section 3:
Nits:
* "The contents of <factory-default> is defined  ..." -- s/is/are/

Section 5:
* "This document registers one URI in the IETF XML Registry [RFC3688]. ..."
should say explicitly that it's the "ns" sub-registry receiving a new entry.