RE: Last Call: <draft-ietf-rtgwg-yang-rip-06.txt> (A YANG Data Model for Routing Information Protocol (RIP)) to Proposed Standard

Xufeng Liu <Xufeng_Liu@jabil.com> Tue, 23 January 2018 22:18 UTC

Return-Path: <Xufeng_Liu@jabil.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DA04912D852; Tue, 23 Jan 2018 14:18:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.121
X-Spam-Level:
X-Spam-Status: No, score=-1.121 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_NEUTRAL=0.779, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=jabil.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 Snm6BitpoNQD; Tue, 23 Jan 2018 14:18:22 -0800 (PST)
Received: from NAM03-CO1-obe.outbound.protection.outlook.com (mail-co1nam03on0097.outbound.protection.outlook.com [104.47.40.97]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 628F912D84E; Tue, 23 Jan 2018 14:18:22 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=jabil.onmicrosoft.com; s=selector1-jabil-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=DGPCfQivC5b8bcIYiMXfPVD5UkIXsdS03L+yT0b+yIA=; b=vaPy1w9oSGh1Ipd5U/wwSe5Xy3Ff3JWIjsSTNFyclZkH3GFrqlBL/MqkeiVPvcckK8JogZSvu4o4MdFBy5051N5Icu6H+ANMl1js3aJj1lzX876xKO3QFVYpPAgodHqRGLIfRpi9aN7FxgWY78BRnj4j5qthFsWDKSfrxxi2A1o=
Received: from BN3PR0201MB0867.namprd02.prod.outlook.com (10.160.154.13) by BN3PR0201MB1027.namprd02.prod.outlook.com (10.161.207.149) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.428.17; Tue, 23 Jan 2018 22:18:18 +0000
Received: from BN3PR0201MB0867.namprd02.prod.outlook.com ([10.160.154.13]) by BN3PR0201MB0867.namprd02.prod.outlook.com ([10.160.154.13]) with mapi id 15.20.0428.019; Tue, 23 Jan 2018 22:18:18 +0000
From: Xufeng Liu <Xufeng_Liu@jabil.com>
To: "tom p." <daedulus@btconnect.com>, ietf <ietf@ietf.org>
CC: "draft-ietf-rtgwg-yang-rip@ietf.org" <draft-ietf-rtgwg-yang-rip@ietf.org>, "rtgwg-chairs@ietf.org" <rtgwg-chairs@ietf.org>, "yingzhen.qu@huawei.com" <yingzhen.qu@huawei.com>, Alia Atlas <akatlas@gmail.com>
Subject: RE: Last Call: <draft-ietf-rtgwg-yang-rip-06.txt> (A YANG Data Model for Routing Information Protocol (RIP)) to Proposed Standard
Thread-Topic: Last Call: <draft-ietf-rtgwg-yang-rip-06.txt> (A YANG Data Model for Routing Information Protocol (RIP)) to Proposed Standard
Thread-Index: AQHTebW41e2nHONyqEGMpcnPNFWcJKNwyEIggAEiZCeAEE898A==
Date: Tue, 23 Jan 2018 22:18:18 +0000
Message-ID: <BN3PR0201MB0867B279FC28A1FA9D2A8155F1E30@BN3PR0201MB0867.namprd02.prod.outlook.com>
References: <005b01d379b6$03415b60$4001a8c0@gateway.2wire.net> <CY1PR0201MB08757725C08E5AE869CA24BEF1170@CY1PR0201MB0875.namprd02.prod.outlook.com> <022201d38c6e$e0854d40$4001a8c0@gateway.2wire.net>
In-Reply-To: <022201d38c6e$e0854d40$4001a8c0@gateway.2wire.net>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-dg-ref: PG1ldGE+PGF0IG5tPSJib2R5LnR4dCIgcD0iYzpcdXNlcnNceGxpdVxhcHBkYXRhXHJvYW1pbmdcMDlkODQ5YjYtMzJkMy00YTQwLTg1ZWUtNmI4NGJhMjllMzViXG1zZ3NcbXNnLWQ1MDRlZGVkLTAwOGEtMTFlOC05YzQxLTE4NWUwZmUzYzQ1Y1xhbWUtdGVzdFxkNTA0ZWRlZS0wMDhhLTExZTgtOWM0MS0xODVlMGZlM2M0NWNib2R5LnR4dCIgc3o9IjY4MDgiIHQ9IjEzMTYxMjE5MjkzMzg3OTk4MyIgaD0ibVA2S1l3VERjd1paYkxvdkNTaDl4T1lDcnE4PSIgaWQ9IiIgYmw9IjAiIGJvPSIxIi8+PC9tZXRhPg==
authentication-results: spf=none (sender IP is ) smtp.mailfrom=Xufeng_Liu@jabil.com;
x-originating-ip: [98.191.72.170]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; BN3PR0201MB1027; 7:AW6Q5HyEuZ6WgieGuqapJv8EJmJ/oZqyQs4VYmnCBREyjgZBzu0jBSAoL4Hc0S8smnjmyCRwNXa9veKzZtYLM4cZGaZ4vkOJYNKaQBpMt9v4UUtPSiOScwGWVHRCs3U52JdFvYD+CiyrLvSzOpj97+LIYLYtNxXMBSlUmNFf1isG+aCoDQlFZH1w0Cn4JPq/GrFjKseEdFOobeTx2gXbf6b6ciN0M2HMmY0oZfEruqJsCRyu1IEdPIwIBXUETUOE
x-ms-exchange-antispam-srfa-diagnostics: SSOS;
x-ms-office365-filtering-ht: Tenant
x-ms-office365-filtering-correlation-id: 95f75933-e7e3-425e-75c2-08d562af34d6
x-microsoft-antispam: UriScan:(178726229863574); BCL:0; PCL:0; RULEID:(7020095)(4652020)(48565401081)(4534165)(4627221)(201703031133081)(201702281549075)(5600026)(4604075)(3008032)(2017052603307)(7153060)(7193020); SRVR:BN3PR0201MB1027;
x-ms-traffictypediagnostic: BN3PR0201MB1027:
x-microsoft-antispam-prvs: <BN3PR0201MB10273AE876A4A19928ED334DF1E30@BN3PR0201MB1027.namprd02.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(178726229863574)(50582790962513)(85827821059158)(100405760836317)(21534305686606);
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040501)(2401047)(5005006)(8121501046)(3231023)(2400081)(944501161)(3002001)(10201501046)(93006095)(93001095)(6055026)(6041288)(201703131423095)(201703031522075)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(20161123562045)(20161123560045)(20161123558120)(6072148)(201708071742011); SRVR:BN3PR0201MB1027; BCL:0; PCL:0; RULEID:; SRVR:BN3PR0201MB1027;
x-forefront-prvs: 05610E64EE
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(39860400002)(376002)(346002)(396003)(366004)(39380400002)(13464003)(199004)(189003)(51444003)(51914003)(7696005)(68736007)(6506007)(76176011)(59450400001)(3280700002)(3660700001)(6116002)(5660300001)(3846002)(2906002)(305945005)(230783001)(4326008)(7736002)(74316002)(6246003)(106356001)(105586002)(39060400002)(66066001)(33656002)(25786009)(6436002)(229853002)(2900100001)(8666007)(6306002)(55016002)(9686003)(72206003)(966005)(53546011)(53936002)(110136005)(99286004)(8936002)(14454004)(81156014)(54906003)(81166006)(478600001)(8676002)(2950100002)(4001150100001)(102836004)(80792005)(86362001)(296002)(316002)(77096007)(97736004)(26005); DIR:OUT; SFP:1102; SCL:1; SRVR:BN3PR0201MB1027; H:BN3PR0201MB0867.namprd02.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en;
received-spf: None (protection.outlook.com: jabil.com does not designate permitted sender hosts)
x-microsoft-antispam-message-info: g2pV0hgH6RJjtgjZW7oZdfs452RfAonDrAUrylr+GylD0XK6Z5ZHIk0O2Fv5UnXOVh/WJ92uBIoAfnQ0OWT5tg==
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: jabil.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 95f75933-e7e3-425e-75c2-08d562af34d6
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Jan 2018 22:18:18.5718 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: bc876b21-f134-4c12-a265-8ed26b7f0f3b
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR0201MB1027
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/fneaZtn7Xub6cQ1gp-Bvw9OB2Ks>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 23 Jan 2018 22:18:26 -0000

Hi Tom,

Thanks for the thought. Section 3 includes the complete tree diagram for the model. We have received different suggestions and opinions on this topic. I have read your comments on the NETMOD mailing list. Your opinion is appreciated. However, we have also received comments to request the complete, un-altered tree diagram for easy referencing. Some implementers mentioned that the tree diagram is the first section they look at, and the one that needs to be referenced often. 

I'd hope a consensus on this. For now, we'd keep it as is.

Thanks,

- Xufeng

> -----Original Message-----
> From: tom p. [mailto:daedulus@btconnect.com]
> Sent: Saturday, January 13, 2018 8:03 AM
> To: Xufeng Liu <Xufeng_Liu@jabil.com>; ietf <ietf@ietf.org>
> Cc: draft-ietf-rtgwg-yang-rip@ietf.org; rtgwg-chairs@ietf.org;
> yingzhen.qu@huawei.com; Alia Atlas <akatlas@gmail.com>
> Subject: Re: Last Call: <draft-ietf-rtgwg-yang-rip-06.txt> (A YANG Data Model
> for Routing Information Protocol (RIP)) to Proposed Standard
> 
> Xufeng
> 
> Looks good
> 
> The only outstanding thought is about the tree diagram where the netmod I-D
> says "As tree diagrams are intended to provide a simplified
>    view of a module, diagrams longer than a page should generally be
>    avoided.   "
> but, as was discussed on the netmod WG list, this can be hard to achieve.  You
> currently have four pages and the only way I can see to split this would be to
> separate the ipv4 and ipv6 sections with a brief paragraph, just a sentence,
> separating the three parts of the tree diagram, albeit with one long part and two
> short parts.  I would consider this worth the effort but leave it up to you.
> 
> If you look at the OSPF and BFD YANG tree diagrams you can see how sub-
> dividing can work.
> 
> (My own take is that too a long tree diagram reflects a too flat module structure
> and that the module structure should be changed, but this view has yet to gain
> traction!)
> 
> I take your point about the description clauses.
> 
> Tom Petch
> 
> 
> ----- Original Message -----
> From: "Xufeng Liu" <Xufeng_Liu@jabil.com>
> Sent: Friday, January 12, 2018 8:18 PM
> 
> 
> > Hi Tom,
> >
> > Thanks for your valuable comments. We have updated the document with
> https://tools.ietf.org/html/draft-ietf-rtgwg-yang-rip-08, to address these
> comments.
> >
> > Regards,
> > - Xufeng
> >
> > > -----Original Message-----
> > > From: tom p. [mailto:daedulus@btconnect.com]
> > > Sent: Wednesday, December 20, 2017 12:13 PM
> > >
> > >  I think that this I-D falls somewhat short of the standard
> necessary for
> > > advancement.
> > >
> > >  'reference' statements are almost wholy lacking from the YANG
> module and
> > > while it might be reasonable to expect the reader to know where to
> find
> > > information on RIP or RIPng, I do not think that that extends to
> other IGP or
> > > IPsec.  If you want to see how it SHOULD be done, look at
> > >         draft-ietf-netmod-rfc7277bis-01  One or more 'reference'
> > > statement per 'container' or'leaf'
> statement is  a good
> > > starting point.
> > [Xufeng] The situation is different from RFC7277, where attributes
> from different referenced documents are put together in a same container. In
> the RIP model, almost all attributes refer to the same three documents RFC2453,
> RFC2080, and RFC1724. If we add them to each container or leaf, we'd have to
> repeat these three everywhere. Therefore we put the references at the
> beginning to avoid the repetition. In case when some specific reference is
> needed, such as authentication, we add the reference to RFC8177 in that
> container. Is this ok?
> >
> > >
> > >  Talking of which,
> > >     [I-D.bjorklund-netmod-rfc7223bis]
> > >     [I-D.bjorklund-netmod-rfc7277bis]
> > >     [I-D.acee-netmod-rfc8022bis]
> > >  have all been replaced.  I am unclear whether or not this
> invalidates  the
> > > announcement, since these appeared in the announcement as downrefs.
> > [Xufeng] Updated in the new version.
> > >
> > >  Common (best) practice is to then include all the references from
> the  YANG
> > > module in a separate section immediately prior to the module itself
> so that the
> > > reader can readily find them.
> > >  Again
> > >         draft-ietf-netmod-rfc7277bis-01  Section 4 is an example of
> > > how to do this.
> > [Xufeng] We use Sec 1.3 for this purpose.
> > >
> > >  The YANG module does reference
> > >            RFC 1724
> > >  but I think that that makes it Normative not Informative, as it
> currently is.
> > [Xufeng] Changed it to normative as you suggested.
> > >
> > >  The Abstract is limp.
> > >  "This document describes a data model for the Routing Information
> > >     Protocol (RIP).  "
> > >  So what?.  This should tell me what I can do, e.g. configure,
> manage,  get
> > > statistics or what?
> > >  draft-ietf-netmod-rfc7277bis-01
> > >  gives a better example.  At this point in time, with NMDA causing
> significant
> > > changes, the Abstract would do well to mention where the I-D  stands
> with
> > > regard to this.
> > [Xufeng] Updated with more information as you suggested.
> > >
> > >  There is now an emerging RFC on tree diagrams
> > >  draft-ietf-netmod-yang-tree-diagrams-03
> > >  The authors might consider using and referencing this.
> > [Xufeng] New version references the latest draft now.
> > >
> > >  Tom Petch
> > >
> > > > ----- Original Message -----
> > > > From: "The IESG" <iesg-secretary@ietf.org>
> > > > Sent: Tuesday, November 28, 2017 4:29 PM
> > > > >
> > > > > The IESG has received a request from the Routing Area Working
> Group
> > > WG
> > > > > (rtgwg) to consider the following document: - 'A YANG Data Model
> for
> > > > Routing
> > > > > Information Protocol (RIP)'
> > > > >   <draft-ietf-rtgwg-yang-rip-06.txt> as Proposed Standard
> > > > >
> > > > > The IESG plans to make a decision in the next few weeks, and
> > > solicits
> > > > final
> > > > > comments on this action. Please send substantive comments to the
> > > > > ietf@ietf.org mailing lists by 2017-12-12. Exceptionally,
> comments
> > > may
> > > > be
> > > > > sent to iesg@ietf.org instead. In either case, please retain the
> > > > beginning of
> > > > > the Subject line to allow automated sorting.
> > > > >
> > > > > Abstract
> > > > >
> > > > >
> > > > >    This document describes a data model for the Routing
> Information
> > > > >    Protocol (RIP).  Both RIP version 2 and RIPng are covered.
> > > > >
> > > >
> >
> >