Last Call Review of draft-ietf-netconf-zerotouch-15
review-ietf-netconf-zerotouch-15-yangdoctors-lc-bogdanovic-2017-09-08-00

Request Review of draft-ietf-netconf-zerotouch-14
Requested rev. 14 (document currently at 25)
Type Last Call Review
Team YANG Doctors (yangdoctors)
Deadline 2017-07-31
Requested 2017-07-10
Requested by Mehmet Ersue
Other Reviews Secdir Last Call review of -22 by David Mandelberg (diff)
Secdir Last Call review of -25 by David Mandelberg
Genart Last Call review of -25 by Roni Even
Review State Completed
Reviewer Dean Bogdanović
Review review-ietf-netconf-zerotouch-15-yangdoctors-lc-bogdanovic-2017-09-08
Posted at https://mailarchive.ietf.org/arch/msg/yang-doctors/XrGtBc9PX1W11TFKhiZRfO_whzA
Reviewed rev. 15 (document currently at 25)
Review result Ready with Nits
Draft last updated 2017-09-08
Review completed: 2017-09-08

Review
review-ietf-netconf-zerotouch-15-yangdoctors-lc-bogdanovic-2017-09-08

Reviewer: Dean Bogdanovic
Review result: Ready with caveat


Modules:
ietf-zerotouch-information.yang
ietf-zerotouch-bootstrap-server.yang

YANG Usage:

I did not find anything wrong in any module, but there are no tools to valid usage of YANG data, which is used by ietf-zerotouch-information.yang module.
So couldn’t get full validation of the module with pyang and lintyang tools.

Comments:
1. As stated above, couldn’t get the full validation with the tools of the modules, but looking through the code and the draft, the authors are following the general recommendations by draft-dsdt-nmda-guidelines. Saying that, am fine with moving the draft along, but this is an issues that has to be addressed very soon, as there will be more drafts using YANG data that will not be able to get validated.

2. I couldn’t see how this modules would work well with draft-ietf-rtgwg-lne-model. That draft describes how different virtual systems with different administrators can run on the same HW. How can we boot different versions of SW (lnes) to run on the HW? Or should it state that this draft has no plans to support use cases described in draft-ietf-rtgwg-lne-model.

Regards,

Dean