Skip to main content

Early Review of draft-ietf-teas-actn-vn-yang-10

Request Review of draft-ietf-teas-actn-vn-yang-10
Requested revision 10 (document currently at 25)
Type Early Review
Team YANG Doctors (yangdoctors)
Deadline 2020-12-18
Requested 2020-11-13
Requested by Vishnu Pavan Beeram
Authors Young Lee , Dhruv Dhody , Daniele Ceccarelli , Igor Bryskin , Bin Yeong Yoon
I-D last updated 2020-12-20
Completed reviews Yangdoctors Last Call review of -24 by Andy Bierman (diff)
Rtgdir Early review of -22 by Darren Dukes (diff)
Yangdoctors Early review of -10 by Andy Bierman (diff)
Assignment Reviewer Andy Bierman
State Completed
Request Early review on draft-ietf-teas-actn-vn-yang by YANG Doctors Assigned
Posted at
Reviewed revision 10 (document currently at 25)
Result Ready w/issues
Completed 2020-12-20
Major Issues:


Moderate Issues:

   leaf /ap/access-point-list/access-point-id
   leaf /vn/vn-list/vn-id
   leaf /vn/vn-list/vn-member-list/vn-member-id

   These list keys use type inet:uri.
   You should consider the implementation complexity here.
   Will servers all correctly convert any arbitrary URI to its canonical
   representation? The draft should address this issue.

  leaf /vn/vn-list/oper-status
  leaf /vn/vn-list/admin-status

  These objects use vn-status-type, vn-admin-state-type
  The use of identities for even simple "up/down" status types
  seems extreme. The conformance for an enumeration is clear
  (mandatory), but not for an identityref type.
    - E.g., Is is mandatory for a vendor to support vn-state-up,down?
      A vendor could write their own identities and ignore the standard

  rpc /vn-compute
  The procedure for this operation is not explained here.
  A full description or reference to normative test is needed.
   - what does the server do with the input?
   - what output is expected? Any variants based on the inputs
     should be explained.
   - any interoperability considerations wrt/ use of these
     common groupings in this RPC context?
   - what errors can occur? Specify any error-tags, etc. that
     the server MUST/SHOULD include in the response

Minor Issues:

 - naming inconsistent within /ap and /vn
   access-point is spelled out and vn is not
   Suggest: shorten access-point to ap

 - naming a list entry with the the suffix -list is redundant.
   YANG lists do not have any conceptual container or way to
   reference all the entries (if that what this naming intends)

  - leaf /vn/vn-list/vn-member-list/src/multi-src
  - leaf /vn/vn-list/vn-member-list/dest/multi-dest
    Looks like these leafs should each have a default-stmt.
    What does it mean if the multi-src-dest feature is supported
    but these leafs are missing from the config?