Document: draft-ietf-netmod-intf-ext-yang Title: Common Interface Extension YANG Data Models Reviewer: Satoru Matsushima Review result: Ready with Nits
Reviewer: Satoru Matsushima Review Result: Ready with Nits I have reviewed this document as part of the IntArea directorate's ongoing effort to review all IETF documents being processed by the IESG. These comments were written primarily for the benefit of the internet area directors. Document editors and WG chairs should treat these comments just like any other last call comments. Overall this draft was well written and very useful not only from ops view but from the int-area point of view as well, suppress-flapping, dampening to make network stable. Nits: - The lower/higher-layer-if definitions in RFC8343 would need to be better to clarify the differences between sub-interface defined in this draft. - How the module can manage linux networking abstraction for example, bridge, vrf interfaces would be better to mention in somewhere, like if a vlan interfaces has one parent ethernet interface, the vlan interface also have the bridge or vrf interface as another parent (or master). Please check whether the module support that or not. - A peer-interface of an interface pair would usually be associated to a netns. It would be better to mention that, and would be better to make the module to be able to do that. _______________________________________________ netmod mailing list -- [email protected] To unsubscribe send an email to [email protected]
