With reference to "YANG Data Model for IEEE 1588v2" 
https://tools.ietf.org/html/draft-ietf-tictoc-1588v2-yang-05

I have a concern about the structure of the YANG, with 3 Top-Level Data 
Definitions - instance-list, transparent-clock-default-ds and 
transparent-clock-port-ds-list, all of them being configuration lists or 
containers.

This structure seems to go against the advice of RFC 6087 (Guidelines for 
Authors and Reviewers of YANG Data Model Documents) Section 4.9: Top Level Data 
Definitions https://tools.ietf.org/html/rfc6087#section-4.9 which recommends 
that only one TLD should exist in a YANG module: "There SHOULD only be one 
top-level data node defined in each YANG module".

I think this could be easily remedied in your YANG module with the addition of 
a top level container (for example ietf-ptp) to contain all the three mentioned 
above.

I fear the structure as is could cause implementation problems with some YANG 
or NETCONF implementations.

Sean Condon
=======================
Sean Condon
System & Software Architect
Frequency & Timing Division
Microsemi Inc.
[email protected]
_______________________________________________
TICTOC mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/tictoc

Reply via email to