Early Review of draft-ietf-netmod-intf-ext-yang-04
review-ietf-netmod-intf-ext-yang-04-yangdoctors-early-bierman-2017-04-06-00
| Request | Review of | draft-ietf-netmod-intf-ext-yang-04 |
|---|---|---|
| Requested revision | 04 (document currently at 10) | |
| Type | Early Review | |
| Team | YANG Doctors (yangdoctors) | |
| Deadline | 2017-04-10 | |
| Requested | 2017-03-13 | |
| Requested by | Mehmet Ersue | |
| Authors | Robert Wilton , David Ball , tapsingh@cisco.com , Selvakumar Sivaraj | |
| Draft last updated | 2017-04-06 | |
| Completed reviews |
Yangdoctors Early review of -04
by
Andy Bierman
(diff)
|
|
| Assignment | Reviewer | Andy Bierman |
| State | Completed | |
| Review |
review-ietf-netmod-intf-ext-yang-04-yangdoctors-early-bierman-2017-04-06
|
|
| Reviewed revision | 04 (document currently at 10) | |
| Result | Almost Ready | |
| Completed | 2017-04-06 |
review-ietf-netmod-intf-ext-yang-04-yangdoctors-early-bierman-2017-04-06-00
Overall:
The document is well-written and almost ready for publication
Comments:
1) No examples or guidance for "encaps-type"
The document does not really define the standards value
for this empty choice. There are no examples showing its use.
More work is needed for this part of the module.
The following TODO needs to be resolved and the note removed
from the document:
/*
* TODO - Should we introduce an abstract type to make this
* extensible to new interface types, or vendor
* specific interface types?
*/
2) normative overlap
The text and layout of sec. 3 is good.
The issue is that the YANG module text copies from this section
in some places (leafs) and references it in others (feature definitions)
Some parts like "half-life" are more detailed in the leaf definition
than the plain text in sec. 3
Perhaps the leaf definitions can have reference-stmts added
as needed, so it is clear that the YANG leaf is not the
entire normative text. The YANG descriptions are good,
but maybe not complete wrt/ sec. 3 additional text.
sec 3.7 typo:
the existing the sub-interface
^^^ remove extra 'the'
3) identity ethSubInterface
This identity is used in the encapsulation container when-stmt.
It is not clear if this is intended as a base identity (like identity
sub-interface)
An example for the encapsulation container would help clarify the
expected usage
This also has 2 bases (sub-interface and l2vlan).
Some explanation in the identity-stmt would be helpful
(since this is a new YANG 1.1 construct)
Andy