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

mohamed.boucadair@orange.com Mon, 29 March 2021 06:48 UTC

From: <mohamed.boucadair@orange.com>
To: =?utf-8?B?UmFkZWsgS3JlasSNw60=?= <rkrejci@cesnet.cz>, "yang-doctors@ietf.org" <yang-doctors@ietf.org>
CC: "draft-ietf-opsawg-vpn-common.all@ietf.org" <draft-ietf-opsawg-vpn-common.all@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "opsawg@ietf.org" <opsawg@ietf.org>
Thread-Topic: Yangdoctors last call review of draft-ietf-opsawg-vpn-common-06
Thread-Index: AQHXI+OjgiGeGL4bEU2vg57rS4bZdKqahQTw
Date: Mon, 29 Mar 2021 06:48:38 +0000
Message-ID: <19651_1617000519_60617847_19651_103_1_787AE7BB302AE849A7480A190F8B93303535E068@OPEXCAUBMA2.corporate.adroot.infra.ftgroup>
References: <161694385847.20490.9530808274534750800@ietfa.amsl.com>
In-Reply-To: <161694385847.20490.9530808274534750800@ietfa.amsl.com>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/g1Dx2rz5pjFZ0u99ZCcwH-Gg3XI>
Subject: Re: [yang-doctors] Yangdoctors last call review of draft-ietf-opsawg-vpn-common-06
Precedence: list

Hi Radek, 

Many thanks for the review. 

Please see the changes at: https://github.com/IETF-OPSAWG-WG/lxnm/commit/e856ce054bfc942f2522f683da14d6b99f52a9a2 

Cheers,
Med

> -----Message d'origine-----
> De : Radek Krejčí via Datatracker [mailto:noreply@ietf.org]
> Envoyé : dimanche 28 mars 2021 17:04
> À : yang-doctors@ietf.org
> Cc : draft-ietf-opsawg-vpn-common.all@ietf.org; last-call@ietf.org;
> opsawg@ietf.org
> Objet : Yangdoctors last call review of draft-ietf-opsawg-vpn-common-
> 06
> 
> 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.
> 
> 


_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.

[yang-doctors] Yangdoctors last call review of dr… Radek Krejčí via Datatracker
Re: [yang-doctors] Yangdoctors last call review o… mohamed.boucadair