Re: [i2rs] Yangdoctors last call review of draft-ietf-i2rs-yang-l2-network-topology-13

mohamed.boucadair@orange.com Thu, 25 June 2020 08:58 UTC

Return-Path: <mohamed.boucadair@orange.com>
X-Original-To: i2rs@ietfa.amsl.com
Delivered-To: i2rs@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D5BAF3A0884; Thu, 25 Jun 2020 01:58:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.096
X-Spam-Level:
X-Spam-Status: No, score=-2.096 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=orange.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id dEd9_BtLUFQW; Thu, 25 Jun 2020 01:58:58 -0700 (PDT)
Received: from relais-inet.orange.com (relais-inet.orange.com [80.12.70.36]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 42EF23A0882; Thu, 25 Jun 2020 01:58:58 -0700 (PDT)
Received: from opfednr06.francetelecom.fr (unknown [xx.xx.xx.70]) by opfednr22.francetelecom.fr (ESMTP service) with ESMTP id 49sv806c2wzyfZ; Thu, 25 Jun 2020 10:58:56 +0200 (CEST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=orange.com; s=ORANGE001; t=1593075536; bh=Y31LQ8HywCNTwKgdxNdwCW8dhK+NJvydJ56Rz2A5Ayo=; h=From:To:Subject:Date:Message-ID:Content-Type: Content-Transfer-Encoding:MIME-Version; b=R37Yv08MmNOBZ0wWhFV9wePCtMXkHXSU1SjVKqb5860iDA0WwTmxwRjrx9MARpDGA 0qmHj3iMollCvFJDwtcPdt9KvbZatQkYixjerkCIbrwXQWU3k240K8ba0di6drVWO/ x8GV4BtPadPZ1XuuxgdWp0jBIkycTvk4cCf2LwqHdDT/rOdfOfJfqBVN0pAaX2iecB X76A8SriMMk+sEGPghkynuQdmQ00nEXuC8WxoYh1k2tl7BSdLnByjuF0gWS6Tu0xms o2dJjxN71eLYI3J/WeoTgvRWGOJZYuiGZqbHgmc7BL2XoAhcBHeTa+56MgGfp9akPf Hm27VZMsbhvmg==
Received: from Exchangemail-eme6.itn.ftgroup (unknown [xx.xx.13.23]) by opfednr06.francetelecom.fr (ESMTP service) with ESMTP id 49sv805nlDzDq7L; Thu, 25 Jun 2020 10:58:56 +0200 (CEST)
From: <mohamed.boucadair@orange.com>
To: Ladislav Lhotka <lhotka@nic.cz>, "yang-doctors@ietf.org" <yang-doctors@ietf.org>
CC: "last-call@ietf.org" <last-call@ietf.org>, "draft-ietf-i2rs-yang-l2-network-topology.all@ietf.org" <draft-ietf-i2rs-yang-l2-network-topology.all@ietf.org>, "i2rs@ietf.org" <i2rs@ietf.org>
Thread-Topic: Yangdoctors last call review of draft-ietf-i2rs-yang-l2-network-topology-13
Thread-Index: AQHWSWV73QgI8kN43ka/Q+H8n+VA0ajpCGMw
Date: Thu, 25 Jun 2020 08:58:55 +0000
Message-ID: <22029_1593075536_5EF46750_22029_153_1_787AE7BB302AE849A7480A190F8B9330314E6E61@OPEXCAUBMA2.corporate.adroot.infra.ftgroup>
References: <159292032253.25999.14074401115344645476@ietfa.amsl.com>
In-Reply-To: <159292032253.25999.14074401115344645476@ietfa.amsl.com>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.114.13.245]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2rs/Qx9iXfW-JNt18mGUi4eSPcsQqA0>
Subject: Re: [i2rs] Yangdoctors last call review of draft-ietf-i2rs-yang-l2-network-topology-13
X-BeenThere: i2rs@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Interface to The Internet Routing System \(IRS\)" <i2rs.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/i2rs>, <mailto:i2rs-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/i2rs/>
List-Post: <mailto:i2rs@ietf.org>
List-Help: <mailto:i2rs-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/i2rs>, <mailto:i2rs-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 25 Jun 2020 08:59:00 -0000

Hi Lada, 

Thank you for the review. 

We have already fixed the first two comments in our local copy. We updated the text to fix the two remaining ones: 

FWIW, the updated text can be seen at: 
https://github.com/boucadair/draft-ietf-i2rs-yang-l2-network-topology/blob/master/draft-ietf-i2rs-yang-l2-network-topology-14.txt 

diff:
https://github.com/boucadair/draft-ietf-i2rs-yang-l2-network-topology/blob/master/diff-IETF-LC.pdf 

Cheers,
Med

> -----Message d'origine-----
> De : Ladislav Lhotka via Datatracker [mailto:noreply@ietf.org]
> Envoyé : mardi 23 juin 2020 15:52
> À : yang-doctors@ietf.org
> Cc : last-call@ietf.org; draft-ietf-i2rs-yang-l2-network-
> topology.all@ietf.org; i2rs@ietf.org
> Objet : Yangdoctors last call review of draft-ietf-i2rs-yang-l2-network-
> topology-13
> 
> Reviewer: Ladislav Lhotka
> Review result: Ready with Nits
> 
> I already reviewed revision -04 of this document with the conclusion that
> from
> YANG point of view it is ready to be published. It is still the case with
> the
> current revision -13. All my earlier comments have been addressed.
> 
> I appreciate the example in Appendix B, it is really useful. However, I
> discovered several problems with the JSON instance data:
> 
> - In all 6 entries of the "ietf-network-topology:link" list, commas are
> missing
> after the "source" object.
> 
> - The identifier "ietf-l2-topology:l2-termination-point-attributes" is
> split
> between two lines (7 times), which makes it invalid. While this is
> explained in
> the introductory text, I would suggest to find another way of satisfying
> the 72
> character limit that doesn't affect the instance data validity. One option
> is
> to use the convention of draft-ietf-netmod-artwork-folding-12, but it is
> also
> possible to simply dedent the offending lines.
> 
> - According to the rules of RFC 7951, the identifier of "termination-point"
> list needs to be qualified with module name, i.e.
> "ietf-network-topology:termination-point".
> 
> - The format of "mac-address" leaves doesn't match the regex pattern of
> their
> types: semicolons rather than dashes have to be used as octet separators.
> 


_________________________________________________________________________________________________________________________

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.