Re: [Lime] Comments on draft-ietf-lime-yang-connectionless-oam-01

wangzitao <wangzitao@huawei.com> Thu, 27 October 2016 06:37 UTC

Return-Path: <wangzitao@huawei.com>
X-Original-To: lime@ietfa.amsl.com
Delivered-To: lime@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 13ADE1295E2 for <lime@ietfa.amsl.com>; Wed, 26 Oct 2016 23:37:48 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.651
X-Spam-Level:
X-Spam-Status: No, score=-4.651 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-0.431, 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 QqW-Zt0Bl217 for <lime@ietfa.amsl.com>; Wed, 26 Oct 2016 23:37:45 -0700 (PDT)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2252E1295A3 for <lime@ietf.org>; Wed, 26 Oct 2016 23:37:43 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml706-cah.china.huawei.com) ([172.18.7.190]) by lhrrg01-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id CZD23063; Thu, 27 Oct 2016 06:37:41 +0000 (GMT)
Received: from SZXEML431-HUB.china.huawei.com (10.82.67.208) by lhreml706-cah.china.huawei.com (10.201.5.182) with Microsoft SMTP Server (TLS) id 14.3.235.1; Thu, 27 Oct 2016 07:37:39 +0100
Received: from SZXEML501-MBX.china.huawei.com ([169.254.1.73]) by szxeml431-hub.china.huawei.com ([10.82.67.208]) with mapi id 14.03.0235.001; Thu, 27 Oct 2016 14:37:38 +0800
From: wangzitao <wangzitao@huawei.com>
To: Greg Mirsky <gregimirsky@gmail.com>
Thread-Topic: [Lime] Comments on draft-ietf-lime-yang-connectionless-oam-01
Thread-Index: AQHSL5nP6KmTfMCPikahbJkvnWJXeqC7cFBg///HmoCAAJ85UA==
Date: Thu, 27 Oct 2016 06:37:37 +0000
Message-ID: <E6BC9BBCBCACC246846FC685F9FF41EADBAA34@szxeml501-mbx.china.huawei.com>
References: <CA+RyBmXTXzXVedHfZGJ-Hdn2YzbfiQk1t42bNYTAoKLf4rvGHg@mail.gmail.com> <E6BC9BBCBCACC246846FC685F9FF41EADB9871@szxeml501-mbx.china.huawei.com> <CA+RyBmUunNwBGONueGV+eOKvzkDwfYU9oba3RqM7PEXR8STBMw@mail.gmail.com>
In-Reply-To: <CA+RyBmUunNwBGONueGV+eOKvzkDwfYU9oba3RqM7PEXR8STBMw@mail.gmail.com>
Accept-Language: en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.136.78.69]
Content-Type: multipart/alternative; boundary="_000_E6BC9BBCBCACC246846FC685F9FF41EADBAA34szxeml501mbxchina_"
MIME-Version: 1.0
X-CFilter-Loop: Reflected
X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A090202.5811A0B6.0028, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0, ip=169.254.1.73, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32
X-Mirapoint-Loop-Id: 5c1d38566a0d83f7fc0ad05982872f59
Archived-At: <https://mailarchive.ietf.org/arch/msg/lime/jX-9EwVT05KU6W2TiykDdY09m3M>
Cc: "lime@ietf.org" <lime@ietf.org>
Subject: Re: [Lime] Comments on draft-ietf-lime-yang-connectionless-oam-01
X-BeenThere: lime@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "Layer Independent OAM Management in Multi-Layer Environment \(LIME\) discussion list." <lime.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lime>, <mailto:lime-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lime/>
List-Post: <mailto:lime@ietf.org>
List-Help: <mailto:lime-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lime>, <mailto:lime-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 27 Oct 2016 06:37:48 -0000

Hi Greg,

Does you concern about item 2 ---“ tp-tools” need be “ro”?

And if yes, in 01 version ( https://tools.ietf.org/html/draft-ietf-lime-yang-connectionless-oam-01.html), we have done below updates:


grouping tp-tools {

      description

        "Test Point OAM Toolset.";

      choice tools {

           default tools-empty;

           config false;

           description

           "choice of test point tools.

            Empty tools means based on Test Point it's implicit

            all OAM tools are present and no further configuration

            is supported.";
         ……

We add “config false” statement, therefore the entire “tools” attributes are modified to read-only (ro).

Best Regards!
-Michael
发件人: Greg Mirsky [mailto:gregimirsky@gmail.com]
发送时间: 2016年10月27日 12:57
收件人: wangzitao
抄送: lime@ietf.org
主题: Re: [Lime] Comments on draft-ietf-lime-yang-connectionless-oam-01

Hi Michael,
I'm looking at https://tools.ietf.org/html/draft-ietf-lime-yang-connectionless-oam-01.html and still find the following:

      |        |  +--:(tools-ip)

      |        |  |  +--ro rfc792<https://tools.ietf.org/html/rfc792>?          boolean

      |        |  |  +--ro rfc4443<https://tools.ietf.org/html/rfc4443>?         boolean

      |        |  |  +--ro rfc4884<https://tools.ietf.org/html/rfc4884>?         boolean

      |        |  |  +--ro rfc5837<https://tools.ietf.org/html/rfc5837>?         boolean

      |        |  +--:(tools-bfd)

      |        |  |  +--ro rfc5881<https://tools.ietf.org/html/rfc5881>?         boolean

      |        |  |  +--ro rfc5883<https://tools.ietf.org/html/rfc5883>?         boolean

      |        |  |  +--ro rfc5884<https://tools.ietf.org/html/rfc5884>?         boolean

      |        |  |  +--ro rfc5885<https://tools.ietf.org/html/rfc5885>?         boolean

      |        |  +--:(tools-mpls)

      |        |  |  +--ro rfc4379<https://tools.ietf.org/html/rfc4379>?         boolean

      |        |  |  +--ro rfc4687<https://tools.ietf.org/html/rfc4687>?         boolean

      |        |  |  +--ro rfc4950<https://tools.ietf.org/html/rfc4950>?         boolean

      |        |  |  +--ro mpls-rfc5884?    boolean

      |        |  +--:(tools-pw)

      |        |     +--ro rfc5085<https://tools.ietf.org/html/rfc5085>?         boolean

      |        |     +--ro pw_rfc5885?      boolean

      |        |     +--ro rfc6423<https://tools.ietf.org/html/rfc6423>?         boolean

      |        |     +--ro rfc6310<https://tools.ietf.org/html/rfc6310>?         boolean

      |        |     +--ro rfc7023<https://tools.ietf.org/html/rfc7023>?         boolean



as result of the following:

        case tools-bfd {

          leaf rfc5881<https://tools.ietf.org/html/rfc5881> {

            type boolean;

                    description

                    "rfc5881<https://tools.ietf.org/html/rfc5881> supported.";

          }

          leaf rfc5883<https://tools.ietf.org/html/rfc5883> {

            type boolean;

                    description

                    "rfc5883<https://tools.ietf.org/html/rfc5883> supported.";

          }

          leaf rfc5884<https://tools.ietf.org/html/rfc5884> {

            type boolean;

                    description



                    "rfc5884<https://tools.ietf.org/html/rfc5884> supported.";

          }

          leaf rfc5885<https://tools.ietf.org/html/rfc5885> {

            type boolean;

                    description

                    "rfc5885<https://tools.ietf.org/html/rfc5885> supported.";

          }

        }



Am I looking at the wrong version?



Regards, Greg

On Wed, Oct 26, 2016 at 5:37 PM, wangzitao <wangzitao@huawei.com<mailto:wangzitao@huawei.com>> wrote:
Hi Greg and All,

In 00-version we already adopted Greg’s comments and modified the CL model. In 00-version, the “tools” be modified to “ro”, the “tools-mpls-tp” and “l3vpn case”  be removed.
And in 01 version we add a section to demonstrate the CL model’s capability and relationship with other exist models.
The attachment is a diff, and Greg’s comments is showed below.

Since I update the model in 00-version, maybe it cause misunderstanding.

Best Regards!
-Michael


Dear Authors, WG chairs, et. al,

I don't support adoption of the current version of the draft by the LIME WG. Below are my comments that may explain the reason for non-support:

•         the scope of the model isn’t clearly defined. The title suggests modeling OAM protocols that provide Fault Management (FM) and Performance Measurement (PM) in connectionless networks but the latter part seems absent in the proposed model. If only FM OAM is in the scope of this document, then I’d suggest updating the name of the draft;

•         the model uses references to the number of RFCs, e.g. RFC 5881,  within grouping tp-tools. If these are indication of OAM capability at a Test Point, then why is it rw as it seems to be more logical to be part of operational information, thus be ro;

•         the model still  refers to MPLS-TP even though WG agreed that MPLS-TP OM is an example of OAM for connection-oriented networks;

•         the model extensively refers to VPN cases, e.g. L3VPN, as example of connectionless network. I think that that yet has to be discussed as at the VPN layer network demonstrates characteristics of being connection-oriented;

•         the model does not reference, nor provides examples of use case of existing YANG data models of OAM protocols that are used in IP and IP/MPLS networks:

o   BFD (draft-ietf-bfd-yang);

o   LSP Ping (draft-zheng-mpls-lsp-ping-yang-cfg).



Perhaps another virtual interim meeting like one we had on May 27, 2016 would be helpful.



                Regards,

                                Greg

发件人: Lime [mailto:lime-bounces@ietf.org<mailto:lime-bounces@ietf.org>] 代表 Greg Mirsky
发送时间: 2016年10月26日 23:01
收件人: lime@ietf.org<mailto:lime@ietf.org>
主题: [Lime] Comments on draft-ietf-lime-yang-connectionless-oam-01

Dear All,
the -01version has great discussion of how the proposed YANG model may be changed but doesn't change the model itself. And thus all my comments from the adoption call remain to be addressed through changes to the model itself.

Regards,
Greg