Re: Rtgdir early review of draft-ietf-ospf-mpls-elc-09
"Acee Lindem (acee)" <acee@cisco.com> Fri, 13 September 2019 19:46 UTC
Return-Path: <acee@cisco.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 583961200FF; Fri, 13 Sep 2019 12:46:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.5
X-Spam-Level:
X-Spam-Status: No, score=-14.5 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_HI=-5, 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=V7pgmsIh; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=yxtHQjv2
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 G2JVdaVY8v7v; Fri, 13 Sep 2019 12:46:04 -0700 (PDT)
Received: from rcdn-iport-8.cisco.com (rcdn-iport-8.cisco.com [173.37.86.79]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A2B181200FA; Fri, 13 Sep 2019 12:46:04 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6582; q=dns/txt; s=iport; t=1568403964; x=1569613564; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=PvRBZY7Tk9BYE9YhuXup9aHzVqX3Zibq7eKnGQEL2HM=; b=V7pgmsIhMvRWy/Ph9zCE80ErbstjZsyFuFt6JZrQ4PT3apGcKG2NmGc9 uxk2xa/UQk8LBFBXl5TL/ZNkCGF/i70EfXcTcNk5Ip7OpQwpFXfJ07g2s WH0Tz/5ORAc03/ozlS1chFEJaqehNxZpLaWtSaONgpzrmRq+oLE/A/Yg5 8=;
IronPort-PHdr: 9a23:6B/ayRzYqFs8c/jXCy+N+z0EezQntrPoPwUc9psgjfdUf7+++4j5YRGN/u1j2VnOW4iTq+lJjebbqejBYSQB+t7A+GsHbIQKUhYEjcsMmAl1CcWIBGXwLeXhaGoxG8ERHFI=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0AYAACb8Xtd/4MNJK1mGwEBAQEDAQEBBwMBAQGBUwYBAQELAYFEUANtViAECyqEIYNHA4RShhyCNyWXcIEugSQDVAkBAQEMAQEjCgIBAYQ/AheCSSM0CQ4CAwkBAQQBAQECAQYEbYUuDIVLAgEDEhERDAEBMgUBDwIBCBQDAwImAgICMBUICAIEAQ0FIoMAAYFqAx0BAgyfaQKBOIhhc4Eygn0BAQWBRkGDChiCFgmBDCgBhH+GeBiBf4EQAScME4JMPoJWCwIDAYRoMoImj0CdCwqCIYYdZIl6g3wbgjRwhlCPFo1/iASQagIEAgQFAg4BAQWBUjiBWHAVGksBgkEJgjmDcoUUhT9zAYEojn8BAQ
X-IronPort-AV: E=Sophos;i="5.64,501,1559520000"; d="scan'208";a="628936044"
Received: from alln-core-1.cisco.com ([173.36.13.131]) by rcdn-iport-8.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 13 Sep 2019 19:46:03 +0000
Received: from XCH-RCD-003.cisco.com (xch-rcd-003.cisco.com [173.37.102.13]) by alln-core-1.cisco.com (8.15.2/8.15.2) with ESMTPS id x8DJk3rw025854 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 13 Sep 2019 19:46:03 GMT
Received: from xhs-rtp-001.cisco.com (64.101.210.228) by XCH-RCD-003.cisco.com (173.37.102.13) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 13 Sep 2019 14:46:02 -0500
Received: from xhs-rcd-001.cisco.com (173.37.227.246) by xhs-rtp-001.cisco.com (64.101.210.228) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 13 Sep 2019 15:46:01 -0400
Received: from NAM04-BN3-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; Fri, 13 Sep 2019 14:46:01 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Dsas7BRtQMikl76WhlEaTeMaoo+cfG646Z8KU8BnyDwexlQdpqalUL9A8JESWdQbsfWAPkYwgmirpn00CRKZkKXAtlxrnV9LvRkjExb7DQPoetvrwvCXtJoT+HS0RILGQvdfOfUeLqy+f5lRS6pC07L7DwAmMzzMeDNs2tTCSKhQ/GPcBi+zqTaI+SgMQfOjTRnRuV5Xm22qHpjSIQKdId8xxD3ohEWS1EfcdzRpTCvzbLxx3pC6Fo+OnYcD9/DDnHX3jcCfGZT44/dZ2SrQD51bW0G1VkFEW3Akab2mh/OQd+kChlrtEdWARCsTrwdWzlgvnz1YmcBGUowMlno7eA==
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=PvRBZY7Tk9BYE9YhuXup9aHzVqX3Zibq7eKnGQEL2HM=; b=gd9hXqA/ZL08ipBzXMVYgeMh+eByZvNBkK1a0hZQAMgI9ZwdNOvsjcFS1MCZZjHmbYqrAwWraMvEXoMunMrnJDDViSEqbx6o7X7NmL2yyzJ40OmKOGY3CcnFy+Nn7pQlW2dsw9nfnRsczRJgsjvPcYcDbmDx4gUHR6MBSk7CVrBNYjvGG8nXebrq/Iv03R4pY7A79uAeTaruDRaK1WZwEa+bf4tksFu5w9RP4F7wZYUiYMx+k4J40IecHiJFJI54gw0y5ygOENYAUaXU6QzYtmjt2MdVf12RX7riDfH+Ks4U3bGTVpXT/0dzg5XGv2TTnKmb3sU0vDuToSObnl1RZg==
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=PvRBZY7Tk9BYE9YhuXup9aHzVqX3Zibq7eKnGQEL2HM=; b=yxtHQjv2OTv9TX9t3uDt6V9V0WLP2eggYWKBQa21U2ndRI0afDKvvST+qsGW+LF126/adKDyK21n7id/mf2IRyC3qbH8aXszWu9XJMxDYbxunJCsRHGRR4qcmpmnI6ZU4nDMBMInIklVZhh47dVkv9yHvRZPuH3HrRmEYS4qMps=
Received: from MN2PR11MB4221.namprd11.prod.outlook.com (52.135.38.14) by MN2PR11MB4013.namprd11.prod.outlook.com (10.255.181.154) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2263.21; Fri, 13 Sep 2019 19:46:00 +0000
Received: from MN2PR11MB4221.namprd11.prod.outlook.com ([fe80::cdc1:a2cf:eb3:a420]) by MN2PR11MB4221.namprd11.prod.outlook.com ([fe80::cdc1:a2cf:eb3:a420%6]) with mapi id 15.20.2263.021; Fri, 13 Sep 2019 19:46:00 +0000
From: "Acee Lindem (acee)" <acee@cisco.com>
To: Dhruv Dhody <dhruv.ietf@gmail.com>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>
CC: "lsr@ietf.org" <lsr@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "draft-ietf-ospf-mpls-elc.all@ietf.org" <draft-ietf-ospf-mpls-elc.all@ietf.org>
Subject: Re: Rtgdir early review of draft-ietf-ospf-mpls-elc-09
Thread-Topic: Rtgdir early review of draft-ietf-ospf-mpls-elc-09
Thread-Index: AQHVaVKOtAcj5qDiOEmOWCHdRgjr3KcpwgYA
Date: Fri, 13 Sep 2019 19:46:00 +0000
Message-ID: <59322EE4-4E90-46B3-9654-E08CD0A38057@cisco.com>
References: <156828311401.16614.6386779752971919411@ietfa.amsl.com>
In-Reply-To: <156828311401.16614.6386779752971919411@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=acee@cisco.com;
x-originating-ip: [2001:420:c0c4:1008::18f]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 01941a02-040c-4b27-eeb2-08d7388300e5
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(5600166)(711020)(4605104)(1401327)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7193020); SRVR:MN2PR11MB4013;
x-ms-traffictypediagnostic: MN2PR11MB4013:
x-ms-exchange-purlcount: 2
x-microsoft-antispam-prvs: <MN2PR11MB4013D09927902C91D7A8497DC2B30@MN2PR11MB4013.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 0159AC2B97
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(396003)(376002)(136003)(39860400002)(346002)(366004)(189003)(199004)(33656002)(6506007)(76176011)(102836004)(99286004)(71190400001)(71200400001)(66556008)(5660300002)(6246003)(64756008)(66476007)(66946007)(2906002)(478600001)(229853002)(6486002)(14454004)(86362001)(6116002)(6436002)(486006)(6512007)(53936002)(7736002)(25786009)(305945005)(4326008)(66446008)(256004)(14444005)(36756003)(8936002)(81166006)(81156014)(8676002)(76116006)(446003)(11346002)(316002)(54906003)(110136005)(186003)(46003)(2501003)(476003)(2616005); DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR11MB4013; H:MN2PR11MB4221.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-message-info: 6Twuc9H2Rui9R4XlpOILHbvvkhj63nQoxRcAXSI+0RV85jbn8Z9jWkL3t9cBHzFs7fd/1vExbHpQWyum6Ydb/H/UUD7FuWyO1YdR8PB6+PMvMXRDw6fpA/5PzMOIylrC8ycf0LiCaHf6cG8EzBPOr6b/IvgZUobT8NPd6Clpnl7/n9VfGPK1sbm2useT0MCQTW2u7YOqtwPBURZSOh4vNnWQNxVo1rPQH4ARetY1X0GtyTk+LKljmGXlvAO7zLLuqTVPTtE34yV+TmTCzMKEKboUji4pahuh4KLflVZm5jDLLBoTXJOW7z/U41hHpike30s0+N4bNIGjjzRnXSqYTtYv4qe7r6Heukisd6LCRMDqBp1W/u0X6ujqBmv1L5B90U4PDBRgiLYlnyrTDeU8i3VzZxbmaO5uFiVCzCOP6XY=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-ID: <D10475DEE5FDF44A89CD7C63E8D1B205@namprd11.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 01941a02-040c-4b27-eeb2-08d7388300e5
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Sep 2019 19:46:00.1815 (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: wt34bsoimDW4Q+eXxqjxIIuXV/x7NZuUaYGp93PdmIsSGV1rC82GektmSG0bF53k
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4013
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.37.102.13, xch-rcd-003.cisco.com
X-Outbound-Node: alln-core-1.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/oFnfd-E1a0yihMv9C82oH4DF2eE>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.29
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: Fri, 13 Sep 2019 19:46:08 -0000
Thanks Dhruv for the review... Peter and other authors, Please include Dhruv's comments or respond as to why they are being omitted. Thanks, Acee On 9/12/19, 6:12 AM, "Dhruv Dhody via Datatracker" <noreply@ietf.org> wrote: Reviewer: Dhruv Dhody Review result: Has Issues Subject: RtgDir Early review: draft-ietf-ospf-mpls-elc-09 Hello I have been selected to do a routing directorate “early” review of this draft. https://datatracker.ietf.org/doc/draft-ietf-ospf-mpls-elc/ The routing directorate will, on request from the working group chair, perform an “early” review of a draft before it is submitted for publication to the IESG. The early review can be performed at any time during the draft’s lifetime as a working group document. The purpose of the early review depends on the stage that the document has reached. As this document is in working group last call, my focus for the review was to determine whether the document is ready to be published. Please consider my comments along with the other working group last call comments. For more information about the Routing Directorate, please see http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir Document: draft-ietf-ospf-mpls-elc-09 Reviewer: Dhruv Dhody Review Date: 12-09-2019 Intended Status: Standards Track Summary: I have some minor concerns about this document that I think should be resolved before it is submitted to the IESG. The draft is focused and straightforward, the reader needs to be aware of RFC6790 and draft-ietf-mpls-spring-entropy-label beforehand. I have reviewed this and the IS-IS I-D together and you will find similar comments for both I-Ds. Minor ***** (1) Please use updated requirement language text as per RFC 8174, as you do have a mix of upper-case and lower-case terms in your I-D. The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and "OPTIONAL" in this document are to be interpreted as described in BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all capitals, as shown here. (2) Could you mark that the codepoints mentioned in the draft are early allocated by IANA? This would make it clear that you are not squatting on them. I also suggest following change in Section 7 (IANA Considerations) - OLD: This document requests IANA to allocate one flag from the OSPFv2 Extended Prefix TLV Flags registry: 0x20 - E-Flag (ELC Flag) This document requests IANA to allocate one flag from the OSPFv3 Prefix Options registry: 0x04 - E-Flag (ELC Flag) NEW: IANA is requested to confirm the early allocation of the following code point in the OSPFv2 Extended Prefix TLV Flags registry: 0x20 - E-Flag (ELC Flag) IANA is requested to confirm the early allocation of the following code point in the the OSPFv3 Prefix Options registry: 0x04 - E-Flag (ELC Flag) END (3) Section 4, I think a reference to RFC 8476 is needed as well to state the ERLD is advertised as part of Node MSD advertisement as defined in [RFC8476]. As mentioned in my review of the IS-IS I-D, what happens if one receives ERLD in the Link MSD advertisement? As per my understanding this is not allowed, better to add normative text for the case then. (4) Section 8, suggest to also add one sentence for the impact of advertising incorrect ERLD. If there isn't any, that can also be stated. Nits **** (1) Suggested ordering of sections - ..ELC/ERLD/BGP-LS/ACK.. [matching between OSPF/ISIS] (2) Section 2, add [I-D.ietf-mpls-spring-entropy-label] for terminology reference (3) Section 3, Add reference to draft-ietf-mpls-spring-entropy-label for the definition and usage of ERLD (4) Section 6, The ERLD MSD-type introduced for OSPF in Section 4 is advertised using the Node MSD TLV (TLV 266) of the BGP-LS Node NLRI Attribute as defined in section 3 of [I-D.ietf-idr-bgp-ls-segment-routing-ext]. I think you mean draft-ietf-idr-bgp-ls-segment-routing-msd here! Also, maybe change the title "BGP-LS Extension" as there is no 'extension' required, ELC/ERLD is BGP-LS would be automatically supported. (5) Expand MSD on first use. Thanks! Dhruv
- Rtgdir early review of draft-ietf-ospf-mpls-elc-09 Dhruv Dhody via Datatracker
- Re: Rtgdir early review of draft-ietf-ospf-mpls-e… Acee Lindem (acee)
- Re: Rtgdir early review of draft-ietf-ospf-mpls-e… Peter Psenak
- Re: Rtgdir early review of draft-ietf-ospf-mpls-e… Dhruv Dhody
- Re: Rtgdir early review of draft-ietf-ospf-mpls-e… Peter Psenak
- Re: Rtgdir early review of draft-ietf-ospf-mpls-e… Peter Psenak