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 29) | |
| Type | Last Call Review | |
| Team | YANG Doctors (yangdoctors) | |
| Deadline | 2017-07-31 | |
| Requested | 2017-07-10 | |
| Requested by | Mehmet Ersue | |
| Draft last updated | 2017-09-08 | |
| Completed reviews |
Yangdoctors Last Call review of -15 by Dean Bogdanović
(diff)
Secdir Last Call review of -22 by David Mandelberg (diff) Secdir Last Call review of -25 by David Mandelberg (diff) Genart Last Call review of -25 by Roni Even (diff) |
|
| Assignment | Reviewer | Dean Bogdanović |
| State | Completed | |
| Review | review-ietf-netconf-zerotouch-15-yangdoctors-lc-bogdanovic-2017-09-08 | |
| Reviewed rev. | 15 (document currently at 29) | |
| Review result | Ready with Nits | |
| 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