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
>