Re: [mpls] Genart last call review of draft-ietf-mpls-ldp-yang-06

"Kamran Raza (skraza)" <skraza@cisco.com> Fri, 20 March 2020 20:32 UTC

Return-Path: <skraza@cisco.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6D13E3A0DFC; Fri, 20 Mar 2020 13:32:38 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.6
X-Spam-Level:
X-Spam-Status: No, score=-9.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=C8+3scRP; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=u0Qh8nCP
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 fqfo8lI5qviB; Fri, 20 Mar 2020 13:32:35 -0700 (PDT)
Received: from alln-iport-8.cisco.com (alln-iport-8.cisco.com [173.37.142.95]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D56753A09C6; Fri, 20 Mar 2020 13:32:34 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=5912; q=dns/txt; s=iport; t=1584736355; x=1585945955; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=Y+aUwS6sg5eplkXBUTY28dtMWtU8OCNqW4Gfa8bQra8=; b=C8+3scRPvUVd5jyZSD7OdSEfj0t1GqTs0Y+DFdDAKRn8GpCEQiZi069q +ErgtEKbR4neuYsaWPAU/b7jIdJtBI2Ret1LaZWHlkbgE9QsSj6PMRW3J qddbddOqBB0CK8Ba4STF06G5MeWzmyxL+ucScPdHFrjyCecVVHYj6LxdS w=;
IronPort-PHdr: =?us-ascii?q?9a23=3AbxoU4BG9N82yluqVbgDz4Z1GYnJ96bzpIg4Y7I?= =?us-ascii?q?YmgLtSc6Oluo7vJ1Hb+e4w3Q3SRYuO7fVChqKWqK3mVWEaqbe5+HEZON0pNV?= =?us-ascii?q?cejNkO2QkpAcqLE0r+eeXpdSUoFexJVURu+DewNk0GUMs=3D?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0CcBQCYJ3Ve/4ENJK1mHAEBAQEBBwE?= =?us-ascii?q?BEQEEBAEBgXuBVFAFbFggBAsqhBiDRQOKb4JfmByBQoEQA1QJAQEBDAEBIwo?= =?us-ascii?q?CBAEBhEQCF4INJDgTAgMBAQsBAQUBAQECAQUEbYVWDIVkAgEDEhEEDQwBATc?= =?us-ascii?q?BDwIBCBQGAiYCAgIwFRACBAENBSKDBAGCSwMuAQ6heAKBOYhidX8zgn8BAQW?= =?us-ascii?q?FFhiCDAMGgQ4qjC8aggCBEScggk0+gmQCgTABEgEhF4J6MoIsjikCgg07n1g?= =?us-ascii?q?KgjyHWIpuhDgdgkuIKpBgjwqJBZJhAgQCBAUCDgEBBYFpImdxcBU7KgGCQVA?= =?us-ascii?q?YDY4dg3OFFIVBdIEpjWkBAQ?=
X-IronPort-AV: E=Sophos;i="5.72,285,1580774400"; d="scan'208";a="463098467"
Received: from alln-core-9.cisco.com ([173.36.13.129]) by alln-iport-8.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 20 Mar 2020 20:32:33 +0000
Received: from XCH-ALN-002.cisco.com (xch-aln-002.cisco.com [173.36.7.12]) by alln-core-9.cisco.com (8.15.2/8.15.2) with ESMTPS id 02KKWWfJ031508 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 20 Mar 2020 20:32:33 GMT
Received: from xhs-rtp-002.cisco.com (64.101.210.229) by XCH-ALN-002.cisco.com (173.36.7.12) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 20 Mar 2020 15:32:32 -0500
Received: from xhs-aln-001.cisco.com (173.37.135.118) by xhs-rtp-002.cisco.com (64.101.210.229) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 20 Mar 2020 16:32:31 -0400
Received: from NAM04-SN1-obe.outbound.protection.outlook.com (173.37.151.57) by xhs-aln-001.cisco.com (173.37.135.118) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Fri, 20 Mar 2020 15:32:30 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; =?utf-8?q?b=3DWDY7BUjBFamOZ2ItrCcHwSTpTFJfCg2QO3iZ2qmC3YlznqbawRkl2sj4Jk+rz?= =?utf-8?q?o/+vqDkhTAXF3Gap4ELpb0qMGrgPIl6/xZFS1rHDLxQPxknnXd6UMQsRaLSBV6wQB?= =?utf-8?q?/WVFmvcgfi2czkhZDSnSHlaInDG2qDafDVpdo6FnPqHgP64j9D91R4nVs47z+sL5o?= =?utf-8?q?2EYZCFhwYWl63KjP8e7UGajk6s55QqYr7r94spcCTUnNaKPuD9e0D+Lo4B1EAEXMP?= =?utf-8?q?3d2qW/5lVzfsZMDeaRUxQrsPEW9H73vEqWltlbSAaHfwzx1NN5RK13DPI7Mb9Zw/K?= =?utf-8?q?QTJC0LkysAxXJsfxb9FiQ=3D=3D?=
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; =?utf-8?q?h=3DFrom=3ADate=3ASubject=3AMessage-ID=3ACont?= =?utf-8?q?ent-Type=3AMIME-Version=3AX-MS-Exchange-SenderADCheck=3B?= =?utf-8?q?bh=3DY+aUwS6sg5eplkXBUTY28dtMWtU8OCNqW4Gfa8bQra8=3D=3B_b=3DKGQ0Kx?= =?utf-8?q?3xKEF8DjctzWbr6lWt0Lgl1gD/3Hk2NxoXVFkLMIJ/usp+FZ1b4kqXQLHxdMhFjyh?= =?utf-8?q?TY7o2KE8wr/qseUZK1N84Uqh9NjbXX09LXjpd2yzxSlatcz2mlGZhzNGqZMO7NYzI?= =?utf-8?q?xZWAvXQjgWmHk6yumyZHBtzUVVxWFxL+Nq1REOj71tuBRz876fDsaapO5716kj5DU?= =?utf-8?q?+gvXTQHirf6bxKPjeQF2pKF+ZQlGyxBiiPAuvQPuT9/iJ2jNIQIz6uA2h60o6PIvB?= =?utf-8?q?Yeb9pjOxKKwVRjJDnlausPGctXlObvPEL0vl5hft4uIu6gih6Uytp6PZD6RtkhvyX?= =?utf-8?q?eEn1OqhqgCA=3D=3D?=
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; =?utf-8?q?h=3DFrom=3ADate=3ASubject=3AM?= =?utf-8?q?essage-ID=3AContent-Type=3AMIME-Version=3AX-MS-Exchange-SenderADC?= =?utf-8?q?heck=3B_bh=3DY+aUwS6sg5eplkXBUTY28dtMWtU8OCNqW4Gfa8bQra8=3D=3B_b?= =?utf-8?q?=3Du0Qh8nCPrt0mEhJzy31qbv7FZ9U/ZBc2Wd4OGLnAk/aginD5ohvL0X6icYsvg5?= =?utf-8?q?GsNdw0D5myxlzeSjK8tJXo9lF5wmIMbk6rxAICwPiJ84auPXkbPe51Yc3fFxHX10g?= =?utf-8?q?MG7h+HRhFb2Rxh3FnA/KaSLdI1axpgyUKXxM8cjoioaM=3D?=
Received: from BL0PR11MB3412.namprd11.prod.outlook.com (2603:10b6:208:7c::32) by BL0PR11MB3267.namprd11.prod.outlook.com (2603:10b6:208:6b::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2814.22; Fri, 20 Mar 2020 20:32:30 +0000
Received: from BL0PR11MB3412.namprd11.prod.outlook.com ([fe80::9d83:ab47:2249:e024]) by BL0PR11MB3412.namprd11.prod.outlook.com ([fe80::9d83:ab47:2249:e024%5]) with mapi id 15.20.2835.017; Fri, 20 Mar 2020 20:32:30 +0000
From: "Kamran Raza (skraza)" <skraza@cisco.com>
To: Theresa Enghardt <theresa@inet.tu-berlin.de>, "gen-art@ietf.org" <gen-art@ietf.org>
CC: "mpls@ietf.org" <mpls@ietf.org>, "draft-ietf-mpls-ldp-yang.all@ietf.org" <draft-ietf-mpls-ldp-yang.all@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "draft-ietf-mpls-ldp-yang@ietf.org" <draft-ietf-mpls-ldp-yang@ietf.org>, "BRUNGARD, DEBORAH A" <db3546@att.com>
Thread-Topic: Genart last call review of draft-ietf-mpls-ldp-yang-06
Thread-Index: AQHVc3X5Cf1F/fCNsk+ckNA4TRxlLqhSw1+A
Date: Fri, 20 Mar 2020 20:32:30 +0000
Message-ID: <23B171FE-44E0-46B4-A231-471E20CE71B0@cisco.com>
References: <156939784126.29030.13643769362107489881@ietfa.amsl.com>
In-Reply-To: <156939784126.29030.13643769362107489881@ietfa.amsl.com>
Accept-Language: en-CA, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.22.0.200209
authentication-results: spf=none (sender IP is ) smtp.mailfrom=skraza@cisco.com;
x-originating-ip: [2001:420:c0c4:1002::f5]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: e6335a6d-42e7-4807-928a-08d7cd0dd008
x-ms-traffictypediagnostic: BL0PR11MB3267:
x-microsoft-antispam-prvs: =?utf-8?q?=3CBL0PR11MB326791B6EBB5A668AC8784DCD0F?= =?utf-8?q?50=40BL0PR11MB3267=2Enamprd11=2Eprod=2Eoutlook=2Ecom=3E?=
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 03484C0ABF
x-forefront-antispam-report: SFV:NSPM; =?utf-8?q?SFS=3A=2810009020=29=284636?= =?utf-8?b?MDA5KSgzNzYwMDIpKDM2NjAwNCkoMzQ2MDAyKSgzOTYwMDMpKDEzNjAwMykoMzk4?= =?utf-8?q?60400002=29=28199004=29=2866476007=29=2886362001=29=285660300002?= =?utf-8?b?KSg0Nzg2MDAwMDEpKDE4NjAwMykoNjUxMjAwNykoMzE2MDAyKSg4OTM2MDAyKSg2?= =?utf-8?q?506007=29=2871200400001=29=2864756008=29=2876116006=29=286486002?= =?utf-8?b?KSg2NjU1NjAwOCkoODExNjYwMDYpKDY2OTQ2MDA3KSgyNjE2MDA1KSgyOTA2?= =?utf-8?b?MDAyKSg4MTE1NjAxNCkoNjY0NDYwMDgpKDQzMjYwMDgpKDg2NzYwMDIpKDU0?= =?utf-8?q?906003=29=2836756003=29=28110136005=29=2833656002=29=3B?= DIR:OUT; SFP:1101; SCL:1; SRVR:BL0PR11MB3267; H:BL0PR11MB3412.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: =?utf-8?q?T90cg7K7EPrT7EhOf+JzArdcR5/sr46?= =?utf-8?q?0rNchsfkcUvelBh3X7m30cI53xQ1Z02NwydH1tWZkbF9kSr/5PMWRNOjDZTvtelM9?= =?utf-8?q?NnG6jg31u172B8b6HiPXv7DkcuG6tS6vvtWVb9utWLajJsi6b93TLChWdqqrowlJM?= =?utf-8?q?sj3UD5xXFQ5/7qNa8qv9Eo6C9ZSdX/2A7p/ZhZPdy77cWObKAL/cKAMBKRUwloj4o?= =?utf-8?q?Lt22Xn57MBkeWp9M1ofzMDFtRFcOHEhHRjL6wZJmWZsNWwiGB+e4z5Jwmni/rMoPP?= =?utf-8?q?/BFjJUc+4+ai+tAMP8Pq4Ds/He9FKjcesRA3rC+E1uWwL2HpHXuSRxMEgJmpQrHgc?= =?utf-8?q?hxm4eneVdE89DvJB9gxJT2uu/StokJeZZtctRdZNlCtdOHw0aolRqqItY38whJeJf?= =?utf-8?q?Us2ic77PWDZUZXaU65LJp6u7cjLCBJFbjethMiHDKIWmeOhOJ2N9i1sdJorUN19BY?= =?utf-8?q?HrXR8pJXuzC05LqyfOe4uXGTL1azmoOClQf/harURdGQe1wQ=3D=3D?=
x-ms-exchange-antispam-messagedata: =?utf-8?q?CwpkzT2OcCB3dBMBx/MOpOEkpFRNGb?= =?utf-8?q?bFYRoJlOKnn0Bhtu01Kh82zgZ82k0UyadbjNIuqDI6HY1+MgNTJIYTKr2UHEsk0aG?= =?utf-8?q?1enqU/TAKt89ZO8g7yGec6xlJQoToQJRIxV4pnGW5rZCNC/K5lHexRnK8j7L5S023?= =?utf-8?q?qbDXeCdpEco=3D?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-ID: <F0FCE59531311F4A92E82596E7535F04@namprd11.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: e6335a6d-42e7-4807-928a-08d7cd0dd008
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Mar 2020 20:32:30.2737 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: =?utf-8?q?DRD+V+WzD0s82ThoTKF8i?= =?utf-8?q?ovxmH4hsvx8Cu4tQlLlMRk9z7qibxOzw6DXf73ItGuX5WAsRZfhlCVi/BM2xjDLOQ?= =?utf-8?q?=3D=3D?=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR11MB3267
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.12, xch-aln-002.cisco.com
X-Outbound-Node: alln-core-9.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/4QyHjb4AMt5szqOdBRvFSHAfoz4>
Subject: Re: [mpls] Genart last call review of draft-ietf-mpls-ldp-yang-06
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 20 Mar 2020 20:32:40 -0000

Hi Theresa,

We somehow missed this email - Our apologies.
We have updated few rev of drafts since then . The latest one being uploaded later today is rev-09 
Please see inline [skraza20]

On 2019-09-25, 3:51 AM, "Theresa Enghardt via Datatracker" <noreply@ietf.org> wrote:

    Reviewer: Theresa Enghardt
    Review result: Ready with Issues
    
    I am the assigned Gen-ART reviewer for this draft. The General Area
    Review Team (Gen-ART) reviews all IETF documents being processed
    by the IESG for the IETF Chair.  Please treat these comments just
    like any other last call comments.
    
    For more information, please see the FAQ at
    
    <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
    
    Document: draft-ietf-mpls-ldp-yang-06
    Reviewer: Theresa Enghardt
    Review Date: 2019-09-25
    IETF LC End Date: 2019-10-04
    IESG Telechat date: Not scheduled for a telechat
    
    Summary:
    
    This draft is basically ready for publication, but has some minor issues that
    should be fixed before publication.
    
    Major issues: None.
    
    Minor issues:
    
    Section 1.1:
    
    Why is LDP IPv6 grouped in the "extended" category and not in the "base"
    category, which the draft states to be the "minumum requirements for a typical
    base LDP deployment" and "suffice for small deployments"? Are typical, small
    deployments usually IPv4-only, and is this expected to remain true? Please
    consider briefly explaining this design decision.
[skraza20]: This was raised few times and we have clarified in emails and some edits in the doc.
    
    What does "igp sync" refer to? Is this the same as "igp-synchronization-delay"
    in the extended model? Please consider expanding this abbreviation and/or
    providing a reference.
    
[skraza20]: In one of prev rev, added a ref to the RFC.

    Section 3:
    
    Could you provide references for the "widely deployed non-RFC features", which
    are part of the extended model, please?
    
[skraza20]: This text has been reworked in latest rev.

    "GR session is in recovery state" - What does "GR" refer to?
[skraza20]: Graceful Restart. Expanding in rev -09.
    
    Section 10:
    
    In the Security Considerations, it would be great if you could provide some
    examples of writable/creatable/deletable data nodes which may be considered
    sensitive or vulnerable, and what negative effects on network operations one
    could expect if an attacker wrote to them.
[skraza20]: Done in rev -08.
    

    Nits/editorial comments:
    
    The document doesn't use any RFC 2119 keywords, yet has text resembling RFC
    2119 boilerplate text. Please consider removing the RFC 2119 boilerplate text.
    
[skraza]: Already fixed in rev -07.

    The document contains a few typos and grammar issues.
    To improve readability, please check for consistency of upper/lower case terms,
    for the use of definite and indefinite articles, and consider running a
    spellchecker.
[skraza20]: ack. Have fixed some known.
    
    Some examples:
    
    Section 1.1:
    
    "The configuration and state items are divided into following two broad
    categories" --> "The configuration and state items are divided into the
    following two broad categories"
    
    "This is worth higlighting " --> "It is worth highlighting"
    
    Section 3:
    
    "yang" - should this be all caps?
[skraza20]: Already fixed.
    
    "rpc" - should this be all caps?
[skraza20]: fixed.
    
    "grapically" --> "graphically"
[skraza20]: Already fixed.
    
    Section 5.2.1:
    
    "This container falls under global tree" --> "This container falls under the
    global tree"
    
[skraza20]: Fixing.

    "The example of former is interface hello timers, and example of latter is
    enabling hellos for a given AF under an interface." --> "The example of the
    former is interface hello timers, and an example of the latter is enabling
    hellos for a given AF under an interface."
[skraza20]: Fixing.    

    "A peer is uniquely identified using its LSR Id and hence LSR Id is the key for
    peer list" [missing punctuation]
[skraza20]: Already reworked.