Re: RtgDir Early review: draft-ding-rtgwg-arp-yang-model-02
tom petch <ietfa@btconnect.com> Fri, 03 August 2018 09:04 UTC
Return-Path: <ietfa@btconnect.com>
X-Original-To: rtgwg@ietfa.amsl.com
Delivered-To: rtgwg@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 43DB6130E02; Fri, 3 Aug 2018 02:04:19 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 3.188
X-Spam-Level: ***
X-Spam-Status: No, score=3.188 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RATWARE_MS_HASH=2.148, RATWARE_OUTLOOK_NONAME=2.95, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, T_DKIMWL_WL_MED=-0.01, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=btconnect.onmicrosoft.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 MIrg-jv4tpVP; Fri, 3 Aug 2018 02:04:16 -0700 (PDT)
Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80103.outbound.protection.outlook.com [40.107.8.103]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4DDB1130DC7; Fri, 3 Aug 2018 02:04:16 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector1-btconnect-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=imJ6iKlnOJBum4mUli7fZuGIx8erkU9vUr/FYEhOwf0=; b=aCi7Df9QxxJgs7YwwRWynozm8kEDiXqnR+03HyIhUwVyWU6cKm6DMAR8JflFoQZz2P4vlovpe8fCTymw/O7QLTHY+p98IhxZoC2fXdyJJd0ZEk9r/IRpTfYbqcHRujz2mHzqtO6rL/pmUwOPrMeNW6uC0LcWxJEgMRj9y7X/V8M=
Received: from HE1PR0701MB2970.eurprd07.prod.outlook.com (10.168.93.17) by HE1PR0701MB2860.eurprd07.prod.outlook.com (10.168.91.150) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1017.11; Fri, 3 Aug 2018 09:04:13 +0000
Received: from HE1PR0701MB2970.eurprd07.prod.outlook.com ([fe80::2d65:c030:51f1:abf0]) by HE1PR0701MB2970.eurprd07.prod.outlook.com ([fe80::2d65:c030:51f1:abf0%5]) with mapi id 15.20.1017.010; Fri, 3 Aug 2018 09:04:13 +0000
From: tom petch <ietfa@btconnect.com>
To: Mach Chen <mach.chen@huawei.com>, "draft-ding-rtgwg-arp-yang-model.all@ietf.org" <draft-ding-rtgwg-arp-yang-model.all@ietf.org>, "rtgwg-chairs@ietf.org" <rtgwg-chairs@ietf.org>, Robert Wilton <rwilton@cisco.com>
CC: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "rtgwg@ietf.org" <rtgwg@ietf.org>
Subject: Re: RtgDir Early review: draft-ding-rtgwg-arp-yang-model-02
Thread-Topic: RtgDir Early review: draft-ding-rtgwg-arp-yang-model-02
Thread-Index: AQHUKk4X5NIb0BYvyEiB3Z7+Y+y5GA==
Date: Fri, 03 Aug 2018 09:04:13 +0000
Message-ID: <017f01d42b08$850ad780$4001a8c0@gateway.2wire.net>
References: <F73A3CB31E8BE34FA1BBE3C8F0CB2AE292563246@dggeml510-mbx.china.huawei.com> <30620f3f-be62-bdbd-72fb-fab27439351b@cisco.com> <F73A3CB31E8BE34FA1BBE3C8F0CB2AE292566105@dggeml510-mbx.china.huawei.com> <037301d42a4d$aa9dbe60$4001a8c0@gateway.2wire.net> <1e7f439a-b050-0ab2-bce3-9b91e2926b22@cisco.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-clientproxiedby: VI1PR07CA0148.eurprd07.prod.outlook.com (2603:10a6:802:16::35) To HE1PR0701MB2970.eurprd07.prod.outlook.com (2603:10a6:3:4c::17)
authentication-results: spf=none (sender IP is ) smtp.mailfrom=ietfa@btconnect.com;
x-ms-exchange-messagesentrepresentingtype: 1
x-originating-ip: [86.165.129.102]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; HE1PR0701MB2860; 6:4VxWYcRrWaJ+qk4HNFD82lWJ8tt4SsAv3W/ibKwuOGXs9bpAC+Xsy1Ja74761NGJ6EPOWXnmrXFu09GE+y6+1e3o4DxhLLIdahtHy3GrHbXh6vQoB0NDmDre10+wVBIlRNuooSIvQJNq4KCbiUDoqKxwPtp4ts/UB5o+e/jbRWwgpPwq29lZ997XerXSm5dL0pnhuQR6uTznq9RkTybyOM0ZG5rn7LBL1zYowVlL/yAb12zry1NDGre97+1vvVD6XMGQa/tDZqsfdmsaWLkB0FHGiUu1WodNXwMOqbVGtplgt4VfZQFZWyrqq00aJKXl+Sag4F0u5GBiEFfVXBo++y5MboShE/uggkDftS2KYd0IOqiAqqO7h0lplB7usi8jBcABlJXcisAknEM1Y/VUEQXzjO8oZcru1tBdHZTXnaPH68mlCfinUS6csds6s9hE5JkFDEulc9u+F8O7MRL0Cg==; 5:yo/Dw40lVXHHVNzr2cvj2ZSFjPXW4PHz8ZnQNmYGDM8koIFnYyHnFB/Hc23hbgmRXBTDJ4HlWjowm0WUik+c3vHmTTdxJDazvT+3x1jwQ44gcvM0HkBaKd4sL+G2xwnTDGtnyoGbnjdZqdP+ZGO15Q6/AMCWXFs+jgknXxTK3BY=; 7:VAMOgjn1TDnL99Gfu1KvW1fKNVW0pu52tXNBDbCwjcrIehz4i37V28qPeE0SOL/pyaJQnY6G2CH5W/qublLuDRn4XzLM1FhpIMwqO+r3/uZ52HN335zmI92j4C7Qged+Ib2ilfqGuu1ITawaI/KgHrmjfDT9R68bROQK3ItzPubABReTr/mVlvIYVz1CyQlswz4JUNGQAp45hSLzAqr4L1f/BSt8fXAtFpa5DZTBA8LtKxWofh38oYaB+conAYDl
x-ms-office365-filtering-correlation-id: fcd2aa70-8a42-4a35-ebf3-08d5f9201501
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989117)(4534165)(4627221)(201703031133081)(201702281549075)(8990107)(5600074)(711020)(2017052603328)(7193020); SRVR:HE1PR0701MB2860;
x-ms-traffictypediagnostic: HE1PR0701MB2860:
x-microsoft-antispam-prvs: <HE1PR0701MB28600D95F7ECE8A4779A55B9A2230@HE1PR0701MB2860.eurprd07.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(178726229863574)(50582790962513)(219612443155931)(95692535739014);
x-ms-exchange-senderadcheck: 1
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040522)(2401047)(5005006)(8121501046)(3002001)(93006095)(93001095)(3231311)(944501410)(52105095)(10201501046)(6055026)(149027)(150027)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(20161123558120)(20161123564045)(20161123560045)(6072148)(201708071742011)(7699016); SRVR:HE1PR0701MB2860; BCL:0; PCL:0; RULEID:; SRVR:HE1PR0701MB2860;
x-forefront-prvs: 0753EA505A
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(39860400002)(346002)(376002)(366004)(136003)(396003)(13464003)(199004)(189003)(51914003)(86362001)(44736005)(8936002)(6116002)(5250100002)(53936002)(3846002)(6486002)(256004)(14444005)(84392002)(229853002)(1720100001)(1556002)(86152003)(2201001)(52116002)(6512007)(6436002)(6306002)(110136005)(9686003)(2501003)(97736004)(14496001)(2906002)(54906003)(2900100001)(68736007)(66066001)(186003)(5660300001)(316002)(486006)(6506007)(386003)(53546011)(93886005)(7736002)(4326008)(305945005)(76176011)(14454004)(25786009)(446003)(6246003)(26005)(8676002)(81156014)(33896004)(81166006)(105586002)(966005)(102836004)(99286004)(478600001)(106356001)(476003); DIR:OUT; SFP:1102; SCL:1; SRVR:HE1PR0701MB2860; H:HE1PR0701MB2970.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:0; MX:1;
received-spf: None (protection.outlook.com: btconnect.com does not designate permitted sender hosts)
x-microsoft-antispam-message-info: TU1+BAtkQJzz11J8SKA+g2dyJSidfO2/JqwsigsKF1O6RZqhk0rVHErMjd0c43foQhHbTfE50c8OdQpLAwS11XCT1bka5lX1VeoqRHHo9oJHaZTYQWzNNWRzWYemveaRYjhotdfRuo1nBpmWiP734p/I/4zpeiOp91u0xAHnAMBLLJREoPMsziIbgLZAlXkyfxdPcYEMaEuSdBIbbtK0xTkVW3icg2BVV4wF371ReG6g87ZhCZlF6o1bygr2vXARLCniWXD4hFonxg6jF+/Ew5KsJ9F85U28y4Ry8uFjdoIx4u+G0u/05LvYxsZKuJd1yabveOiKjhezDpcS55n+dpUH0b8zc/+bVHHUcH+tod8=
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-ID: <1F3CEF9333A9D6489EF596C2770652B3@eurprd07.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fcd2aa70-8a42-4a35-ebf3-08d5f9201501
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Aug 2018 09:04:13.3885 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0701MB2860
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtgwg/xSmL4ZDxuUxq-9vA_L7p83U2IV4>
X-BeenThere: rtgwg@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: Routing Area Working Group <rtgwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtgwg/>
List-Post: <mailto:rtgwg@ietf.org>
List-Help: <mailto:rtgwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 03 Aug 2018 09:04:20 -0000
----- Original Message ----- From: "Robert Wilton" <rwilton@cisco.com> To: "tom petch" <ietfa@btconnect.com>; "Mach Chen" <mach.chen@huawei.com>; <draft-ding-rtgwg-arp-yang-model.all@ietf.org>; <rtgwg-chairs@ietf.org> Cc: <rtg-dir@ietf.org>; <rtgwg@ietf.org> Sent: Thursday, August 02, 2018 12:06 PM > > On 02/08/2018 11:46, tom petch wrote: > > ----- Original Message ----- > > From: "Mach Chen" <mach.chen@huawei.com> > > Sent: Thursday, August 02, 2018 2:58 AM > > > >> Hi Rob, > >> > >> Looks good to me! > > Well it would if we were allowed to have [References] in the Abstract > > which we are not allowed to have:-) > OK, I can remove those. But the NMDA one I copied verbatim from the > abstract in RFC 8343 ;-) Um, no. Let me try again. RFC8343 has The YANG data model in this document conforms to the Network Management Datastore Architecture (NMDA) defined in RFC 8342. which is fine. For this I-D, you propose >>> The YANG data model in this document conforms to the Network >>> Management Datastore Architecture defined in [RFC8342]. and it is the [ ] that I see as a problem. Abstracts, like YANG Modules, must be plain text so that they can exist independently of an RFC which means that XML/HTML type anchors are not allowed, and I read the [ ] as reflecting an underlying anchor, which needs changing to plain text. Tom Petch > Thanks, > Rob > > > > > > Tom Petch > > > >> Best regards, > >> Mach > >> > >> From: Robert Wilton [mailto:rwilton@cisco.com] > >> Sent: Wednesday, August 01, 2018 11:37 PM > >> To: Mach Chen <mach.chen@huawei.com>; > > draft-ding-rtgwg-arp-yang-model.all@ietf.org; rtgwg-chairs@ietf.org > >> Cc: rtg-dir@ietf.org; rtgwg@ietf.org > >> > >> Hi Mach, > >> > >> Thanks for the comments, we will address all of these. > >> > >> Specifically for the abstract, I propose changing the text to: > >> > >> " > >> > >> This document defines a YANG data model for the management of the > >> > >> Address Resolution Protocol (ARP). It extends the basic ARP > >> > >> functionality contained in the ietf-ip YANG data model, defined in > >> > >> [RFC8344], to provide management of optional ARP features and > >> > >> statistics. > >> > >> > >> > >> The YANG data model in this document conforms to the Network > >> > >> Management Datastore Architecture defined in [RFC8342]. > >> > >> " > >> Thanks, > >> Rob > >> > >> On 01/08/2018 09:48, Mach Chen wrote: > >> > >> Hello > >> > >> > >> > >> I have been selected to do a routing directorate “early” review of > > this draft. > >> https://tools.ietf.org/html/draft-ding-rtgwg-arp-yang-model-02 > >> > >> > >> > >> The routing directorate will, on request from the working group chair, > > perform an “early” review of a draft before it is submitted for > > publication to the IESG. The early review can be performed at any time > > during the draft’s lifetime as a working group document. The purpose of > > the early review depends on the stage that the document has reached. As > > this document is in working group last call, my focus for the review was > > to determine whether the document is ready to be published. Please > > consider my comments along with the other working group last call > > comments. > >> > >> > >> For more information about the Routing Directorate, please see > > http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir > >> > >> > >> Document: draft-ding-rtgwg-arp-yang-model-02 > >> > >> Reviewer: Mach Chen > >> > >> Review Date: 01 August 2018 > >> > >> Intended Status: Standards Track > >> > >> > >> > >> Summary > >> > >> > >> > >> The draft defines a YANG model for ARP configurations, which covers > > static ARP, ARP caching, proxy ARP and gratuitous ARP. The model is very > > short and the content is straightforward. It can be a reasonable start > > point for WG adoption call. > >> > >> > >> General comments: > >> > >> > >> > >> Although I am not a native English speaker, I also feel that the > > document needs some enhancements on its wording and grammar to make it > > more clean and readable. > >> > >> > >> For example, the following text needs some rewording or may be > > removed. > >> Abstract: > >> > >> "The data model performs as > >> > >> a guideline for configuring ARP capabilities on a system. It is > >> > >> intended this model be used by service providers who manipulate > >> > >> devices from different vendors in a standard way." > >> > >> > >> > >> Specific comments: > >> > >> > >> > >> 1. It's lack of the IANA section. > >> > >> > >> > >> 2. Section 3.1 and Section 3.3, suggest to add relevant references to > > ARP caching and gratuitous ARP. > >> > >> > >> 3. import ietf-interfaces { > >> > >> prefix if; > >> > >> description > >> > >> "A Network Management Datastore Architecture (NMDA) > >> > >> compatible version of the ietf-interfaces module > >> > >> is required."; > >> > >> } > >> > >> import ietf-ip { > >> > >> prefix ip; > >> > >> description > >> > >> "A Network Management Datastore Architecture (NMDA) > >> > >> compatible version of the ietf-ip module is > >> > >> required."; > >> > >> } > >> > >> > >> > >> Lack of the reference RFCs. > >> > >> And the descriptions seem not appropriate, some of other descriptions > > in this document have the similar issue, suggest to revise those > > descriptions. > >> > >> > >> In addition, idnits tool shows: > >> > >> > >> > >> == Missing Reference: 'RFC826' is mentioned on line 77, but not > > defined > >> > >> > >> == Missing Reference: 'RFC6536' is mentioned on line 583, but not > > defined > >> > >> > >> ** Obsolete undefined reference: RFC 6536 (Obsoleted by RFC 8341) > >> > >> > >> > >> == Unused Reference: 'I-D.ietf-netmod-rfc7223bis' is defined on line > > 606, > >> but no explicit reference was found in the text > >> > >> > >> > >> == Unused Reference: 'RFC0826' is defined on line 636, but no > > explicit > >> reference was found in the text > >> > >> > >> > >> > >> > >> Best regards, > >> > >> Mach > >> > >> > >> > >> > >> > >> > > > > ---------------------------------------------------------------------- -- > > -------- > > > > > >> _______________________________________________ > >> rtgwg mailing list > >> rtgwg@ietf.org > >> https://www.ietf.org/mailman/listinfo/rtgwg > >> >
- RtgDir Early review: draft-ding-rtgwg-arp-yang-mo… Mach Chen
- Re: RtgDir Early review: draft-ding-rtgwg-arp-yan… Robert Wilton
- Re: RtgDir Early review: draft-ding-rtgwg-arp-yan… Jeff Tantsura
- RE: RtgDir Early review: draft-ding-rtgwg-arp-yan… Mach Chen
- Re: RtgDir Early review: draft-ding-rtgwg-arp-yan… tom petch
- Re: RtgDir Early review: draft-ding-rtgwg-arp-yan… Robert Wilton
- Re: RtgDir Early review: draft-ding-rtgwg-arp-yan… tom petch
- Re: RtgDir Early review: draft-ding-rtgwg-arp-yan… Robert Wilton
- defining signaling parameters for GMPLS tom petch
- Re: [RTG-DIR] defining signaling parameters for G… Lou Berger
- Re: [RTG-DIR] defining signaling parameters for G… tom petch