[RTG-DIR] RtgDir Last Call review: draft-ietf-spring-segment-routing-policy-14
"Bocci, Matthew (Nokia - GB)" <matthew.bocci@nokia.com> Fri, 28 January 2022 11:51 UTC
Return-Path: <matthew.bocci@nokia.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D140F3A0E31; Fri, 28 Jan 2022 03:51:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.473
X-Spam-Level:
X-Spam-Status: No, score=-7.473 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.576, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_NONE=0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=nokia.onmicrosoft.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 Uu0kJVs1kMX5; Fri, 28 Jan 2022 03:51:46 -0800 (PST)
Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-vi1eur04on0725.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe0e::725]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B5F6A3A0E30; Fri, 28 Jan 2022 03:51:45 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=RebmXFm1VDBBopnnLkBZbPLC1qdSX1UsBIc8p4Gh100bR32KESd7MmyO7ZSGBFYKh00q1Xc94ejHLRtCzrcr/3pt7bTFL6uikh23Fw3Bb9GqIT+ZeDaLHmVkyXI3NWjqCTXe3GTH9mqe7lv/ATgV8I4SEPsxfgyJDl3BXUI4uwR5MSjpClfYQNokQ/JvQ4bn+ikHAKcsb/+0cZHunhvKfkyGqKZgTsQfsUUiRJaBd1qiEs0pgT6WuRY6SkGZOt91VWMmsoQn/NF7FzIBexJhrXNGYvfi3UDNQSFQBKL3fDXRF4XZcB8tGapuRHygFZGUC98pQMH1Fg2CjG5ROwdJjA==
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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=/BfubEn4XmOiH+WiI1or9LOUXZvX6/ITi5+OGa5hzq4=; b=NBdOAa+jVlTMuZ+KetHER9bnc9JDTz/pXEP1xDJ1H/RDmZdVRisvQs/nTLrkQWec9XMsHYjT67bD80ghj/B0RaQnxlciE0MutalbD3tV8xPcaUEimYOFt1vKgfWEgfHbEHl6nmkZH8fN/4qzU/Ka9aLuLJH31DuS0jAKNmMy4ml3/h4vfGz3BzlRln1LD4dbw7tgBg3o70LAAVG2WpeRU4j9u7PJn2OANaavElDALQ1sQVWflCCG1Cxh5di2804IsnfCEU34nteEfkgP6UVssGWgn+r4Sy2t5928179OpImMGaPkL7hN2BK1auVxaXirUqZLZTXZDvY8Y5Jt/eDMZQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/BfubEn4XmOiH+WiI1or9LOUXZvX6/ITi5+OGa5hzq4=; b=ulsPIv4ZghePBkiBImOWJPA8OQc5WB8x6sqC9OdwY3vte+baKz6+skc15UJbX+g2oFbs65h256PQGsMw6asadHEDxVfp6C0J164HRYdC6H5PhKFfy5pcvSeTpZXgYH304cOThrigLMxUIflN1FvycqVkFg2YsES368gWw7tww4k=
Received: from VI1PR0701MB6991.eurprd07.prod.outlook.com (2603:10a6:800:17d::22) by VI1PR07MB4560.eurprd07.prod.outlook.com (2603:10a6:803:66::30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.6; Fri, 28 Jan 2022 11:51:41 +0000
Received: from VI1PR0701MB6991.eurprd07.prod.outlook.com ([fe80::8c5e:f395:67c0:a223]) by VI1PR0701MB6991.eurprd07.prod.outlook.com ([fe80::8c5e:f395:67c0:a223%4]) with mapi id 15.20.4930.019; Fri, 28 Jan 2022 11:51:41 +0000
From: "Bocci, Matthew (Nokia - GB)" <matthew.bocci@nokia.com>
To: "<rtg-ads@ietf.org>" <rtg-ads@ietf.org>
CC: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "spring@ietf.org" <spring@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "draft-ietf-spring-segment-routing-policy.all@ietf.org" <draft-ietf-spring-segment-routing-policy.all@ietf.org>
Thread-Topic: RtgDir Last Call review: draft-ietf-spring-segment-routing-policy-14
Thread-Index: AQHYFDzafFfVYeWBskCYJFPvsKw3RQ==
Date: Fri, 28 Jan 2022 11:51:41 +0000
Message-ID: <VI1PR0701MB6991A2F3A8D2FDFB4B255302EB229@VI1PR0701MB6991.eurprd07.prod.outlook.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nokia.com;
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 431b15cb-6b54-41b1-7926-08d9e2548c7f
x-ms-traffictypediagnostic: VI1PR07MB4560:EE_
x-microsoft-antispam-prvs: <VI1PR07MB45603408330AC070C52BF276EB229@VI1PR07MB4560.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: /54sFmkJUQdcHlkdWeCDuBVUOa47LN14t7LJRWHAwQ04gYPqIq3YTxwdwyOTT03NQANWKVlI/DhuOxPLwemyxVrArqwulMdh8Nn93yJh5lBCF4iHVJZFcFjhM/h11TNr2nk02fCYNeDqbqvD9SyCa7lrhna/Q9DVVBU61B2Hx2wtNoebrQiNlYqZ0D9V3WGTO0HSOl/yoxFe3uxBcKFY4MV49RJqpmu47eH367YUG/mhsVQbrqsOYPMRjdd3PRWJ68pMjckE82px2f0gYijFsj+o59TI/457N3bQsK2OhqcC+tpm9SI/k6y80fKmdYHHmsF6a4MxswtIdG2mgEiaoZij3prDD3lwAsUtitKGTx8bIF58ua/4ay6UnazPQlTL+GSNZl2gSaYcJIUxTooA5gZfbcK7955/4YEzG1MLRWehgDS+ALjn5sG+2d29LIotfPD3nw1y5+px+TT7sK0rsERtY/IwIWwvLwjSlmlD7gDPumbzcYlIVb9+z4z9x+JVMWx50NgAdHMYDO1repNCll9HmOQXY9Q2kYJ0B57brpZmQHftMa/XREU2cn2p9tBLAMP4uHAQ3YMZtjEv3wMCpt0sOlT4ifcTc6lZgSnZ3yZ1AEhZUehP6jn4eItNCosj7zfhfeMNu4YS+u/c4ehwI+MOS4Q+5R9K6X2dQLMiezO7a2IpRHGR93vc4RTFOPKX56L8Q5fv4BhDM1UEZImYj1U6nbrMdTwiRobQ4O3komJ+2fQDQRBSS8S77lSK252/kF5pphw9tUA7uxv3ZD2/aQW/YPoWtbyoAIbe0ASU/MmApQwqOHXw2y4EsAcCmKNX9M0g1x3tSn84lREGUmiLJlZl6J2NQQBAK/Gzc0c60cw=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR0701MB6991.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(4636009)(366004)(6506007)(7696005)(38100700002)(9686003)(33656002)(122000001)(55016003)(71200400001)(2906002)(26005)(186003)(66556008)(4326008)(508600001)(8676002)(9326002)(66946007)(8936002)(66476007)(66446008)(52536014)(64756008)(82960400001)(86362001)(54906003)(38070700005)(83380400001)(76116006)(316002)(5660300002)(450100002)(91956017)(491001)(20210929001); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: kwDf5+6sXW1qZmLqLLzNMo+F18xKe19YYurBH2XdMqaueT88elKCNLcwDI03NOL26My0MlO+/RJzgxE0bf99nQMbON4WtgPIuUwEvggLyraq/ZjoF78FfdXCcWDMxIu6omKsoABraIRvg7BPw/OBmR/4mw/QO3jJaICmHZ9BtmbfTpKRGCbZMCTujskZnnhouFs41S4IU1Ot2DoLTJ3Q1xI9w5LCLD/cowSlZIWKypWgZRryRMcEG2+v4qNLBBunxg9n+7kAIkvqBZCd9HTXqqpRIJf+g599JNVdGkc6xE6VKNVVQktjYLouUQc5dR+cE+xvWzyrGxaESnFtDqsg00GvWdHAHdoUVC2QG8shu6H8KDJFacg7SK+cbqBGLg9YBmt6U8Ub7EhFcCmdiGXHea4URvSlAdPMq0NslJAViBUCcXQZdyJP1wseTmPdCsUMuHejseRCAZpONouAewJ9EcsJGo9RXBSvPE3bAI3o17Wgr+hEOUaa3wB7LLZnh+gq20R4zyNbLPOuN3JFCdzJ7vEZTdJRjuz0VTZKT3TlxowEx/yxvW1U6lbp2oNCFghuhJdVWzU5SWHmlQ5oG53zxCBhrUud7btUMzqcbYTtWYEkK14CJoGMhzH5NIE3nq69hXHqjFr5AECTQXfLHiItn/DcqxokhEQKJOpHnqYtKP7C9PNgPqcYqIC8h5UUctO5zLTJrf5q6dAKD4Ybj5WV88GIfqpIvsaBqMlun4zaZueLVGL9Dtir8twxwnRYmW9aHpbpnZPwB7Yue5bv26qiyakkqEjhHZZS6r4NAHsCBG9EQcCu1M+yadZGAN6YTWFMsKdRwB3j6lwciV4mDGVWymOysMoLpHpHrcnXzxPrKEUNnIOVNDXPP3CogUATzq2uxkSZ8Gkwg3x0yr606SQXD0JdCGDgJ/KaewAkBS02C8P3P1x4KftNFBfhRnIhp+du3L9kScPxf5USqwZ4gaJ3+jqV8rHO7En4xyKKsNwbxSdSDdVgW/ecGRL1w2dyT+oarF3hhW2OJNskeFG4yW7PY81+2YaUbvxU6mppklx/BPuD4WzgvFSoP86UgkW8gIQfCNB8xxk3Mv3g1tgRA/7GzSftS4M6tbTtTHUp0yVGjDhsOfpKDoQRYel2Z55eFhODxFI32q3DaCLtIMPytFUuMknqHydg0XAGHa1RhUAC9Mg2bJV8OdnzCbIlfi622/+mCvK1Rm7iXT+XuaGqhMlUoJInpo6bvDq1Jo+FG/3BVXRSwnsXWqTny2DU5bUBoeRL6oaT93ggcqHHhSC51VxJU7uVacBhfuvQeTEo9zJyqkINxbRXyro0VLS80Uc5PMEXsWjj4ZxDyEzKgfCmaW9BUbszTj/2fCpo4dk2TD8/6tLh0ouyJlq7K9Q/QWY23ChwO65kGD3dGRNu/6weGlLBNHjeHJKTAiwaBKFc2syPTzz23f2m2K0kfbqyvBUKKQ2pQ2+Y2aDrdXoEP0orBhCl1SJBWrUMIBlIaMU3rcrWXvhDdXbGnnCVzOFeiDsUptf8YWX1YN6Asfw1zFj6ihZqXOMQNnW4Qbh++iRqoD6VlQzTHqQwqTD9jeDNsiq60+amr8uRlCVVudkZij4kvkJw6wUcza9YrTMOFUfjrUP/7+M=
Content-Type: multipart/alternative; boundary="_000_VI1PR0701MB6991A2F3A8D2FDFB4B255302EB229VI1PR0701MB6991_"
MIME-Version: 1.0
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR0701MB6991.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 431b15cb-6b54-41b1-7926-08d9e2548c7f
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Jan 2022 11:51:41.1617 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 6e2qZjiDTDzEXawd1qgaNH7XpDAZDOfRNtTBJw9E+oStL2cEyhJJlJnNkh2A6QuY4drHH5i9xHKI7Y1x/azLWg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB4560
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/e70ZtfgvsR4YCtPeD8vFeb1spkc>
Subject: [RTG-DIR] RtgDir Last Call review: draft-ietf-spring-segment-routing-policy-14
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-dir/>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 28 Jan 2022 11:51:50 -0000
Hello, I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft. Document: draft-ietf-spring-segment-routing-policy-14 Reviewer: Matthew Bocci Review Date: 28 January 2022 Intended Status: Standards Track Summary: In general, this is a well written document. Thank you. However, I have some minor concerns about this document that I think should be resolved before publication. This mostly revolve around the clarity of the document and the use (or lack thereof) of RFC2119 language. Comments: Major Issues: No major issues found Minor Issues: 1) This is a standards track document, but in general I found that clear specification language is missing. For example, in section 2.3: "A headend may be.." Should this be "A headend MAY be..."? There are many other cases like this where MUST/SHOULD/MAY would be better used rather than 'is' or 'can'. 2) The references to control planes for provisioning and maintaining SR Policies are only informational, but they are referred to in a manner in the text that I read as normative (although the language is not always clear). For example, in section 2.5: "When signaling is via PCEP..." and then the paragraph refers to an informative reference to the PCE draft for the SR policy control plane. Given that this is a standards track architecture document, it would be much better to be clear about what the normative parts of the architecture are. If these parts are not normative (for example even if I use BGP it is not mandatory to use it according to a particular specification) then please be explicit and use 'MAY' or 'SHOULD'. 3) Section 2.2: Candidate Path and Segment List. This section describes a hierarchical relationship between composite candidate paths, SR Policies, candidate paths, and segment lists. It would be much clearer if you could provide a diagram illustrating this hierarchy. 3) Terminology section. Since this draft is really the overall guide to all things SR Policy, it would really help to include a terminology section summarising new terms and acronyms. Nits: 1) The definite/indefinite article ('the', 'a', etc) is missing from the text in many places. I would suggest going through the text carefully and correcting these issues. 2) Section 2.13: In the information model: SR policy POL1 <headend = H1, color = 1, endpoint = E1> Candidate-path CP1 <protocol-origin = 20, originator = 100:1.1.1.1, discriminator = 1> Preference 200 Priority 10 Weight W1, SID-List1 <SID11...SID1i> Weight W2, SID-List2 <SID21...SID2j> ^^^^^^^^^ These are referred to as segment lists in the main text, so maybe you should align the terminology. Section 4: Segment Types. Type A: SR-MPLS Label: "...Additionally, reserved labels..." These are now commonly referred to in MPLS as "special purpose labels".
- [RTG-DIR] RtgDir Last Call review: draft-ietf-spr… Bocci, Matthew (Nokia - GB)
- Re: [RTG-DIR] RtgDir Last Call review: draft-ietf… Ketan Talaulikar
- Re: [RTG-DIR] RtgDir Last Call review: draft-ietf… Bocci, Matthew (Nokia - GB)
- Re: [RTG-DIR] RtgDir Last Call review: draft-ietf… Ketan Talaulikar