Re: [Lime] Ben Campbell's No Objection on draft-ietf-lime-yang-connectionless-oam-14: (with COMMENT)

Qin Wu <bill.wu@huawei.com> Thu, 26 October 2017 04:48 UTC

Return-Path: <bill.wu@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 EB66413A5BC; Wed, 25 Oct 2017 21:48:37 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.22
X-Spam-Level:
X-Spam-Status: No, score=-4.22 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, URIBL_BLOCKED=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 Kxc3EIqX6HUK; Wed, 25 Oct 2017 21:48:36 -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 08826138BE2; Wed, 25 Oct 2017 21:48:34 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml708-cah.china.huawei.com) ([172.18.7.190]) by lhrrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id DRI52318; Thu, 26 Oct 2017 04:48:32 +0000 (GMT)
Received: from NKGEML414-HUB.china.huawei.com (10.98.56.75) by lhreml708-cah.china.huawei.com (10.201.108.49) with Microsoft SMTP Server (TLS) id 14.3.361.1; Thu, 26 Oct 2017 05:48:31 +0100
Received: from NKGEML513-MBX.china.huawei.com ([169.254.1.105]) by nkgeml414-hub.china.huawei.com ([10.98.56.75]) with mapi id 14.03.0235.001; Thu, 26 Oct 2017 12:48:27 +0800
From: Qin Wu <bill.wu@huawei.com>
To: Ben Campbell <ben@nostrum.com>, The IESG <iesg@ietf.org>
CC: "draft-ietf-lime-yang-connectionless-oam@ietf.org" <draft-ietf-lime-yang-connectionless-oam@ietf.org>, Ron Bonica <rbonica@juniper.net>, Carlos Pignataro <cpignata@cisco.com>, "lime-chairs@ietf.org" <lime-chairs@ietf.org>, "cpignata@cisco.com" <cpignata@cisco.com>, "lime@ietf.org" <lime@ietf.org>
Thread-Topic: Ben Campbell's No Objection on draft-ietf-lime-yang-connectionless-oam-14: (with COMMENT)
Thread-Index: AQHTTflyxuC7ESX9Ak6dVWHyQdtO/6L1jiyw
Date: Thu, 26 Oct 2017 04:48:26 +0000
Message-ID: <B8F9A780D330094D99AF023C5877DABA9AC17700@nkgeml513-mbx.china.huawei.com>
References: <150898118424.24092.2532947954971145433.idtracker@ietfa.amsl.com>
In-Reply-To: <150898118424.24092.2532947954971145433.idtracker@ietfa.amsl.com>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.136.79.163]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A010202.59F16921.002D, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0, ip=169.254.1.105, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32
X-Mirapoint-Loop-Id: 15ad4b93254d89c886cab80357ed03d8
Archived-At: <https://mailarchive.ietf.org/arch/msg/lime/IjWU70UFnPs_2l4471haEQq8Ytk>
Subject: Re: [Lime] Ben Campbell's No Objection on draft-ietf-lime-yang-connectionless-oam-14: (with COMMENT)
X-BeenThere: lime@ietf.org
X-Mailman-Version: 2.1.22
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, 26 Oct 2017 04:48:38 -0000

Thanks Ben.
-----邮件原件-----
发件人: Ben Campbell [mailto:ben@nostrum.com] 
发送时间: 2017年10月26日 9:26
收件人: The IESG
抄送: draft-ietf-lime-yang-connectionless-oam@ietf.org; Ron Bonica; Carlos Pignataro; lime-chairs@ietf.org; cpignata@cisco.com; lime@ietf.org
主题: Ben Campbell's No Objection on draft-ietf-lime-yang-connectionless-oam-14: (with COMMENT)

Ben Campbell has entered the following ballot position for
draft-ietf-lime-yang-connectionless-oam-14: No Objection

When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-lime-yang-connectionless-oam/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

I have some mostly editorial comments:

- I agree with Adam's comments about readability. Please also consider an editing pass for grammar--I noted some missing articles, plural disagreement, etc.

[Qin]: Will fix them. Thanks.

- Please check IDNits. It has some complaints about references. I will leave it to the authors to determine if they are real.

[Qin]: will investigate, thanks.

-1, paragraph starting with "The different OAM tools...": The phrase "After the connection is established" is missing an article.

[Qin]: Good catch, will fix this.
-2.2: "RPC operation - A specific Remote Procedure Call.": That seems like a circular definition.

[Qin]: This follows definition in terminology section of RFC7950, we can add reference to RFC7950 for this.

-3, first paragraph: 'The model augments "/networks/network/node" path...':
missing article.

[Qin]: Okay.

-3.1, 6th paragraph: "... these parameters are not explicit configured":
s/explicit/explicitly

[Qin]:Okay, will fix this.

-5.2.1.2, 2nd paragraph: s/user/users

[Qin]: Okay.

-6, third paragraph: " writable/creatable/deletable": Please don't use "/" as a substitute for conjunctions.

[Qin]: This follows template defined in YANG security guideline
https://trac.ietf.org/trac/ops/wiki/yang-security-guidelines
But if you like, we can change this.