Re: [i2rs] OPS-DIR review of draft-ietf-i2rs-yang-l3-topology-10
Hariharan Ananthakrishnan <hari@packetdesign.com> Wed, 20 September 2017 17:41 UTC
Return-Path: <hari@packetdesign.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 6D3FF13320B; Wed, 20 Sep 2017 10:41:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.921
X-Spam-Level:
X-Spam-Status: No, score=-1.921 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_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=packetdesign.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 MF06n5iCiAWQ; Wed, 20 Sep 2017 10:41:35 -0700 (PDT)
Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0088.outbound.protection.outlook.com [104.47.42.88]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 564D0133063; Wed, 20 Sep 2017 10:41:32 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=packetdesign.onmicrosoft.com; s=selector1-packetdesign-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=XVAHrpYo6iGz0jYNJ9molZtpV7MGbMnALdXMF36te8I=; b=a3lahUYsrJ2+Bk2GUHZWkUi+lX/giM9iOYC8wn6PvqOuNIVF5JVwInd8r+tYUNSJ5NHiNMhwAbeJywuEkOenaHQmBhelPTJfvKivF6yxwyCJ8yasiu06d3xHpojmpFZ/zGiluId5kUVCekEQzQQ4zjhPes9wDXwaOhqT27MoTu4=
Received: from CY1PR0401MB0921.namprd04.prod.outlook.com (10.160.160.145) by CY1PR0401MB1420.namprd04.prod.outlook.com (10.161.213.142) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Wed, 20 Sep 2017 17:41:30 +0000
Received: from CY1PR0401MB0921.namprd04.prod.outlook.com ([10.160.160.145]) by CY1PR0401MB0921.namprd04.prod.outlook.com ([10.160.160.145]) with mapi id 15.20.0056.018; Wed, 20 Sep 2017 17:41:30 +0000
From: Hariharan Ananthakrishnan <hari@packetdesign.com>
To: Ignas Bagdonas <ibagdona.ietf@gmail.com>, "draft-ietf-i2rs-yang-l3-topology.all@ietf.org" <draft-ietf-i2rs-yang-l3-topology.all@ietf.org>
CC: "i2rs@ietf.org" <i2rs@ietf.org>, "ops-dir@ietf.org" <ops-dir@ietf.org>
Thread-Topic: OPS-DIR review of draft-ietf-i2rs-yang-l3-topology-10
Thread-Index: AQHTBiHJsMDl1I9o20OVuhwPobft4aK+Yx2A
Date: Wed, 20 Sep 2017 17:41:30 +0000
Message-ID: <55D5C108-FBC2-46EF-B90F-046C41958F43@packetdesign.com>
References: <f63f0a34-302b-5f5b-85f4-58d9c7692253@gmail.com>
In-Reply-To: <f63f0a34-302b-5f5b-85f4-58d9c7692253@gmail.com>
Accept-Language: en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=hari@packetdesign.com;
x-originating-ip: [38.99.127.66]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; CY1PR0401MB1420; 6:QAX4wNjY0/E4S+g2dveLhAiNlYUaL1tqjxCKKCWxDNknXtB6e81F6I72D42NiKJxXqbUce9X+d4wn3lH84H7RaELSCiZqheQmw51zYq/I3XmzKdP0Gc/nFZa+Z1T+BuNfaIrwpE92D8UI3haJp1gxlaXGxyXN4rGUZiaM1Mh+lFqMbvEv5xt718rD1hFub2ak+Tr3HWymwqXNLcLbMJBsTD2IQVDV23OYJOBztTpI/zL5eKTuh29ExhJUcm8PsCdBlLoorJm3Gs5eB7lYpcYl9/zY4zNBU8BhSpxCsffuTNVbMrrxkqzXiSa/kjWMG14m2z3imSoChuoYw10HanMpQ==; 5:A8RtqHd8+ygBeTSgvsCbMmHKnqz2Wf3EAcP+cFDDeM5SV2yuybbRcwj4okCzBUXXHSWmYlqfS95I5N7DHhMxRfiltqPZcwq9cOijq6lVkibvOFlLrdZbWiGYcbOO/9j+4wsCfPnWzzurvO0Y9TyP1w==; 24:ZljdpqpEqqymFM9Cij8+M77pvIGwWbueExSVSvzCDl463C4JxgY1t5FR5aMUBhc6BAGUsHXvCQrOU3YLBgSPNoO6cTat3LocXwmCUfnmUI4=; 7:EDJ5u19xGdUR6BTdavsekupfaePkDY+nAXB7f0MtdP5e7A9s6/Bu8VXlWKhl6zjxIxHkmybC6tnHMDTxLKPy4SQ9YtMv/boSPBDqtBywZiM92id4VPc7RAtXk6ahLR+L5AHlGqBenSTIQWwpuKetHGfeJJ74Fyv2eFXFmOFs6gi3rvC8w60VhnKtuGkdqXhCvGQfnGDJ0thdGLxa0lLx9ShmFWlqCiIkQWT52iIW+PE=
x-ms-exchange-antispam-srfa-diagnostics: SSOS;
x-ms-office365-filtering-correlation-id: 9dfb4c66-cb25-4cf6-352a-08d5004ed3d7
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(2017082002075)(300000503095)(300135400095)(2017052603199)(201703131423075)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:CY1PR0401MB1420;
x-ms-traffictypediagnostic: CY1PR0401MB1420:
x-exchange-antispam-report-test: UriScan:;
x-microsoft-antispam-prvs: <CY1PR0401MB1420C81731F93336A81255E2BD610@CY1PR0401MB1420.namprd04.prod.outlook.com>
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(3002001)(93006095)(93001095)(100000703101)(100105400095)(10201501046)(6041248)(20161123558100)(2016111802025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123560025)(20161123555025)(6072148)(6043046)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CY1PR0401MB1420; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CY1PR0401MB1420;
x-forefront-prvs: 04362AC73B
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(6009001)(376002)(346002)(51914003)(199003)(189002)(24454002)(6436002)(36756003)(6506006)(229853002)(76176999)(50986999)(54356999)(106356001)(86362001)(101416001)(2900100001)(6486002)(77096006)(54906003)(6116002)(105586002)(3846002)(102836003)(53936002)(8936002)(97736004)(82746002)(14454004)(4326008)(316002)(39060400002)(6246003)(2501003)(2906002)(478600001)(8676002)(81156014)(81166006)(7736002)(189998001)(6512007)(66066001)(25786009)(5660300001)(2950100002)(305945005)(33656002)(99286003)(68736007)(3280700002)(3660700001)(230783001)(53546010)(83716003)(110136005); DIR:OUT; SFP:1101; SCL:1; SRVR:CY1PR0401MB1420; H:CY1PR0401MB0921.namprd04.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en;
received-spf: None (protection.outlook.com: packetdesign.com does not designate permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-ID: <6278EE3EFF46D945935D01922E1F8989@namprd04.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: packetdesign.com
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Sep 2017 17:41:30.1213 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 1df5b6db-3686-4e87-9323-024e5bfe00f0
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR0401MB1420
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2rs/s4EmdaM2gVlzOOJf_nGtSmHdECk>
Subject: Re: [i2rs] OPS-DIR review of draft-ietf-i2rs-yang-l3-topology-10
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: Wed, 20 Sep 2017 17:41:39 -0000
Hi Ignas, Thanks for the review. We have updated the draft addressing your review comments. The multi-topology references have been removed and example OSPF topology has been moved to appendix. We also removed the ISIS example. The intended use of this model is to provide a L3 Unicast topology model. The topology related use cases are defined in section 6 of draft-i2rs-usecase-reqs-summary. Thanks, Hari, Alex, Jan, Xufeng, Nitin, Robert - Hari On 26/07/2017, 08:13, "Ignas Bagdonas" <ibagdona.ietf@gmail.com> wrote: Hi there, I have been selected as OPS-DIR reviewer for this document. Document: draft-ietf-i2rs-yang-l3-topology-10 Reviewer: Ignas Bagdonas Review result: Has issues. Major question that I get after reading this document is on how the proposed model would be used and provide practically useful topology abstraction interface, especially in the context of multitopology routing. The document covers multitopology as augmentation to L3 unicast topology model, and this exposes per-protocol MT specifics, which seems to fall exactly opposite to the intention of the document to provide abstracted topology view. MT topology view is a subset of protocol specific topology plus it can contain routes coming from other sources too, and the model approach specified in this document does not seem to allow for expressing such constructs. One practical use case is multicast RPF lookup control, typically that is an IGP MT view plus local override routes, typically statically injected. For this use case the model described would require the topology to be strictly coming from a single IGP source, either IS-IS or OSPF, with no ability to intermix both, and with no ability to represent routes external to IS-IS or OSPF topology sources. Having separate models for IS-IS and OPSF for augmenting the base L3 topology model does not allow for hiding the differences in representation of IS-IS and OSPF derived topology aspects, in particular the MT identifiers. For the user this would require to know which one of the IGPs is used instead of getting just the specific topology view regardless of what are the underlying topology sources. What is the intended use of this model then - is it on providing interface to protocol specific topology (which then raises a question on how it correlates to protocol specific models' operational part, and section 1 second paragraph mentions precisely that), or is it on providing routing information source independent interface to topology? As a summary, the document would benefit from operational considerations section that would explain the intended use of this model and how it correlates and interworks with IGP specific models. Without a clear answer where the model is positioned it is difficult to provide more detailed review on the model structure itself. Metrics being a single uint32 value - that may not be flexible enough to represent realistic scenarios where a topology instance may have different routing information sources with incompatible or incomparable metrics. At least two independent metric components would be needed for that. Nits: Title page has 6 authors listed. Are Xufeng's contact details correct? s/Netconf/NETCONF s/Parantheses/Parentheses NET-id, NET Id should all be NET ISIS model has a reference to OSPF MT RFC. ISIS model for multi-topology-id has max-elements "128" limit. MT TLV can be repeated and thus a larger number of topology ids can be signalled. s/paper/document Overall the document would benefit from grammar check. Abstract, Introduction, and Overview sections contain quite much of repetitive text. Thank you. Ignas
- [i2rs] OPS-DIR review of draft-ietf-i2rs-yang-l3-… Ignas Bagdonas
- Re: [i2rs] [OPS-DIR] OPS-DIR review of draft-ietf… Susan Hares
- Re: [i2rs] OPS-DIR review of draft-ietf-i2rs-yang… Hariharan Ananthakrishnan