Skip to main content

Last Call Review of draft-ietf-netconf-tls-client-server-24
review-ietf-netconf-tls-client-server-24-yangdoctors-lc-bierman-2021-05-25-00

Request Review of draft-ietf-netconf-tls-client-server-23
Requested revision 23 (document currently at 41)
Type Last Call Review
Team YANG Doctors (yangdoctors)
Deadline 2021-05-08
Requested 2021-04-20
Requested by Mahesh Jethanandani
Authors Kent Watsen
I-D last updated 2021-05-25
Completed reviews Yangdoctors Last Call review of -03 by Andy Bierman (diff)
Yangdoctors Last Call review of -24 by Andy Bierman (diff)
Secdir Last Call review of -25 by Watson Ladd (diff)
Comments
This document was reviewed by a YANG doctor at revision -03. We are now at revision -23, and the document has changed substantially since then. Thus a request to review it again.
Assignment Reviewer Andy Bierman
State Completed
Request Last Call review on draft-ietf-netconf-tls-client-server by YANG Doctors Assigned
Posted at https://mailarchive.ietf.org/arch/msg/yang-doctors/Dx4jZAfW9XiiI4Vo7E0-xrGq2HM
Reviewed revision 24 (document currently at 41)
Result Ready
Completed 2021-05-25
review-ietf-netconf-tls-client-server-24-yangdoctors-lc-bierman-2021-05-25-00
Comments:

I am not commenting on the TLS 1.0 and 1.3 onging discussions.
The WG decision does not impact the YANG module review.

1) Measuring Interoperability for groupings and identities

[same comment for SSH and TLS drafts]

These modules are intentionally abstract.
There are no protocol-accessible objects defined at all.
Interoperability is usually measured in the context of a
specific protocol (e.g., NETCONF).

There is an assumption that interoperability will be achieved
by some other RFCs that will have "uses" statements to create
protocol-accessible or otherwise implementable objects.

There is also an assumption that the groupings will be used the
same everywhere, and the only difference will be the
path from root to the objects in these groupings.
In fact, the "refine" statement allows each usage to be
different.

Perhaps the drafts should mention these interoperability issues.


2)  mandatory choice of only optional-to-implement cases

The choice /ietf-tls-client:client-identity/auth-type
is mandatory but all cases have if-feature-stmts.
Does draft mention 1 of the 4 features MUST be implemented?