Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-18.txt
Leeyoung <leeyoung@huawei.com> Mon, 25 March 2019 05:29 UTC
Return-Path: <leeyoung@huawei.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0E7E2120350 for <ccamp@ietfa.amsl.com>; Sun, 24 Mar 2019 22:29:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HK_RANDOM_ENVFROM=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 NOTNj9K43Ow2 for <ccamp@ietfa.amsl.com>; Sun, 24 Mar 2019 22:29:41 -0700 (PDT)
Received: from huawei.com (lhrrgout.huawei.com [185.176.76.210]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 73AE712034E for <ccamp@ietf.org>; Sun, 24 Mar 2019 22:29:41 -0700 (PDT)
Received: from lhreml704-cah.china.huawei.com (unknown [172.18.7.106]) by Forcepoint Email with ESMTP id 815AC289AAC4F83C081E for <ccamp@ietf.org>; Mon, 25 Mar 2019 05:29:39 +0000 (GMT)
Received: from SJCEML701-CHM.china.huawei.com (10.208.112.40) by lhreml704-cah.china.huawei.com (10.201.108.45) with Microsoft SMTP Server (TLS) id 14.3.408.0; Mon, 25 Mar 2019 05:29:38 +0000
Received: from SJCEML521-MBS.china.huawei.com ([169.254.2.5]) by SJCEML701-CHM.china.huawei.com ([169.254.3.224]) with mapi id 14.03.0439.000; Sun, 24 Mar 2019 22:29:32 -0700
From: Leeyoung <leeyoung@huawei.com>
To: tom petch <ietfc@btconnect.com>, "ccamp@ietf.org" <ccamp@ietf.org>
Thread-Topic: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-18.txt
Thread-Index: AQHU0Czod+Lst4P3mEqT94WFCPr+K6Yb6aCA
Date: Mon, 25 Mar 2019 05:29:32 +0000
Message-ID: <7AEB3D6833318045B4AE71C2C87E8E173D104D9F@sjceml521-mbs.china.huawei.com>
References: <051a01d4d02c$a3a190c0$4001a8c0@gateway.2wire.net>
In-Reply-To: <051a01d4d02c$a3a190c0$4001a8c0@gateway.2wire.net>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.220.69.130]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/jagjmiJ0wYd9WQHOw4RLS6G1rJg>
Subject: Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-18.txt
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 25 Mar 2019 05:29:44 -0000
Hi Tom, My apologies for my late response. I agree with all your comments. Version 20 has been uploaded. https://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-wson-yang-20 Please see inline for my comment. If there is still something that I have missed, please let me know. Thanks. Young -----Original Message----- From: tom petch [mailto:ietfc@btconnect.com] Sent: Friday, March 1, 2019 6:47 AM To: Leeyoung <leeyoung@huawei.com>; ccamp@ietf.org Subject: Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-18.txt Looking at -19: - you still have 'wavelenghth' which has a 'h' too many, twice YL>> Fixed -you still have The YANG model described in this document is a WSON technology-specific YANG model augmenting the generic TE topology module [TE-TOPO] based on the information model developed in [RFC7446] and the two encoding documents [RFC7581] and [RFC7579] that developed protocol independent encodings based on [RFC7446]. in the YANG module which MUST be plain text so HTML/XML references are not allowed YL>> rid of all brackets; instead of [TE-TOPO], refer to the YANG module: ietf-te-topology. And the YANG has reference "ITU-T G.698.2 (11/2009) Section 5.3"; - always good to see reference clauses - but it means "ITU-T G.698.2 (11/2009)" needs to appear in the I-D references and needs to be referenced from the body of the I-D YL>> Corrected as you suggested. Introduced G.698.2 as the reference in the introduction of the I-D. reference "TBD"; well, is TBD! YL>> Replaced TBD with RFC 7581. Tom Petch ----- Original Message ----- From: "t.petch" <ietfc@btconnect.com> To: "Leeyoung" <leeyoung@huawei.com>; <ccamp@ietf.org> Sent: Friday, December 07, 2018 12:45 PM > ----- Original Message ----- > From: "Leeyoung" <leeyoung@huawei.com> > To: <ccamp@ietf.org> > Sent: Wednesday, December 05, 2018 5:23 PM > > > > This revision, we hope, incorporated all comments from Tom and others. > The co-authors believe that it is ready for WG LC. > > Almost but I fear that there is a step backwards as well. > > You still have two > Wavelenghth > which should probably be > Wavelength > > More problematically, you have introduced > > description > " ... > The YANG model described in this document is a WSON > technology-specific YANG model augmenting the generic TE > topology module [TE-TOPO] based on the information model > developed in [RFC7446] and the two encoding documents > [RFC7581] and [RFC7579] that developed protocol independent > encodings based on [RFC7446]. > > Those [RFC..] look like XML/HTML style references which will not do. A > YANG module must be plain text so > RFC6163 > is fine in a YANG module but > [RFC6163] > is not, which is why, in my last comment, I suggested putting a > reference to RFC6163 in section 1.4 lest you get an unused reference for > the I-D; which you have done, so that is fine, but all those [RFC...] in > the YANG module must go. AFAICS it is only the description clause that > has them. > > For TE-TOPO, you need to use the same text as you have correctly done on > the YANG import statement i.e. > > "RFC YYYY: YANG Data Model for Traffic Engineering (TE) Topologies"; > > /* Note: The RFC Editor will replace YYYY with the number assigned to > the RFC once draft-ietf-teas-yang-te-topo becomes an RFC.*/ > > Whether this is fixed before or after Last Call is one for the WG > Chairs. > > I probably said before that the nature of the module, which you are > forced to use because of the way that tet: is structured, makes the > module very hard to read and that I have not done; I hope to but may > never find the time. > > Tom Petch > > > Thanks & best regards, > > Young > > > > -----Original Message----- > > From: CCAMP [mailto:ccamp-bounces@ietf.org] On Behalf Of > internet-drafts@ietf.org > > Sent: Wednesday, December 5, 2018 11:18 AM > > To: i-d-announce@ietf.org > > Cc: ccamp@ietf.org > > Subject: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-18.txt > > > > > > A New Internet-Draft is available from the on-line Internet-Drafts > directories. > > This draft is a work item of the Common Control and Measurement Plane > WG of the IETF. > > > > Title : A YANG Data Model for WSON (Wavelength > Switched Optical Networks) > > Authors : Young Lee > > Dhruv Dhody > > Aihua Guo > > Victor Lopez > > Daniel King > > Filename : draft-ietf-ccamp-wson-yang-18.txt > > Pages : 77 > > Date : 2018-12-05 > > > > Abstract: > > This document provides a YANG data model for the routing and > > wavelength assignment (RWA) TE topology in wavelength switched > > optical networks (WSONs). The YANG data model defined in this > > document conforms to the Network Management Datastore Architecture > > defined in RFC 8342. > > > > > > The IETF datatracker status page for this draft is: > > https://datatracker.ietf.org/doc/draft-ietf-ccamp-wson-yang/ > > > > There are also htmlized versions available at: > > https://tools.ietf.org/html/draft-ietf-ccamp-wson-yang-18 > > https://datatracker.ietf.org/doc/html/draft-ietf-ccamp-wson-yang-18 > > > > A diff from the previous version is available at: > > https://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-wson-yang-18 > > > > > > Please note that it may take a couple of minutes from the time of > submission until the htmlized version and diff are available at > tools.ietf.org. > > > > Internet-Drafts are also available by anonymous FTP at: > > ftp://ftp.ietf.org/internet-drafts/ > > > > _______________________________________________ > > CCAMP mailing list > > CCAMP@ietf.org > > https://www.ietf.org/mailman/listinfo/ccamp > > > > _______________________________________________ > > CCAMP mailing list > > CCAMP@ietf.org > > https://www.ietf.org/mailman/listinfo/ccamp >
- [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-18… internet-drafts
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… tom petch
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Beller, Dieter (Nokia - DE/Stuttgart)
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Aihuaguo (Aihua Guo, CRC)
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… tom petch
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung