[Pce] Shepherd's review of draft-ietf-pce-pcep-exp-codepoints
Jonathan Hardwick <Jonathan.Hardwick@metaswitch.com> Sun, 12 November 2017 04:04 UTC
Return-Path: <Jonathan.Hardwick@metaswitch.com>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 587FB1270A0; Sat, 11 Nov 2017 20:04:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.02
X-Spam-Level:
X-Spam-Status: No, score=-2.02 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=-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=metaswitch.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 ehs_DLl1x4IV; Sat, 11 Nov 2017 20:04:22 -0800 (PST)
Received: from NAM02-CY1-obe.outbound.protection.outlook.com (mail-cys01nam02on0121.outbound.protection.outlook.com [104.47.37.121]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 46ADD1201FA; Sat, 11 Nov 2017 20:04:22 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=metaswitch.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=gvipo/rUHmmdxNt0Ms0OMBziwdJ8gUOiIY/eYkkNbnY=; b=YWwt4PJAlkdYZbAwEube3vF8f0UpR6AcSEde6MXQsBqtdskNVPZ9QzMgD6+cGl8VXS2jaE6+FVOeYEH45acTDLjuNtsJej3B7k/F4Pc31hhSfshFzf0cnxRnp1jN4WlfGNDxS+qqCereG/xdQpdcLGSDApxJ2XdWPRCA0gggIXk=
Received: from CY4PR0201MB3603.namprd02.prod.outlook.com (52.132.99.21) by CY4PR0201MB3603.namprd02.prod.outlook.com (52.132.99.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.218.12; Sun, 12 Nov 2017 04:04:20 +0000
Received: from CY4PR0201MB3603.namprd02.prod.outlook.com ([fe80::dce9:6773:68c7:10f3]) by CY4PR0201MB3603.namprd02.prod.outlook.com ([fe80::dce9:6773:68c7:10f3%13]) with mapi id 15.20.0218.011; Sun, 12 Nov 2017 04:04:20 +0000
From: Jonathan Hardwick <Jonathan.Hardwick@metaswitch.com>
To: "draft-ietf-pce-pcep-exp-codepoints@ietf.org" <draft-ietf-pce-pcep-exp-codepoints@ietf.org>
CC: "pce@ietf.org" <pce@ietf.org>, "pce-chairs@ietf.org" <pce-chairs@ietf.org>
Thread-Topic: Shepherd's review of draft-ietf-pce-pcep-exp-codepoints
Thread-Index: AdNbazxoJX1mpJIUTGiD0iKyZXkw0w==
Date: Sun, 12 Nov 2017 04:04:20 +0000
Message-ID: <CY4PR0201MB36030DD4394ABF68124A5CFA842A0@CY4PR0201MB3603.namprd02.prod.outlook.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=Jonathan.Hardwick@metaswitch.com;
x-originating-ip: [2001:67c:1232:144:38ce:368e:58a3:4004]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; CY4PR0201MB3603; 6:qzKT3ENkwAL+2oaJfqVSwYoCWeqTn8TVC+opFBwIlsjYYkLsuXcZsRJWjzrVjc6B4MuDD0u2I3/ZbDpCxrw62f0pKZ647y07v2yAKRTt9zErkkwxkYh4cS6DJ4yY+XmGQiKRqilCMbfGZacBb4gA0BE7c0Y+yseSPi8UlL+CAgLTCAtEvvF4WxdxX3bwfZmXZdymS+zBHVoGJO2JaMjYhOryLbSxgUKxdxK48+4gZJioe7W7QBA+pPnOTm/1DadcnNtMuIXIS/opmHXtuMN1TFrroG+oIHoQKICIeC0W6E3A1gaIYU+zcy/9MevHunYvvO5Jm43MXEFRLgJoJra8RQWkBkhsdaziboYhZ62pFi0=; 5:sLNaMyIqqCiG08T3t6uOhBGVYKJI54K12wYTnbUHrKV8FrNp9NVtN6q9ehg3BDxDJwmZ2LWOvVEQDsQr/RyPDTMNdBHLTNw0/eFwjWTddjFvREVmVIg8fMsjhEiGnTW4ismVRyrnv9jWnim+PY8LSe9yi9cvD2Asr4/FilT0mEY=; 24:dBcdYexirjOQqO8sRSsht9Aox4gYGy8HckC3ba84B+3dwpWV8BPv3S99LPQ80iOjJ6+0T7KTHVzyphGchm+o6RkHrjGevyiDkCz74sn+VOs=; 7:/ZkSZzMdB1KyrgssRJSLM8oqm7JQLiqcn4bzfoctvldafiAeQjb9XFZEPnPzJYmixegqHneJ1H4Kwmg0PWD808fWeanhbvhVrIYs+rq+qvlaQzQT+DYOkv6H0XQN/ZCvAwcnuNcuIn7rL020D7MYSrgAa5XLC7PQlkjDjGAd6Fn0gAyULfoUZcHwgi+Z1JBcV5SWuD0OxEN4NR4aD/HP+TU5rwbSvp98zkrgc1fuw46EFWT1LF/cSp/G4HZWw1eX
x-ms-exchange-antispam-srfa-diagnostics: SSOS;
x-ms-office365-filtering-correlation-id: ea7af2ec-fa61-4a5c-99d5-08d5298273bf
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(4534020)(4602075)(4627115)(201703031133081)(201702281549075)(2017052603199); SRVR:CY4PR0201MB3603;
x-ms-traffictypediagnostic: CY4PR0201MB3603:
x-microsoft-antispam-prvs: <CY4PR0201MB36035B0C9BA95B6D460B428F842A0@CY4PR0201MB3603.namprd02.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(131327999870524)(21748063052155);
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(10201501046)(3231022)(3002001)(100000703101)(100105400095)(93006095)(93001095)(6041248)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123562025)(20161123560025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CY4PR0201MB3603; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CY4PR0201MB3603;
x-forefront-prvs: 0489CFBAC9
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(376002)(346002)(189002)(199003)(2351001)(790700001)(6506006)(3660700001)(316002)(86362001)(450100002)(54906003)(106356001)(105586002)(4326008)(2906002)(7736002)(81156014)(3280700002)(14454004)(8676002)(6436002)(81166006)(8936002)(5640700003)(6116002)(55016002)(25786009)(102836003)(33656002)(54896002)(6306002)(230783001)(7696004)(5660300001)(9686003)(50986999)(2501003)(53936002)(99286004)(72206003)(189998001)(478600001)(54356999)(6916009)(5250100002)(101416001)(2900100001)(97736004)(5630700001)(53546010)(68736007)(74316002); DIR:OUT; SFP:1102; SCL:1; SRVR:CY4PR0201MB3603; H:CY4PR0201MB3603.namprd02.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en;
received-spf: None (protection.outlook.com: metaswitch.com does not designate permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: multipart/alternative; boundary="_000_CY4PR0201MB36030DD4394ABF68124A5CFA842A0CY4PR0201MB3603_"
MIME-Version: 1.0
X-OriginatorOrg: metaswitch.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ea7af2ec-fa61-4a5c-99d5-08d5298273bf
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Nov 2017 04:04:20.6279 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 9d9e56eb-f613-4ddb-b27b-bfcdf14b2cdb
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR0201MB3603
Archived-At: <https://mailarchive.ietf.org/arch/msg/pce/mcf_iEdDeTidqyU0Mcll4RnmJYM>
Subject: [Pce] Shepherd's review of draft-ietf-pce-pcep-exp-codepoints
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 12 Nov 2017 04:04:25 -0000
Re-sending to the correct DL :) From: Jonathan Hardwick Sent: 12 November 2017 12:02 To: 'draft-ietf-pce-exp-codepoints@ietf.org' <draft-ietf-pce-exp-codepoints@ietf.org> Cc: 'pce@ietf.org' <pce@ietf.org>; pce-chairs@ietf.org Subject: Shepherd's review of draft-ietf-pce-exp-codepoints Hi there I am the document shepherd for this draft. Please find my review of the draft below. Many thanks for writing this draft. It looks in good shape overall. There are just a few clarifications I would like to make before we forward it to the IESG for publication. Cheers Jon Abstract This sentence about new sub-registries is misleading - the allocation policy for new sub-registries is decided by the drafts that create the sub-registries and does not have to be IETF Review. I propose: OLD IANA established a new top-level registry to contain all PCEP codepoints and sub-registries. The allocation policy for each new registry is by IETF Review. NEW IANA established a top-level registry to contain all PCEP codepoints and sub-registries. This top-level registry contains sub-registries for PCEP message, object and TLV types. The allocation policy for each of these sub-registries is IETF Review. END Introduction OLD The Path Computation Element communication Protocol (PCEP) provides mechanisms for Path Computation Elements (PCEs) to perform path computations in response to Path Computation Clients (PCCs) requests. NEW The Path Computation Element communication Protocol (PCEP) [RFC5440] provides mechanisms for Path Computation Elements (PCEs) to perform path computations in response to Path Computation Clients (PCCs) requests. END i.e. add reference to RFC 5440. The second paragraph is superfluous - I suggest deleting: Further, in order to support use cases described in [RFC8051], [I-D.ietf-pce-stateful-pce] specifies a set of extensions to PCEP to enable stateful control of MPLS-TE and GMPLS LSPs via PCEP. [I-D.ietf-pce-pce-initiated-lsp] describes the setup, maintenance and teardown of PCE-initiated LSPs under the stateful PCE model. Please apply the same comments I made for the abstract to the following text: OLD IANA established a new top- level registry to contain all PCEP codepoints and sub-registries. The allocation policy for each new registry is by IETF Review as described in [RFC8126]. NEW IANA established a top- level registry to contain all PCEP codepoints and sub-registries. This top-level registry contains sub-registries for PCEP message, object and TLV types. The allocation policy for each of these sub-registries is IETF Review. END Suggested change for clarity: OLD With some recent advancement, there is an enhanced need to experiment with PCEP. NEW Recently, there have been rapid advancements in PCE technology, which has created an enhanced need to experiment with PCEP. END Section 5 The following paragraph does not tell the whole story. A PCE that does not recognize an experimental PCEP object, will reject the entire PCEP message and send a PCE error message with Error- Type="Unknown Object" or "Not supported object" as described in [RFC5440]. If the P flag is clear in the object header, then the PCE MAY ignore the object instead of generating this error message. Also, you do not discuss what a PCC would do on receipt of a PCUdp or PCInitiate containing an unrecognised experimental object - it is inconsistent that you don't cover these cases. (FWIW, RFC 8231 is a bit ambiguous about what a PCC should do about the PCUpd. Section 6.2 says that a PCErr should be sent, but then it refers to section 7.3.3, which says that a PCRpt should be sent. Hmmm.) Also: s/PCE error message/PCErr message/ Section 7 Nit: add comma after "accidentally" Appendix A I think the text in this Appendix could be clearer. Here is my suggestion. OLD Based on the feedback from the WG, it was decided to focus only on the essentials in the scope of this documents. For others, Experiments can use a new experimental TLV/Object instead. NEW Based on feedback from the PCE WG, it was decided to allocate an Experimental code point range only in the message, object and TLV sub-registries. The justification for this decision is that, if an experiment finds that it wants to use a new code point in another PCEP sub-registry, it can implement the same function using a new experimental object or TLV instead. END Other Please update reference draft-ietf-pce-stateful-pce -> RFC 8231 Please update reference draft-ietf-pce-pce-initiated-lsp-10 -> draft-ietf-pce-pce-initiated-lsp-11
- [Pce] Shepherd's review of draft-ietf-pce-pcep-ex… Jonathan Hardwick
- Re: [Pce] Shepherd's review of draft-ietf-pce-pce… Dhruv Dhody
- Re: [Pce] Shepherd's review of draft-ietf-pce-pce… Jonathan Hardwick
- Re: [Pce] Shepherd's review of draft-ietf-pce-pce… Dhruv Dhody
- Re: [Pce] Shepherd's review of draft-ietf-pce-pce… Julien Meuric
- Re: [Pce] Shepherd's review of draft-ietf-pce-pce… Adrian Farrel
- Re: [Pce] Shepherd's review of draft-ietf-pce-pce… Jonathan Hardwick