Reviewer: Radek Krejčí Review result: Ready with Nits The I-D document contains a single YANG module ietf-vpn-common. The I-D as well as the module itself are in a good shape, I have just two notes: * module's description This is my fault since I came with it in the previous review. There should be really `Section 4.c` as you had. What confused me is how the RFC shows it in its htmlized version, where the link is connected just with the 'Section 4' string and pointing to the RFC itself instead of actually referenced Trust legal document. * `service-status` grouping While the `status-timestamp` grouping was modified since my previous review, the `service-status` still contains only the service-status/status/oper-status container with the config false flag. My arguments are still the same, the description of the other items there says that they are status information, so they should be specified that way. The uses statement doesn't have its own config statement, so if you want to place the mentioned groupings into config true data, an extra grouping or refine will be required. -- last-call mailing list last-call@xxxxxxxx https://www.ietf.org/mailman/listinfo/last-call
