[mpls] Updated rev -08 posted [Re: Éric Vyncke's No Objection on draft-ietf-mpls-ldp-yang-07: (with COMMENT)]

"Kamran Raza (skraza)" <skraza@cisco.com> Fri, 28 February 2020 04:44 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 070B93A0F81; Thu, 27 Feb 2020 20:44:23 -0800 (PST)
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=SVHBEajn; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=PDFOINF9
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 0-kk7VhIyVeI; Thu, 27 Feb 2020 20:44:21 -0800 (PST)
Received: from alln-iport-6.cisco.com (alln-iport-6.cisco.com [173.37.142.93]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id ACF133A0F71; Thu, 27 Feb 2020 20:44:21 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=3338; q=dns/txt; s=iport; t=1582865061; x=1584074661; h=from:to:cc:subject:date:message-id:content-id: content-transfer-encoding:mime-version; bh=dyQqVgjHtZhW5v4YawEOMs2L3pvNjCDr5ptsUzRYRcw=; b=SVHBEajnYiyjdda1KQg3xo/ha+3Q0NCI/uZSHt7ZUg05bTP9wwcrMYmI e4KfQpML409PliCcbBZXqk5LAfUsCPADMG3QJuVOX5CSZBo+khb0G4xE/ AR0dYFYg5SLz+6MMHUjt5IxvWCI39grD+lCeEkyaU0Tf64dLh1h5K+Nzc c=;
IronPort-PHdr: =?us-ascii?q?9a23=3A6jCdYxPD/WRYe0DTV5Ql6mtXPHoupqn0MwgJ65?= =?us-ascii?q?Eul7NJdOG58o//OFDEuKQ/l0fHCIPc7f8My/HbtaztQyQh2d6AqzhDFf4ETB?= =?us-ascii?q?oZkYMTlg0kDtSCDBjgL+TjfSUSF8VZX1gj9Ha+YgBY?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0CaCQBumVhe/5xdJa1mHgELHINPUAW?= =?us-ascii?q?BRCAECyoKhAqDRgOKZppzglIDVAkBAQEMAQEtAgQBAYRAGYFxJDgTAgMNAQE?= =?us-ascii?q?FAQEBAgEFBG2FNwyFZhYREQwBATcBEQEiAh8HAgQwFQUNBAENJ4MEgksDLgG?= =?us-ascii?q?jXwKBOYhidYEygn8BAQWFARiCDAmBDiqMJRqCAIERJyCDCodcMoIsjXCCOju?= =?us-ascii?q?eRXAKgjwEjRqJRxyCSYgbkEqDTIskm0cCBAIEBQIOAQEFgWkigVhwFWUBgkF?= =?us-ascii?q?QGA2OHTiDO4pVdIEpjHsBgQ8BAQ?=
X-IronPort-AV: E=Sophos;i="5.70,493,1574121600"; d="scan'208";a="461975531"
Received: from rcdn-core-5.cisco.com ([173.37.93.156]) by alln-iport-6.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 28 Feb 2020 04:44:20 +0000
Received: from XCH-ALN-003.cisco.com (xch-aln-003.cisco.com [173.36.7.13]) by rcdn-core-5.cisco.com (8.15.2/8.15.2) with ESMTPS id 01S4iKbF012387 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 28 Feb 2020 04:44:20 GMT
Received: from xhs-aln-002.cisco.com (173.37.135.119) by XCH-ALN-003.cisco.com (173.36.7.13) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 27 Feb 2020 22:44:20 -0600
Received: from xhs-rcd-001.cisco.com (173.37.227.246) by xhs-aln-002.cisco.com (173.37.135.119) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 27 Feb 2020 22:44:19 -0600
Received: from NAM12-DM6-obe.outbound.protection.outlook.com (72.163.14.9) by xhs-rcd-001.cisco.com (173.37.227.246) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Thu, 27 Feb 2020 22:44:19 -0600
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EzrvdxbC6D8w1xMjZtnRDk3fqEsh/mwvKbUnsYYm0j0rGpKpxqh5zmJ4D3Kg68xPD9X/ZS81x4NLNMfmosclz2XnyhKBO8srXBtZwRJiEIZQJeyrbIkgXKG4CYrIm9bKM6TEvaY0w2FRRotY927p3deCGHSCxIY0f5E4nWjMVH7M0t5jhVzfqmD2b5+pdsDHJXPBkND4J8KCkz5W+weo2+9yt2cKMcmjDOId2CaafOiGUIyG2rSo95s0QW/QdBZUTwSf2FGdYApnzA53voC7txYsPrY6/ZL9SCk4iYZYzRyeZ4dkMPWqCdhwhwXFMYYXUJuNQYZBAtlUvkeo5sankg==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dyQqVgjHtZhW5v4YawEOMs2L3pvNjCDr5ptsUzRYRcw=; b=W12O7ECTdqHpp8drxXm+aLLfebEAZes+ZPWBk/jDX+cx+4gnitURxlP4j5YyPxfzOlnzUEdBb4itScj0XIkiJ6mNsP8oLZhC8hl0JuxCQ4In9oOQt9dAn6HrjEzU5yzFl11zb+OtWAb2hyBrYER09E/ekRVbaNXSJy5jhUKo8jaUlDsuwPN+SuBwmDO8oEWZgJ3cs1HEy8dmNVZDAC4Pffi2g2bRYL2odsAG/ZcpnWM7S0pXHKLborrA/U7pm0uH5TgdLEIdXtTM/3iwDtcdZfMX1eLD6i9mHWKMjpu0sti+0SmDjPDeYgxQhwwRZ2UJfeOh5d8E79J/6oum7YZZ+w==
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; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dyQqVgjHtZhW5v4YawEOMs2L3pvNjCDr5ptsUzRYRcw=; b=PDFOINF9I9H1gNqph1DF4jYzwQHgI5Ke7Y/cvVEn+RQVzF31BPBj9cg6wKqBacMdpXI4VzJFjbbSQPqbMB3yprVY57RcfBQpMxs9hfNSGp5I8XXszux9VpYFC9NcbixCOsq5adNMVhgKH97c4hgeeEi89yUVT6MEQwzrKcvhOig=
Received: from BL0PR11MB3412.namprd11.prod.outlook.com (2603:10b6:208:7c::32) by BL0PR11MB3347.namprd11.prod.outlook.com (2603:10b6:208:2f::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2772.14; Fri, 28 Feb 2020 04:44:18 +0000
Received: from BL0PR11MB3412.namprd11.prod.outlook.com ([fe80::29ec:dcc7:ed48:3f7e]) by BL0PR11MB3412.namprd11.prod.outlook.com ([fe80::29ec:dcc7:ed48:3f7e%6]) with mapi id 15.20.2750.021; Fri, 28 Feb 2020 04:44:18 +0000
From: "Kamran Raza (skraza)" <skraza@cisco.com>
To: "Eric Vyncke (evyncke)" <evyncke@cisco.com>, The IESG <iesg@ietf.org>
CC: "draft-ietf-mpls-ldp-yang@ietf.org" <draft-ietf-mpls-ldp-yang@ietf.org>, Tarek Saad <tsaad.net@gmail.com>, Nicolai Leymann <n.leymann@telekom.de>, "mpls-chairs@ietf.org" <mpls-chairs@ietf.org>, "mpls@ietf.org" <mpls@ietf.org>
Thread-Topic: =?utf-8?B?VXBkYXRlZCByZXYgLTA4IHBvc3RlZCBbUmU6IMOJcmljIFZ5bmNrZSdzIE5v?= =?utf-8?B?IE9iamVjdGlvbiBvbiBkcmFmdC1pZXRmLW1wbHMtbGRwLXlhbmctMDc6ICh3?= =?utf-8?Q?ith_COMMENT)]?=
Thread-Index: AQHV7fG8569iomnCM069YUu39CA/7Q==
Date: Fri, 28 Feb 2020 04:44:18 +0000
Message-ID: <D87ED156-5BF8-4E56-A5DF-BD4C17A4EEA0@cisco.com>
Accept-Language: en-CA, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.21.0.200113
authentication-results: spf=none (sender IP is ) smtp.mailfrom=skraza@cisco.com;
x-originating-ip: [173.38.117.82]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 1e8c3518-8726-49f7-d8bd-08d7bc08df47
x-ms-traffictypediagnostic: BL0PR11MB3347:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <BL0PR11MB3347827065BB45411DED3F5AD0E80@BL0PR11MB3347.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 0327618309
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(39860400002)(136003)(366004)(396003)(346002)(376002)(189003)(199004)(316002)(110136005)(26005)(4326008)(36756003)(54906003)(2616005)(2906002)(478600001)(186003)(81156014)(81166006)(6486002)(224303003)(33656002)(5660300002)(86362001)(6512007)(6506007)(66476007)(71200400001)(8936002)(66946007)(64756008)(66446008)(66556008)(76116006); DIR:OUT; SFP:1101; SCL:1; SRVR:BL0PR11MB3347; H:BL0PR11MB3412.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; 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: jumGdBm9Jj7Pyz6b9KxFyE3E55drFHhtQCKcRZVwm2HeqDQl/zSIechKay3nt2hcjq7J5jE5awKUd+/rlO0y2xhcncpglPRjs56TDyEK9Ej5+wP8yLEWo85DGiZT0db0db9/AUPp6xV4OQcOtwQL8ytH+2FP46GSpsffjq/ZlbAT5KylaFB/aTgbQ5XsaJI0pCaG1+gshkfD05OXVFGhWRbMAccmzbrIMidkZ6PoRCRAldhbg31DuB2LT0fK911wqZHKLONln1LCM9T04v3XYaz7SWxVyMaEfHYFVC/FhinhzeYa9g8n7EeKusHkZem4NDrjz378YtpRp2T9XLiBAtk82u9K+1ImgnCfXP5N9yZeGjOYyyL3iBlqFILXB+MO0JA+euI9CHOoWU3TtpmH+6C9EDr+ZGVI3GXIUj/iamY0zo1kaF71cKYHTM6wsaa+
x-ms-exchange-antispam-messagedata: NUPMG5gpu3iuPfCMnVMls9It1a7L/rl3LF8HTvQU+BX4EbqHoXu4NMhtdHVxM5t4M/fYCBAWbF4XvhEO/po+B6TIIFUaZCZYOAxb0AKUhd4U6bz3KuTaEaA2vwsFFahzf5+paSAWlYJi7+CF8OBFQg==
Content-Type: text/plain; charset="utf-8"
Content-ID: <01BB2875010D5048AEBCF6A7C46C55D1@namprd11.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 1e8c3518-8726-49f7-d8bd-08d7bc08df47
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Feb 2020 04:44:18.4724 (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: nuX2EODqSt/2u/esI+QcNnRGhZSgaLQ4MK4QCKs9JUc1e0kZkicLJi4PMz/wMADswpumeZI9sbzAROK4Sz2kuA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR11MB3347
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.13, xch-aln-003.cisco.com
X-Outbound-Node: rcdn-core-5.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/BgmS1SuOgTsvXyUNOQM0Y1IUBHc>
Subject: [mpls] =?utf-8?q?Updated_rev_-08_posted_=5BRe=3A_=C3=89ric_Vynck?= =?utf-8?q?e=27s_No_Objection_on_draft-ietf-mpls-ldp-yang-07=3A_=28with_CO?= =?utf-8?q?MMENT=29=5D?=
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, 28 Feb 2020 04:44:30 -0000

Eric, 
Thanks for your review and comments.
We have posted a new rev -08 that takes care of your comments and comments from other IESG reviewers.
On behalf of authors, please see further inline [skraza]:

    
    
    ----------------------------------------------------------------------
    COMMENT:
    ----------------------------------------------------------------------
    
    Thank you for the work put into this document.
    
    I fully support Ben Kaduk's DISCUSS about IPv6 (about why IPv6 is in the
    extended section 6.1.2).  I would have balloted a DISCUSS if Ben haven't
    balloted before me.
[skraza]: Please see our response to Ben's review.
    
    Also concerned about Suresh's question about the "rw ipv4 (or ipv6)" in section
    6.2.1. and 6.2.2. and other places. Thank you, though, for the IPv6 example in
    Appendix A.

[skraza]: Fixed. Also, Added more IPv6 in examples.
    
    Answers to my COMMENTs below will be welcome,
    
    Regards,
    
    -éric
    == COMMENTS ==
    
    -- Section 4 --
    Why having the YANG subtrees for IPv4 and IPv6 different order of their
    subtrees in ietf-mpls-ldp/mpls-ldp/address-families/ipv[46] ?

[skraza]: This is due to augmentation and pyang tools puts the augmentation at the very end - this ordering is not controlled by us but the tool.
    
    -- Section 5.1.2 --
    Another difference about IPv4 and IPv6 in the tree: IPv6 has an "enable" leaf
    but IPv4 does not. Why is that ?
    
[skraza]: IPv4 was/is defacto AFI for LDP and was enabled by default in all implementations. LDPv6 was an add-on and was enabled on demand - hence "enable" is needed here.
BTW, conversely, a "disable" leaf is provided under ipv4.if someone wants to disable IPv4. 

    -- Section 5.2.1.1. --
    In the text "this document recommends an operator to pick a routable IPv4
    unicast address as an LSR Id", what is meant by "routable" ? Globally routable
    ? Domain routable ?
    
[skraza]: LDP typically works within IGP domain so I'd say IGP routing domain or perhaps just routing domain.
Added the text in the draft.
 

    Also, it is expected that design recommendations are done in a document about
    data model?
 
[skraza]: Fair point but this LSR-ID topic had been an important one in the context of manageability of LDP, we thought it better to highlight and recommend.