[yang-doctors] Yangdoctors last call review of draft-ietf-opsawg-vpn-common-06

[Radek Krejčí via Datatracker <[e-mail]>]

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.