Skip to main content

IETF Last Call Review of draft-ietf-netconf-yang-library-augmentedby-13
review-ietf-netconf-yang-library-augmentedby-13-yangdoctors-lc-bierman-2025-12-24-00

Request Review of draft-ietf-netconf-yang-library-augmentedby
Requested revision No specific revision (document currently at 17)
Type IETF Last Call Review
Team YANG Doctors (yangdoctors)
Deadline 2026-01-10
Requested 2025-12-20
Requested by Mahesh Jethanandani
Authors Zhuoyao Lin , Benoît Claise , Ignacio Dominguez Martinez-Casanueva
I-D last updated 2026-03-09 (Latest revision 2026-02-20)
Completed reviews Yangdoctors Early review of -07 by Andy Bierman (diff)
Opsdir Early review of -08 by Sheng Jiang (diff)
Yangdoctors IETF Last Call review of -13 by Andy Bierman (diff)
Opsdir IETF Last Call review of -13 by Xiao Min (diff)
Secdir IETF Last Call review of -14 by Rich Salz (diff)
Secdir Telechat review of -15 by Rich Salz (diff)
Comments
The YANG Doctor's review looked at -08 version of the draft. We are now at -13. It would not be a bad idea for Andy to take a quick pass at it.
Assignment Reviewer Andy Bierman
State Completed
Request IETF Last Call review on draft-ietf-netconf-yang-library-augmentedby by YANG Doctors Assigned
Posted at https://mailarchive.ietf.org/arch/msg/yang-doctors/nD1s4ZAJ5mcJTav4XescYOX9dgk
Reviewed revision 13 (document currently at 17)
Result Ready w/nits
Completed 2025-12-24
review-ietf-netconf-yang-library-augmentedby-13-yangdoctors-lc-bierman-2025-12-24-00
Draft: draft-ietf-netconf-yang-library-augmentedby-13.txt
Module: ietf-yang-library-augmentedby
Revision: 2025-05-28
Status: Ready with nits

Issues:  none

Nits:

1) In the grouping description:

s/Leaf-list of the augmentation used/Leaf-list of the augmentations used/

2) same grouping

"modify the conformance of the module."

RFC 7950 does not describe augment in terms of conformance.

Suggest: "modify the schema tree of the module."

3) cut-and-paste in 3 places

The same normative description appears in 3 places:

-  augment "/yanglib:yang-library/yanglib:module-set/yanglib:module"
-  augment "/yanglib:modules-state/yanglib:module"
-  4.2.1.  The scope of augmented-by

Suggest moving this text to the leaf-list description and not
repeating it anywhere.

4) Security considerations

Suggest mentioning that the security considerations in RFC 8525,
sec. 6 apply here.