Re: [Idr] [Last-Call] Genart last call review of draft-ietf-idr-ext-opt-param-11

Christer Holmberg <christer.holmberg@ericsson.com> Fri, 23 April 2021 07:03 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: idr@ietfa.amsl.com
Delivered-To: idr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 947C63A084A; Fri, 23 Apr 2021 00:03:58 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.002
X-Spam-Level:
X-Spam-Status: No, score=-2.002 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, HTTPS_HTTP_MISMATCH=0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 7qRWB4oDXKC2; Fri, 23 Apr 2021 00:03:53 -0700 (PDT)
Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-eopbgr00051.outbound.protection.outlook.com [40.107.0.51]) (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 2D6743A0846; Fri, 23 Apr 2021 00:03:52 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KnkdqTeLaIlFP2Pz/Ef/gpOtdjrRvsopenob5IVpTJBuj1v0xM/mTopKRoZ26gkZMmvS4MdwTeeW34YotRqfFzRrSu/d0OVBiovtKcntzh5diyMPrVbky67xMusZ1M5BYp/g7O34n5XV+cYzKXzgebKg/nQdSjSSBzXvQgqEL3wrUSbADq9sG5B63vYeeDpSQWfNU4ydaCWptOeE/sGHqfMbbmBKu4u95ARlPTRLUAL7aX0APSpiUhutmXjxwA9fbwUHw7LYaEsICmfQvwp/hLEqrh5tVvaAkHmhLbNABJ/Y2bLMPIDIPgqTEaG2jJfKvn4cgsktUuP2qAaYwbOq9Q==
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=3I+PxiqswonTG9iuN/UcpNawPvEb9/1NiyJv8RjOOHc=; b=L9oIxuzF8NogIlWg9JjmUC4QJLFWI8xfavCZuwQnSEk9jwfIgyJMHZA30jgPxyaE0cZv0T9ib0aK/Sek5k2E/ROijwJdx683Oeez81NolkV9oWemCsMlq2slt5JntEaMQOec52Jz5L2Cv52mf7nY79CwhVUevzqO5dgeu1rYhlCthTC+GGH3pzmTlfNHXXnNXG9Z5xixINlyj5U2tDFUuPHWHjnVnAhxMyFl52HIJIvbJ02Y0LvqPIq1OVO8DxuZTeK4N1vi4N4ZmStMmF2eETC8oqgryJiMvQ4C8lEib4xsiKD4deE/emEwtr6nXfGjbZVHYr9FXmYAsfiWJ7cc7A==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=3I+PxiqswonTG9iuN/UcpNawPvEb9/1NiyJv8RjOOHc=; b=Bjw6pxeRyG6ncJ4zgHfmDkcfgZXBX4zArfnnK4Yr4dYIceO6w/V593AQUViFbP8XCnuU9KI0aLamBj6YOScDd3ShWCYFI6pNZOq4k4JL/ikD3nKbwSM3uuAoywFZbcRez9zN0OEmuOjF0l47FAGxFwe3dAxlqbhp8Yjfqm2pfzo=
Received: from AM0PR07MB3860.eurprd07.prod.outlook.com (2603:10a6:208:4c::18) by AM0PR07MB4305.eurprd07.prod.outlook.com (2603:10a6:208:ba::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4065.6; Fri, 23 Apr 2021 07:03:47 +0000
Received: from AM0PR07MB3860.eurprd07.prod.outlook.com ([fe80::b10f:ebc0:80d:db2]) by AM0PR07MB3860.eurprd07.prod.outlook.com ([fe80::b10f:ebc0:80d:db2%7]) with mapi id 15.20.4087.019; Fri, 23 Apr 2021 07:03:47 +0000
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: John Scudder <jgs=40juniper.net@dmarc.ietf.org>, Christer Holmberg <christer.holmberg=40ericsson.com@dmarc.ietf.org>
CC: "draft-ietf-idr-ext-opt-param.all@ietf.org" <draft-ietf-idr-ext-opt-param.all@ietf.org>, "idr@ietf.org" <idr@ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>
Thread-Topic: [Last-Call] Genart last call review of draft-ietf-idr-ext-opt-param-11
Thread-Index: AQHXNvTAdk+Euvz4ZUWF8/o+0C+TC6rAmjfkgABRnYCAAMNEEA==
Date: Fri, 23 Apr 2021 07:03:47 +0000
Message-ID: <AM0PR07MB3860F77105E2182C53F53C0293459@AM0PR07MB3860.eurprd07.prod.outlook.com>
References: <161851524693.27672.11525659031465457077@ietfa.amsl.com> <E9D6193D-2485-498F-8D1C-42F207724CD2@juniper.net> <AM0PR07MB386095C3B5942BFFF6A6170E93469@AM0PR07MB3860.eurprd07.prod.outlook.com> <CF43ABFC-CA22-4FE3-BBC3-CFD9D0933C4A@juniper.net>
In-Reply-To: <CF43ABFC-CA22-4FE3-BBC3-CFD9D0933C4A@juniper.net>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: dmarc.ietf.org; dkim=none (message not signed) header.d=none;dmarc.ietf.org; dmarc=none action=none header.from=ericsson.com;
x-originating-ip: [80.248.247.159]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 9b117e82-4cb5-46c5-14a7-08d90625f122
x-ms-traffictypediagnostic: AM0PR07MB4305:
x-microsoft-antispam-prvs: <AM0PR07MB4305E26EB79FE2B408F9C9F093459@AM0PR07MB4305.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:8882;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: kpvg3jcmO/Wnr7SU3yCa/Bwky/O3Inn4aN6nDf3yN+L4DDjHahc8J6wI+YqmCq6dQdXhoNv9GnxJppY6DYVcxlgJUjPzx1YvHXr3wjTOvgMZZXxaAk7UG93nbTaPt+4mU+T+4EpRv3AatRKd70iBYC7IljMzxGx+rCcHv8CT5Xg/ViEbK+9eQla0OG9uB9OoELMCzDBrQCBKi1NCNqVskz/OYPw6vd4L48JfKOb8eD3Z12RjTwg//lKzTcvOtLVJSO719wH5njOTVOYPGSkCr5GxFX+TkXnTMcV709gHvaMD34VNwk+KECgavjh97fI11oekYj8Tw1jv7zmpJjAF2pmU7FVqoMLpB3l40Y1+4ZCjM5WXpYYM2+9/pt94ESj3OpcrU2HKSY4Ym1fFqmtQJuvSukccKtpNboeyC6yu02zFONqoeSxegJHB4zCbVWQ/CrSQktVyNDcxyQb2EoRYzHLOldTBMRBaNjh52YXEBhGE4VTAHENUDP2jcqItkT+JbYGmMkVEaEt7690PojL27Peldpl8olY2DCE6hU9rFFIPpwnY7N4+e2m5wskkupcpgrqqmsqOk8dXocjhZ2PDOutvoQvY8ZDAhab0swdjgH6f/Zack9L6FWQi0jYra3ZK9+J3Aew52U7z2g2uI6wIDi1hn6NNAoPjU8XInpbFLmR4Tg2/vzgTnV0j89Lqav/q
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM0PR07MB3860.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(136003)(376002)(366004)(346002)(39860400002)(396003)(122000001)(26005)(33656002)(316002)(38100700002)(71200400001)(8676002)(86362001)(44832011)(186003)(8936002)(76116006)(478600001)(4326008)(55016002)(7696005)(64756008)(83380400001)(5660300002)(110136005)(66476007)(66556008)(52536014)(66446008)(6506007)(53546011)(966005)(66574015)(66946007)(2906002)(54906003)(166002)(9686003); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: =?utf-8?B?NS9HTTV4U1oyYTVleEtubnIyVmVxOU1kUUFMNy9ycjB2NTIzU0luTmRJNjFl?= =?utf-8?B?anJ5dVN6dFYvQnYvcjJnS0tQOUQ5YW5uZFdDSmh5enNkRmI1YVUyMWd5clFU?= =?utf-8?B?UVNUQ2VPVldMaXNrZHh6TWFYZUx3elRkMVh2emxSenYxNzAzODZsTDJJVWxV?= =?utf-8?B?dXZhUzhzNW84RWh1c2c1V085dnZWSU1oWW9EK3RreWxPZkwzV1IvTHZtSXV4?= =?utf-8?B?OWRXNm9oRHNQVEF2djlMSlBPbURRL1pyKzhDMVIraFU3d2FqQnYrL2laVTVR?= =?utf-8?B?OFVjV24zN3Juelp1NzZPTWRVQTRzaGJYYzA1L2xUQmJjMXd5S3NFNTdHbEQ1?= =?utf-8?B?QjQ0NUpjYlFJWVJUM3FOV3B1QW1TZ3BNNllEZXlYNDlySGVyWDdwdVRqZVE5?= =?utf-8?B?MWpOZHR3Rkd6OW9QNHFmSm1WcS90dy9jZzZvRkkzOCtveHJBdXpSVitsWC8y?= =?utf-8?B?aDdMNmhhQTVtYmhweFhsY2RabTlXRVV1ZjA4K1hRQUI4UTlUSjN6aWJWYWQv?= =?utf-8?B?MTlHcUpiclhKaTBYdjhKbHFFNzZUUjFVOFFIbnMySjlQOVBrL2NHTEVhRnph?= =?utf-8?B?dXFPSUFmRXFYSFVOR3BrMXNwelBienlzN29zbVRBc2EzMzl0T1ZnV3FkZ25n?= =?utf-8?B?RXorUGJXdXVTRTh0YTliVEpmTnpFejYzQzhnTHBlcUd6VGlRYTAvNHM2alkx?= =?utf-8?B?MUQwU1hjbnpocllKcVhtUlg2amY3MFJRbW1renJBamJ0L1pRMFBEekhERVpI?= =?utf-8?B?M0xxUGVDMVJFOEdYRlp2RllkWXd6TWlnTHdrbldNZzc2d2taM04yejBvcGFw?= =?utf-8?B?TldraFkwRVNyNEVkZ3VOWnNWSkdkL2VvdWhTeXo1MFdkdjNiL3pUSkw3Z2t3?= =?utf-8?B?VXlZN3A3YWFPTk9uTHA0ZGtxd2FFcHNGcDZZVm1lcHZ0aDlVZjJVcDBlTVpZ?= =?utf-8?B?M2VuMmd0UHNCU1puMjFEOEkyR1BScU5PSjlwQTc4UEJOcWcybDZiOHRIMmt4?= =?utf-8?B?ekFWbDluNWdsTTA0QlNrVE9MNkpCczFPb0hpaWpkR3FVU1MvMU9yRm5kOUd5?= =?utf-8?B?K3lobHh3T2E1SUFna1I1aVl0Z2QzenRTVHVScjVGVFArbVdmS0RoYys3Q2hM?= =?utf-8?B?K1F5dkVSSjkweVd4Z1R1amdEdHBrSkwzME1RTFFoVnFiYnJFUWNPVEJBdkF6?= =?utf-8?B?bXp3UzIwYnlteEhLRUxsZ3U1MGZwNlRJVVB1M1N0OTQ0eUJRLzNhYldSSFFj?= =?utf-8?B?SEZmSCtEM2pRNHJhMVR6YXV6Y3d2NTd2SGF2Y2ZnTEZ1dGFEU1VpZnkvUGtI?= =?utf-8?B?TFFYMFMyajQxK0pIL1U5RHRmQjlGWnBvZGp6RzlqbG1iRjBDNlYySXRydC92?= =?utf-8?B?UmkwNDFOUE1hVmpsUzVlL0wxNkdRM0plNHpaRDdsSzJsSEhvVTNEYXNLSXlp?= =?utf-8?B?WVo0Rm5NQThkWFRhQW5jY21OVlVacDcwMzFMQXNCSHJEVWVEY3BheERNaUpI?= =?utf-8?B?VlhkbmFsOUM4OUM5L1AxSFdZRTVRODA0QWtiUHBzZkVPcUNLb1htV0FJYVBC?= =?utf-8?B?cGdBcWpnQW4zYllLRFFKSUNaRkZDZ1RFZUw4OHlROE5vU0REVXhBV2RVWm1U?= =?utf-8?B?QnFZbjdBMzZyc00zN1pTek00YytEWkxBME9zVzUvMDgxdzQwVHBrTVkxR0Nq?= =?utf-8?B?ZjcwVkhnblBFRm9vbUNGMllLMTFFbmc1VkhFU2htaDhHY1pkejNCMWZUczgz?= =?utf-8?Q?CxNCJj62AV5ED1d86n8DxHK0Gjex1XmS+rzMv1+?=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_AM0PR07MB3860F77105E2182C53F53C0293459AM0PR07MB3860eurp_"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM0PR07MB3860.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9b117e82-4cb5-46c5-14a7-08d90625f122
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Apr 2021 07:03:47.8375 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: PL29nLaZTT3OnEpR38FyzJWmSCzxfCZrEnKRJh/mD86EPi8lpOL3ub96WNNb8UuNGXiMMUfofamFN2nmNaEloQQa1bOypwzZC9Sb34Jb8AM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR07MB4305
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/xcX6xDGHzAgDh4F840D7zIQYAQ0>
Subject: Re: [Idr] [Last-Call] Genart last call review of draft-ietf-idr-ext-opt-param-11
X-BeenThere: idr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Inter-Domain Routing <idr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/idr>, <mailto:idr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/idr/>
List-Post: <mailto:idr@ietf.org>
List-Help: <mailto:idr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/idr>, <mailto:idr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 23 Apr 2021 07:03:59 -0000

Hi John,

Would it be better to say “and how the parsing of the BFP OPEN is modified”?  Because, the document is not only saying that one should do something, but HOW it is done :)

Regards,

Christer

From: Gen-art <gen-art-bounces@ietf.org> On Behalf Of John Scudder
Sent: torstai 22. huhtikuuta 2021 22.23
To: Christer Holmberg <christer.holmberg=40ericsson.com@dmarc.ietf.org>
Cc: draft-ietf-idr-ext-opt-param.all@ietf.org; idr@ietf.org; gen-art@ietf.org; last-call@ietf.org
Subject: Re: [Gen-art] [Last-Call] Genart last call review of draft-ietf-idr-ext-opt-param-11

Hi Christer,

I chose to leave the abstract alone, however I updated the introduction for -13, similar to your suggestion:


   This document updates [RFC4271] by extending, in a backward-

   compatible manner, the length of the Optional Parameters in BGP OPEN.

   This is done by using Optional Parameter Type 255 as a distinguished

   value, that indicates an extended Optional Parameters Length field

   follows and that the parsing of the BGP OPEN should be modified

   according to these procedures.  In this case the Parameter Length

   field of the individual Optional Parameters in the BGP OPEN message

   is also extended.

Thanks again for your feedback,

—John



On Apr 22, 2021, at 10:36 AM, Christer Holmberg <christer.holmberg=40ericsson.com@dmarc.ietf.org<mailto:christer.holmberg=40ericsson.com@dmarc.ietf.org>> wrote:

[External Email. Be cautious of content]


Hi John,

>> Q1: As far as I understand, the document only defines a new BGP OPEN Optional
>> Parameter Type, but does not modify/add procedures in RFC 4271. So, is the
>> document really an update to RFC 4271? And, when reading RFC 5429, I cannot
>> find any text saying that new parameter types would require an update to RFC
>> 4271. I also looked at a few other RFCs that add new values to the BGP IANA
>> registry, and they were not updating any RFC.
>
> The document modifies the way a router parses the OPEN. It doesn’t just add a new type, indeed the new type is only added as a special token to tell the router to use the new procedures.

Would it be good to explicitly indicate that?

Something like:


   "This document updates RFC 4271 by extending, in a backward-compatible

   manner, the length of the Optional Parameters in the BGP OPEN, and by

   modifying the way a router parses the OPEN."

Regards,

Christer



> Nits/editorial comments:
>
> Q2: I suggest that Section 2 is renamed to  "New Optional Parameter Type code",
> or something like that. OR, if the document really is updating RFC 4271,
> perhaps "Update to RFC 4271".

I’ve adopted your second suggestion.

> Q3: I suggest that Section 3 is renamed to "Backward Compatibility", or
> something like that.

Done.

—John
--
last-call mailing list
last-call@ietf.org<mailto:last-call@ietf.org>
https://urldefense.com/v3/__https://www.ietf.org/mailman/listinfo/last-call__;!!NEt6yMaO-gk!VekRXfZ40ytJtNm-1zuv9KJuAV1ydVIzwKvejYNnzo0ti0AsnFotbZxEvgsYrg$<https://protect2.fireeye.com/v1/url?k=d42fed95-8bb4d4d8-d42fad0e-86b568293eb5-db9bb5f1f8c980ab&q=1&e=e63753a2-3e0d-413f-a66e-06d68918c5dd&u=https%3A%2F%2Furldefense.com%2Fv3%2F__https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Flast-call__%3B%21%21NEt6yMaO-gk%21VekRXfZ40ytJtNm-1zuv9KJuAV1ydVIzwKvejYNnzo0ti0AsnFotbZxEvgsYrg%24>