Routing Directorate review of draft-ietf-rtgwg-ni-model-02

"John G. Scudder" <jgs@juniper.net> Fri, 28 April 2017 20:26 UTC

Return-Path: <jgs@juniper.net>
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 0D89B129535; Fri, 28 Apr 2017 13:26:37 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.022
X-Spam-Level:
X-Spam-Status: No, score=-2.022 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=juniper.net
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 TenPJjYE7TZZ; Fri, 28 Apr 2017 13:26:32 -0700 (PDT)
Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0100.outbound.protection.outlook.com [104.47.42.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8CA2E129A9D; Fri, 28 Apr 2017 13:24:03 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=juniper.net; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=OE1rNn1zEIhqX0O4QYUIDaPIvu9Bh9Z+mhKHkvE/rX0=; b=bXlTalcZkuk+8ppJ25WrXeOsijEsOuowGy/AB2J1hO3NSkGZmD9oR+MQdUKhCTn2BWoS52ishV5VyPVfuZsiinjKIO3Bqnzr6MZKNE6wQrPdd19RCYNXUKEYRbsVrm8O62WW4nPG8DD9BeNMkEZ0GSDIXpuboG+vUOzrBJESWaI=
Authentication-Results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=juniper.net;
Received: from pvakharwala-sslvpn-nc.jnpr.net (66.129.241.11) by BN3PR05MB2497.namprd05.prod.outlook.com (10.167.3.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1075.1; Fri, 28 Apr 2017 20:24:01 +0000
From: "John G. Scudder" <jgs@juniper.net>
Content-Type: multipart/mixed; boundary="Apple-Mail=_E8A9E5BB-0E2D-42E0-BE69-AE11A09B0D3B"
Subject: Routing Directorate review of draft-ietf-rtgwg-ni-model-02
Date: Fri, 28 Apr 2017 16:23:57 -0400
Message-ID: <B6095AC5-A3FD-450A-BBD3-FB88FD04800F@juniper.net>
CC: rtgwg@ietf.org, rtg-dir@ietf.org, rtgwg-chairs@ietf.org
To: draft-ietf-rtgwg-ni-model@ietf.org
MIME-Version: 1.0 (Mac OS X Mail 9.3 \(3124\))
X-Mailer: Apple Mail (2.3124)
X-Originating-IP: [66.129.241.11]
X-ClientProxiedBy: BN6PR13CA0041.namprd13.prod.outlook.com (10.171.172.27) To BN3PR05MB2497.namprd05.prod.outlook.com (10.167.3.26)
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: fbdf7874-23b5-4514-73ec-08d48e7482ab
X-MS-Office365-Filtering-HT: Tenant
X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(48565401081)(201703131423075)(201703031133081); SRVR:BN3PR05MB2497;
X-Microsoft-Exchange-Diagnostics: 1; BN3PR05MB2497; 3:6wHCXzN8zgMd1JRplCwcVVcrlN2ljRtZBrkdYs3UfG3tYGUv04kGjjt96x7KJKa3a23PPs6AfZL8CmyU0Cs5VOOuYILr8huV6TABeveR/jWqVdT8WsRKgA4WX8e2Bs4cvDWWpZy7yj+UiugQIZUJRDvk2bBZ4le593JBgOU+3pT8RniJZkrqipnIvGtxWECUy1vLwdlR8tjxMqSJu3JVg9rQb5xud7WpMtD2WHN1aCK3h//nSjv1xbutaNyiNYG5AiyswAMND5jBAloQNZsNU6JIlvoaMovM/uoiW/oc2mO9elhp2UWZIYtNiC9urV05QN/kELzDnNhWBTMuwqJMn3YynJae3rG/1c21RnlCEsU=; 25:d+DbAmaaChHNGGPtoxly4NdGTyOl1zqSTI8xPztEFZBZZLGg6AktirKjypP9aM7/c3sUc635BFcuso4DFrKHEyCi3ALoPkomnoi4++Q5BAhctuRvD6iJZ1ebyFL+px5tVwRyHYmtbbpJXqfPAlUeEHZNOEmA64elFwB4TqSVe48fvzTs7U51hC2vCJPfdW7iI9q1VGRuvqxhQrTs6KWY/n/VXNWLmSkWOFih6IfVeDmoL4aoQL6JmTg4jA9vWKaeDK0gtiGNTtu5TV/nXiAqdq6XsL1Gq24r8jJrgrzg8OsVQDT86m5m1p7F2IOtZt5/0ejaM48E/NFXmxD32hEEG88lTPtmxJNnnZub6+3vx9KDUDIMW6BaHDmMIDNLsEJ9flNfTN1VEGDQrmuJV2oVbdQxYOrGFXsZhYMLZ9dvIXhU/ltai69d8AENpFgmy8nacoz+NsN6EJ39nkz4Om6M5aMxNfX0qySjh1nGdrC+ik8giRI44//fbu86gLi82597hJxfiWx6zcsqx3fF1D3WIg==
X-Microsoft-Exchange-Diagnostics: 1; BN3PR05MB2497; 31:rkESdtDaXki/nBdsbAy/ObbrDZZuXwLdIKBYuXhmgG/IizKRODIvAXYhGHzkgSSDsRdthcAz6pci9vmHKqtME+wOoWvSrUQTTpr71VCchg9u37W+Irm61rsO95KMf6AQsJ4ya7Gw4BtAfMezQxrTp5RLs20ZqR4KSHgahEm5OxDKEKk4FYvS9dqAlFfAHp+cKjyf72dc1latI/jSbVJPRS6fqS10Uqu1fUfFLZqdAUJ0U2L7TQYcIrkroMvXpQ/R79CfPJ0h7vU/LoaYKAvKLL48e6TNMBGrXJGNRWAIpLOIQtxbp2W7CeL/bQi1O88uj8g80mdWTMBkxzvwRQXyfA==
X-Microsoft-Exchange-Diagnostics: 1; BN3PR05MB2497; 20:Xd4SQBlVTb/jJr52SG2WKBbP49+CBbVMgT1ipsoLqW9Jh4sx3+WmpeDmBIC9qKspwhUgW+Q57/4VXzqLWFyI4GU0cWVSpJFct55+MwRsIfPx9xMK8Q271uqrFxZ+ho6OQtEWmh5wyN6BoedMqKP8bCvxGrn3GtIQpZYNlKMXIvfWGqaemrpOtB1vz08Ys9SxgJ9w5yUk0nAlam8cuWk6wiuXQCCz0davKxIDVonGD69R1RW7R5AXao1Qdmsd4IXWR5Ufezs32eb8Vb344yiQtZbpppS3JuvevvnkFhMRIVORCu/t9LbNmlZztlKXDb7ySzO+K6PB0CoBWYKNCGLqrejziY0cgj/X1TGUQKhbwzJTshzUQfbCD+oE9zAQOjrFcYlh8dmTnH3wGn5cujdOvO8vRfwO8qgp3VOYwQqW5UxpLbHOVNoE+UDe7f79+hOxT+XtFfOH6LsUae4a8RMeW7wNm7sJ4gaFhzlYYsDJTF9o3GSuXolsQX1mPEJ6lPBVbZNZ+V05s9TDNJlkyQqELaaf9lWBLi1MWlgVKgyVWDx06ORy4T8X3nd7MYsKW2QnPBiHWE05IVoroFIQ7jGe6tfpcsKLg/ttIM74XWBKCbc=
X-Microsoft-Antispam-PRVS: <BN3PR05MB24975CD22F25B1E3DA54E8DCAA130@BN3PR05MB2497.namprd05.prod.outlook.com>
X-Exchange-Antispam-Report-Test: UriScan:(120809045254105);
X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(102415395)(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(3002001)(10201501046)(6055026)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123564025)(20161123560025)(20161123558100)(20161123555025)(6072148); SRVR:BN3PR05MB2497; BCL:0; PCL:0; RULEID:; SRVR:BN3PR05MB2497;
X-Microsoft-Exchange-Diagnostics: 1; BN3PR05MB2497; 4:D7FDyDUGlKo4AhqgcJ1ZGaA2IMSsvaoX/3knDPcVFoa9WxEjg6+SOBjBSMfYfD7Oe5jjlziw9IqtNpWzuFw2XcrPx7ohkgaSkouG8cVPH4fy5wGAIiLDCQqvWWlVmla4ebKlPAOtkmhTzETjmElWW738nZOG0VkvOv4KLTNt0uBgVQ+VHrqRhE8Ha0VBLL3fQ4smV44V9IfNdpRM/ztmhaxkRkdMCMmJnM+avKz47NJiDa9gbG4DiRUl2wlga1TjzRs+hw4DpB3jZlSGeuEt6/TAbGpk81D9jCiN4n7pCQhF09yTLWLzSTEH+HCyj+RuVNabqedtVSwgnM16BUf2gn8lB8V3JSQ30106XkA5P7y8furaYRta84j8jPcATAFMz6c1Hk+vjBsiQF1JRa72g93UslOGsaXPAV5Jg2UinH0NKtYkp985hrC31QVtSuYqWXYugSxMqu7JX5eFQI1qSa8CNgEBcLWE+BrDBVG06uSbDotdPKU9GLlEsABrCbAN5TnMcU0cloBAqEyyGavV7N3QnSZfYUrERJ1EsGk8JkqVl/SC6/G2ACVQHQxhqg/SlFGY5TT8NszELVzISno0VMvsP4kSu808fdL1T/teF2k7svMlQlEgTpGk31dbpMk6B7Ylko39teH8ButLGAfV6vEUEL7nfqqxEqqCm8Mk2zOY9m0yWGugYFEwy8jdoK7BIAHQAPHm0lwqOpwoK8wD4xyeA2p5acYXWGvRVdySVipTMDrZaWLYdmphQmXHVmbVvzCaUOmffb2RJNkIDcG/TULVX/u8YbhYzCeQjgMA3RGiiT3T9ZMRy8aFyaxiMh7D9Qz8YtVZJATM8UeeluJhkiTxvQVzuP5Z8DW3f36gXcD/HHviE34gLDhBAHBe4sOj
X-Forefront-PRVS: 029174C036
X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10019020)(4630300001)(6009001)(39840400002)(39850400002)(39450400003)(39410400002)(39400400002)(39860400002)(53754006)(305945005)(86362001)(50986999)(4610100001)(7736002)(512874002)(5660300001)(69556001)(2351001)(6512007)(189998001)(6916009)(21480400002)(6486002)(6666003)(568964002)(82746002)(54906002)(6306002)(6506006)(1720100001)(84326002)(83716003)(450100002)(4326008)(33656002)(6116002)(25786009)(3846002)(36756003)(5890100001)(81166006)(66066001)(53936002)(8676002)(53416004)(50226002)(38730400002)(2906002)(42186005)(42262002); DIR:OUT; SFP:1102; SCL:1; SRVR:BN3PR05MB2497; H:pvakharwala-sslvpn-nc.jnpr.net; FPR:; SPF:None; MLV:sfv; LANG:en;
X-Microsoft-Exchange-Diagnostics: 1; BN3PR05MB2497; 23:jPZaZVXAkycox8oH2ZjPABEiCGCRV2lDIEanz5dPoG7InMzZFX1S15VSO66UVrikD0YyXpnn+GiDqJ/sN/uywyrFEGe9K/WFwCsd4mhc9rnzq0fKv2AI0jbzUi207AWTluwmapWCksIr9ZtCqtMYrL3ifzZXyLhsLLsl4IzabXTQAHfn0h1UbG3RQjotFQA+LpWxfZPtJQe/nF0vjxWZfXYBFFTjrNTX+TxhfDL8PIyz4bDHCYJj0JwgrLMkHZDSNxADsuYL45keW3Nr14peD760JVo0OwPVBU9HjhynUPJDRj2UCysSJOrCoWM0MWwtKlsUcRqLVQlNiNz8D16s0XZ4UTyiRTkuCnAhPKtVL4TCpfo6i58nvGyeCQeWG7V9iVD+O6HT2Zin73mI8hhckk+E4ur2ZYkzb8sduIuNdf7HHs90jCOoGb+rVYcNGn4O4xw9qCedbMNmopmt/pzfe/x+ci1zl/YlHAa4uG3Q64yUg33cev8Eup/MS7Sz6VQRjgLq2plr3xJfgAqSDyVa1VDtBLlj6CoYIam85MmDrcXH8cltwhftW+NyceUO0StOfNd1MC1Y5eMpDc4DGb4R0SqzFVA1p/ityDB22KsFnOCRVryHnfwH3huEnZfjMPp/y1EeDXgRws4zBo3tW0Pwwh1NAX/rRhCX5cei0on9kUqwQCpRTUXNWeztK5/D007JSdvhJCeoyJsrGuEye9hOekByNPgrujZhj8getgkpY/FoKzQc6Z3FrLCJissXfZbRjSLuGl/EcKoHw5UpvQeLc3LyoZGhtkc894yz2agFQRGQzbkG7FRv5KaqTZqfNMNA8ySTWye6HEBWcnP/AjalYa8WaglRTfUZyk3oOpCKSN5P2ZLSAaka18Q8aSuk91So0jn4eHbgJr12FqMDb5xhl4KK/6Xru2+V19C/7U4+9SJIs6idQpkzf8hliy02iWRSEoUHJOYFd+DBIcI1yNNZW4kDKOvx1L5UcThAt9y6cUuKyiaFniHVpMQFSuFLYIcm9byIK6sDGhEdLml1OH1qjjFmdcoXel5RZQK2G4OWvaDtQfy2UNDTSa2EHK5yo4zgATrR1I7k+9ANvb0OprcFlGrdIIsnvd/EMysA+EendGfOdNYsHnxR/1bf+3X96TH9AKUqgv4OmCXZ5rR12vEAoO+z0t417EC+IxCawv8apPHzI9E3eTLhsRTDvN9J8YPNX7ATo54biO5oYMmtCvk1gtOCcfguPVqrvChydopKnrY=
X-Microsoft-Exchange-Diagnostics: 1; BN3PR05MB2497; 6:EZqVowQncv/t6uhZkRMB5G7oYnCVPk5MlKDBDxm1TWa4YvoC5VvTZdOufx2SRIulBhgGczJxzNnUdmH/TTxPY+JGoMpT91hUBBdYmyX/kSZ7vtEkNB+g94JYvi1aSu+dvcX9m08E3FqvTD7Kn2GZmL49tnhKY6pYFyi6+vhPwGMNu6sbOuVEuN3yk8EuwF5IEq80l2goHDjWjUc+5PrEXzCFJuCpm1mYCIAyvBOVHGHINnketWlueTxj3b6og+46LYCnqMZLPqQ1BUqz2hA/OH8CnsGvfG/d5tQoqutAWfNWkqO/ySb1kTra8SbJOFO7TrDEG4IWgLd3rGPDkPxEPSYTGIYdMXkWMYHnYgKSj6zkiWnowHcozpjP+Paihcr5J9gqV2sBpZ4F2eAEurM9/hYrDLQxqTiPjCfcLe2/hkgoUPDCAbLl57vR9RwcDi6jSSKCbYleQtAXRycilOJEDiLoSSTRkkSzRjwhXhUKUcABKEZ78cjx7Tt+SSacNTh68kLt3rkel/SAZ4oCLsAkXsQisvJ03YfafF8e9F1ihqo=; 5:D+nr+x/Z2bDkEXSSu7Q/ke5mgAoPhjYw/vWg7lmCxJuEmV6USXwRo/Sq1I4AY82Wp03kU3jYJVKK5l1D1GiF9kDlXoRA72Xqfmi7QrZavIxZ3oJx0WOVFdlcoepDedmdF2ADpR9vsVdvRJAhcam29Zkd0BlF35gUAYRARgATQ2g=; 24:yRa7yg/l3By36udBfEXiF3lyg/vJIeXuGkNCfevFNHmz1p4DfI3vPkhJGX/rHku7UaUnG+uQ2iJssUnHcQSe+y9Gwctm22czH3M9xm2DXTo=
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-Microsoft-Exchange-Diagnostics: 1; BN3PR05MB2497; 7:oK2gwTvS9ivKjPlc0vluqO7hxyy+hPf124VeN4xkrFhqri98z3VwrdoygIIo9+VibSzK+PyQ59OKRHLo8OLgXsLbuWsQiT6oIGml0z9d0cjKsxU18xHZvHQxkc2+PmEDfcxYp16DJSJuxCrI4aQQc2d/r7AAEfzYbQgOVvEMs76jKdlZexP2mcBZ8VZRt8LKSsKKDiGQ8xJaQGixzKR3lipnr0AjPTGU/5LbkvaVvY5rXRAbgxlqNTtgaypAQhSIzCNLjQoIF0pvBuDFDzskk9AiK1dycrVK70HXAB5X3hasThYwQUdUzInQQlCbIDLS6hDEWk6BoPN7/OiYDq694Q==
X-OriginatorOrg: juniper.net
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Apr 2017 20:24:01.8101 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR05MB2497
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtgwg/UIZung-cmJOgbBuRZKVPDkCebmc>
X-BeenThere: rtgwg@ietf.org
X-Mailman-Version: 2.1.22
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, 28 Apr 2017 20:26:37 -0000

Hi All,
 
I have been selected to do a routing directorate aQA review of this draft.
https://datatracker.ietf.org/doc/draft-ietf-rtgwg-ni-model/
 
The QA review is described (https://trac.ietf.org/trac/rtg/wiki/RtgDirDocQa) as:

"The WG Draft Quality Assurance process exists to provide cross-WG and expert review early in the IETF process after a WG has adopted a WG draft or while the WG is deciding to adopt a draft. Since a WG adopts a draft as a good starting point for the work, providing early excellent review of such drafts allows for good technical discussion and the ability to enhance the WG draft to solve identified issues. The earlier in the process that substantial issues (technical or editorial) are resolved, the more quickly and smoothly a WG draft is likely to proceed."
 
For more information about the Routing Directorate, please see ​http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Document: draft-ietf-rtgwg-ni-model-02.txt 
Reviewer: John Scudder
Review Date: April 28, 2017

Summary: 

I found the document easy to follow, clear and almost painless to read, thank you.

Note that my level of Yang expertise is pretty low, so you should be looking elsewhere for a critique of anything other than the grossest Yang-specific aspects of the document. (I found the schema-mount example in section 3.2 particularly opaque.)


Comments and Questions:

1. There's a significant amount of duplication of explanatory and background text between this document and draft-ietf-rtgwg-lne-model-01. In reading it, I tried to consider whether it would be OK to cut out some of the text explaining what an LNE is, but on the whole I think it's better left in -- the document would have been more difficult to read without having that context in-line. However, it does lead to the question, is there some good reason the two documents are separate, instead of a single document? The duplication between them suggests there's at least some motivation to refactor them into one. (I realize there may be many reasons to keep them separate, including "seriously, John? The cost/benefit just isn't there", but I had to ask.)

2. The abstract is almost a copy of the abstract for draft-ietf-rtgwg-lne-model-01. Comments in #1 above notwithstanding, I do think brevity is the soul of wit when it comes to an abstract, so I suggest removing the LNE definition here, and just define the stuff *this* doc is about. (Similar suggestion applies to the companion doc's abstract.)

3. It seems to me the "TBD" for network-instance-policy represents a significant open issue and deserves to be included in your open issues list (section 1.1). Other TBDs sprinkled throughout don't seem to rise to this level, but of course do represent open issues.

4. Speaking of network instance policy, although since it's left TBD there's not much to be said, the examples you give (RTs, RDs, VNIs, VPLS neighbors) mostly don't seem like what I think of as "policy". I suppose it's one of the most overloaded terms in our industry, so maybe someone else does think of it that way, but this choice of terminology was a speed bump for my understanding of the doc.

5. The example given in section 3.2 doesn't seem to follow the same pattern as the one given in section 3. I'm too much of a Yang neophyte to know if there might be some Yang feature in play that makes this make sense, but on the face of it the example in section 3 seems to tell me I'm supposed to bind a network-instance-name to a specific instance of an interface (so, if:interfaces/if:interface), whereas what I see in 3.2 is a network-instance-name at the if:interfaces level -- which doesn't make a lot of intuitive sense, either.


Minor Issues and Nits:

6. I've edited various minor suggestions into a copy of draft-ietf-rtgwg-ni-model-02.txt and attached it, you can look at them using your diff tool of choice.

7. This sentence isn't quite right:

   Network instance policies are used to control how NI information is
   represented at the device level, VRF routing policies, and VRF/VSI
   identifiers.

Without knowing your intent I'm not able to offer you a rewrite. Possibly it would be enough to reorder the items in your list, to put the big compound one at the end, as in "Network instance policies are used to control VRF routing policies, VRF/VSI identifiers, and how NI information is represented at the device level"? 

8. This sentence no verb:

                      For layer 3,
                      this consistent with the routing-instance
                      definition in ietf-routing

Again, without knowing your intent I can't offer a rewrite. Maybe the verb "to be" is what you want?