Re: [i2rs] Yangdoctors early review of draft-ietf-i2rs-rib-data-model-09
Alia Atlas <akatlas@gmail.com> Fri, 09 February 2018 14:51 UTC
Return-Path: <akatlas@gmail.com>
X-Original-To: i2rs@ietfa.amsl.com
Delivered-To: i2rs@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2E676126C0F; Fri, 9 Feb 2018 06:51:21 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.699
X-Spam-Level:
X-Spam-Status: No, score=-2.699 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.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 di-k8EADMROr; Fri, 9 Feb 2018 06:51:17 -0800 (PST)
Received: from mail-oi0-x231.google.com (mail-oi0-x231.google.com [IPv6:2607:f8b0:4003:c06::231]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 9B23C124205; Fri, 9 Feb 2018 06:51:17 -0800 (PST)
Received: by mail-oi0-x231.google.com with SMTP id l8so6290572oig.0; Fri, 09 Feb 2018 06:51:17 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=zdU7YXEWLlpMl41csLgo200OpHLcGqF/o4wFLNnL5Mg=; b=X7PtrNO6QwEHom9qREyLncMtqVav8xCVMmlj4oDmgqCwbgsSVXi9zHJXczKzpxSwds aoOP2vGMZQwzdn1JM7sl1MyL5xu5zyq7gPH3T45Qq0D+bDEiXVYeu0tjx4BSVu79uRBQ YZMwMF6+iBq2KRjmmKgIs+DM738IeE3h+F1G+f2A/bLD81pA8Q/+sYkUBU/aFewgdZqR CFYyF4k1ld2r0ZfaaMkZPYpSFuzCDtOThpx+dZL0K+Hn53vDVQGf+WAksaJHDjiX3BNg eUjmyhC6ZLe3dMktwpb+LOaCF+XZhwlU1QLPgPei4cz3TSrCbhZ103r4KEIOcgtVfUHv oWwQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=zdU7YXEWLlpMl41csLgo200OpHLcGqF/o4wFLNnL5Mg=; b=bxT2G+m7nK8SJqaKML9O/WbGb/jOsW7b+6b1tUO3ony8WfZsWMnKO7jWPNYlo6m3+7 xd4kDYXlGcsruWXt8fqDftdxrw3E7HaJCqQM88KfLWQPbqv3p5gidWIy515Q7qAb6mMx KUhEyuFHxQypB6LlEAgoEoz8nOB3w3dHOiUEDrRB8uJZnNLC+5t/XccfzjY6F+8hIuwo zwTtPyakftZOe1TwabdoZUpXB7kOJu3P593qw152csCSloS5XYTQs/DYkDXuu+OZRN6N 2h+NZv4T1xtMRpf/Ek98T59TrxupUquGO5d2yyDiSuaB3I8ebIzgIYeLHIK4KKw2fDgX /Pog==
X-Gm-Message-State: APf1xPDQGznl8L2fHqECSNzRd4rTs6kl0rbNOgBn5yxyqdrmeekUK36V Tp3emJ7roVVqwaCSJQ9gcNL+vmkFzg9dzjZSQTk=
X-Google-Smtp-Source: AH8x226I67zR2QVedhXplVYR8BIOCR292NULLwR+ZU/WbfTfkENCwTV5KTZbNewN5sxa4m+wf/J3E4uEheGXMGHr6yc=
X-Received: by 10.202.212.70 with SMTP id l67mr1846538oig.43.1518187876785; Fri, 09 Feb 2018 06:51:16 -0800 (PST)
MIME-Version: 1.0
Received: by 10.157.40.246 with HTTP; Fri, 9 Feb 2018 06:51:16 -0800 (PST)
In-Reply-To: <HE1PR0702MB358050A9745D827C522AB23993E80@HE1PR0702MB3580.eurprd07.prod.outlook.com>
References: <151626438402.10775.7250014155413667607@ietfa.amsl.com> <HE1PR0702MB35806D6985E0A7754F840EE193E80@HE1PR0702MB3580.eurprd07.prod.outlook.com> <009801d39042$6a4b08e0$3ee11aa0$@ndzh.com> <HE1PR0702MB358050A9745D827C522AB23993E80@HE1PR0702MB3580.eurprd07.prod.outlook.com>
From: Alia Atlas <akatlas@gmail.com>
Date: Fri, 09 Feb 2018 09:51:16 -0500
Message-ID: <CAG4d1rfHGYtB6iNdRS2cp=Q9qrF8trOizndwQqPx6_E8dZ-obw@mail.gmail.com>
To: Amit Dass <amit.dass@ericsson.com>
Cc: Susan Hares <shares@ndzh.com>, "i2rs@ietf.org" <i2rs@ietf.org>, "draft-ietf-i2rs-rib-data-model.all@ietf.org" <draft-ietf-i2rs-rib-data-model.all@ietf.org>
Content-Type: multipart/alternative; boundary="001a113deff419224d0564c8a924"
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2rs/qg8DpptJZh0Ipa1HnPGloYrVjh0>
Subject: Re: [i2rs] Yangdoctors early review of draft-ietf-i2rs-rib-data-model-09
X-BeenThere: i2rs@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Interface to The Internet Routing System \(IRS\)" <i2rs.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/i2rs>, <mailto:i2rs-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/i2rs/>
List-Post: <mailto:i2rs@ietf.org>
List-Help: <mailto:i2rs-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/i2rs>, <mailto:i2rs-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 09 Feb 2018 14:51:21 -0000
Hi Amit, It has been three weeks. Can you please get the update ASAP? I need to get it reviewed and into IETF Last Call. As you know, I am closing I2RS at IETF 101 - which means this draft needs to be on the March 8 telechat. That means it really should be in IETF Last Call by February 16 - and I am traveling all next week and busy. I REALLY need it ASAP to review. Regards, Alia On Thu, Jan 18, 2018 at 4:58 AM, Amit Dass <amit.dass@ericsson.com> wrote: > Hi Sue, > > I expect to have some free time during this week. Should be able to send > the update by Monday next week. > > Best regards, > Amit > > -----Original Message----- > From: Susan Hares [mailto:shares@ndzh.com] > Sent: Thursday, January 18, 2018 10:55 AM > To: Amit Dass <amit.dass@ericsson.com>; 'Ebben Aries' <exa@juniper.net>; > yang-doctors@ietf.org > Cc: i2rs@ietf.org; draft-ietf-i2rs-rib-data-model.all@ietf.org; > ietf@ietf.org > Subject: RE: Yangdoctors early review of draft-ietf-i2rs-rib-data-model-09 > > Amit: > > Do you think you and your co-authors can do this within a few days. I > would like to forward the publication request. > > Also, please remember to look at the latest Revised datastore draft and > yang tree module drafts. > > Sue Hares > > -----Original Message----- > From: Amit Dass [mailto:amit.dass@ericsson.com] > Sent: Thursday, January 18, 2018 4:53 AM > To: Ebben Aries; yang-doctors@ietf.org > Cc: i2rs@ietf.org; draft-ietf-i2rs-rib-data-model.all@ietf.org; > ietf@ietf.org > Subject: RE: Yangdoctors early review of draft-ietf-i2rs-rib-data-model-09 > > Thanks Ebben for reviewing the draft. I will update the same based on > below comments and feedback. > > > Best regards, > Amit > > -----Original Message----- > From: Ebben Aries [mailto:exa@juniper.net] > Sent: Thursday, January 18, 2018 9:33 AM > To: yang-doctors@ietf.org > Cc: i2rs@ietf.org; draft-ietf-i2rs-rib-data-model.all@ietf.org; > ietf@ietf.org > Subject: Yangdoctors early review of draft-ietf-i2rs-rib-data-model-09 > > Reviewer: Ebben Aries > Review result: On the Right Track > > 1 module in this draft: > - ietf-i2rs-rib@2017-12-05.yang > > No YANG validation errors or warnings (from pyang 1.7.3 and yanglint > 0.14.59) > > 0 examples are provided in this draft (section 3.12 of > draft-ietf-netmod-rfc6087bis-15) > > Module ietf-i2rs-rib@2017-12-05.yang: > - yang-version statement missing - should be 1.1 > - prefix 'iir' is recommended for this module, would 'rib' suffice better? > - import "ietf-inet-types" should reference RFC 6991 per (not as a comment) > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#section-4.7 > - import "ietf-interfaces" should reference RFC 7223 per > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#section-4.7 > - import "ietf-yang-types" should reference RFC 6991 per > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#section-4.7 > - Since this module imports "ietf-interfaces", a normative references must > be > added per > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#section-3.9 > - prefix "if" in the import "ietf-interfaces" can remove quotes to remain > consistent with other imports > - Remove WG Chairs from contact information per > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#appendix-C > - Module description must contain most recent copyright notice per > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#appendix-C > - Module description should contain note to RFC Ed. and placeholder > reference > to RFC when assigned > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#appendix-C > - Add placeholder reference and note to RFC Ed. for RFC when assigned > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-15#appendix-C > - Security Considerations should be updated to reflect new template at > https://trac.ietf.org/trac/ops/wiki/yang-security-guidelines > - Section 1.2 should be replaced with reference to > draft-ietf-netmod-yang-tree-diagrams-02 rather (as-is in other i2rs YANG > drafts in progress) per > https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis- > 15#section-2.5.1 > - This module contains '12' features. While it is understood the purpose > of > these features in the module, take precaution as to complexity for > clients > if they need to understand >= quantity of features per module in use on a > network-element. > - A few comments exist that are either unecessary or redundant. Encode the > comment intent rather in description fields if need be. > - Per NMDA, which datastores are targeted for the module? Will all RPC > operations be acting upon the dynamic/ephemeral datastore? It is not > clear > to me if the intention is to be persistent or ephemeral > > General comments/Nits: > - references to 'def' could be expanded out to 'definition' > - references to 'decap' could be expanded out to 'decapsulation' for > readability (across definitions and descriptions) > - Follow consistent capitalization of 'RIB' throughout document text. > Mixed > use of 'Rib' and 'rib' exists (Outside of YANG node lowercase > definitions). > - Is it necessary to prefix all nodes under the nexthop container with > "nexthop-"? > - Section 2.5 - route-add RPC - text mentions it is required that the > nh-add > RPC be called as a pre-requisite however if the nh already exists and the > nexthop-id is known, this should not be necessary. In addition, the text > reads 'or return' which should rather be a result of querying the > appropriate node in the data tree. > - In 'IANA Considerations' - s/This document requests to register/This > document registers/ > > >
- [i2rs] Yangdoctors early review of draft-ietf-i2r… Ebben Aries
- Re: [i2rs] Yangdoctors early review of draft-ietf… Amit Dass
- Re: [i2rs] Yangdoctors early review of draft-ietf… Susan Hares
- Re: [i2rs] Yangdoctors early review of draft-ietf… Amit Dass
- Re: [i2rs] Yangdoctors early review of draft-ietf… Alia Atlas
- [i2rs] FW: Yangdoctors early review of draft-ietf… Amit Dass
- Re: [i2rs] Yangdoctors early review of draft-ietf… Susan Hares
- Re: [i2rs] Yangdoctors early review of draft-ietf… Alia Atlas
- Re: [i2rs] Yangdoctors early review of draft-ietf… Alia Atlas
- Re: [i2rs] Yangdoctors early review of draft-ietf… Susan Hares
- Re: [i2rs] FW: Yangdoctors early review of draft-… Alia Atlas
- Re: [i2rs] FW: Yangdoctors early review of draft-… Amit Dass
- Re: [i2rs] Yangdoctors early review of draft-ietf… t.petch
- Re: [i2rs] Yangdoctors early review of draft-ietf… Amit Dass
- Re: [i2rs] Yangdoctors early review of draft-ietf… Amit Dass
- Re: [i2rs] [yang-doctors] Yangdoctors early revie… Mahesh Jethanandani
- Re: [i2rs] [yang-doctors] Yangdoctors early revie… t.petch
- Re: [i2rs] Yangdoctors early review of draft-ietf… Ebben Aries
- Re: [i2rs] [yang-doctors] Yangdoctors early revie… Ebben Aries
- Re: [i2rs] [yang-doctors] Yangdoctors early revie… Reshad Rahman (rrahman)