Re: [Tsv-art] Tsvart last call review of draft-ietf-mpls-rfc6374-sfl-08

Stewart Bryant <sb@stewartbryant.com> Tue, 09 February 2021 16:32 UTC

Return-Path: <sb@stewartbryant.com>
X-Original-To: tsv-art@ietfa.amsl.com
Delivered-To: tsv-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7B3203A0E06; Tue, 9 Feb 2021 08:32:26 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.902
X-Spam-Level:
X-Spam-Status: No, score=-1.902 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, 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=stewartbryantcom.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 eDHKUcpnORzp; Tue, 9 Feb 2021 08:32:24 -0800 (PST)
Received: from GBR01-LO2-obe.outbound.protection.outlook.com (mail-eopbgr100056.outbound.protection.outlook.com [40.107.10.56]) (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 938753A0EA7; Tue, 9 Feb 2021 08:32:23 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lhS+qr9Am/HaV1Wme3VfZzjsDPdGb5HFS2zRn4oyJa9YFEOJYmvRjOxIunNzRMXR+YXPwXOwTeXP4H4vSI0JNxV7jH5rWfOxif7eUmtZNhf9p0YXBq+KPtDqn9y9biy4kwHWYQoKnEMj9gk/3ij5CmT06LBwGbDcMh2BHqhT70u8hnJaztIhTLokIzX2ZW1XIv31s6ipH1no2wVZ/smZCAwzIaS1qkkrPEcfhjRksQOm17m89vP06H+BVTTZC3YWD+m+/XerHd4edeLagniix98YsZnsAqGvPOgTDTdCla2jCgQRAk58O0wG28bVB+1TA0z/Ag2Zv5ezd6lP0fxmuQ==
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=1M5gRlWexWnqek+Br+EypcAftdqjRmhh1alQ4r6bIUw=; b=f/DtLOxAe3Abw5VXo6i0Pr3JSPJgN9ZO1Is13y1mQcCqi0/PwXHbAuHD5bx0Ar9sPZWZrjxejLdfuDPsGzCQdnsqvfc39lJnTX6HD8V6Fhe+37g0Ng5V6nIEE8iza4XB371MoztRQnrh13cMmCFpsZ3gRL2adEFZn5xRycBi8tkhMIVfG0Yjn3sUzU/rYWV1HInFguB9Lsob4h3OBjkplAi42CNAgmuRWMDCZZfU5D/i/hUuAG6S8fVc7xKEU1AzENqbDhTUiCtGih9trWMTPQgj5acGNA+loJ+vBV1ox69EYdqlG9dX+vdqCFi8BAGcIyF4+Uae9T3gZxCCdH8Cxg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=stewartbryant.com; dmarc=pass action=none header.from=stewartbryant.com; dkim=pass header.d=stewartbryant.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=stewartbryantcom.onmicrosoft.com; s=selector2-stewartbryantcom-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=1M5gRlWexWnqek+Br+EypcAftdqjRmhh1alQ4r6bIUw=; b=PtpBcmb+TpZzyC5JSFqKQqrOmb4yKnwlgHpsSQAsDMgxTf/Bq/osh34dWj4Y4vFdIgAT3sWCxtdKhuU0v1HmbduX5ueLiSCQVtszqRdFoytZDgi7AyzOHTVGqWa98zdO4PNk72tXlf/eBDT9P8740fkS+LU3pTARGbOeCnlAaJI=
Received: from LO2P123MB4110.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:160::13) by LO2P123MB2671.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:cd::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3825.28; Tue, 9 Feb 2021 16:32:18 +0000
Received: from LO2P123MB4110.GBRP123.PROD.OUTLOOK.COM ([fe80::4cba:dfa8:53c8:679a]) by LO2P123MB4110.GBRP123.PROD.OUTLOOK.COM ([fe80::4cba:dfa8:53c8:679a%7]) with mapi id 15.20.3825.030; Tue, 9 Feb 2021 16:32:18 +0000
From: Stewart Bryant <sb@stewartbryant.com>
To: =?utf-8?B?TWlyamEgS8O8aGxld2luZA==?= <ietf@kuehlewind.net>
CC: Stewart Bryant <sb@stewartbryant.com>, "tsv-art@ietf.org" <tsv-art@ietf.org>, "draft-ietf-mpls-rfc6374-sfl.all@ietf.org" <draft-ietf-mpls-rfc6374-sfl.all@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "mpls@ietf.org" <mpls@ietf.org>
Thread-Topic: Tsvart last call review of draft-ietf-mpls-rfc6374-sfl-08
Thread-Index: AQHW8yNvjc+IqtHuqEWhP08H2SbHE6pQG/uA
Date: Tue, 9 Feb 2021 16:32:18 +0000
Message-ID: <E05FC52A-1795-440D-B04A-8E0C3FE04B67@stewartbryant.com>
References: <161158365315.12303.5036732152845032434@ietfa.amsl.com>
In-Reply-To: <161158365315.12303.5036732152845032434@ietfa.amsl.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-mailer: Apple Mail (2.3608.120.23.2.4)
authentication-results: kuehlewind.net; dkim=none (message not signed) header.d=none;kuehlewind.net; dmarc=none action=none header.from=stewartbryant.com;
x-originating-ip: [2a00:23c5:3395:c901:9142:1783:b37:35e6]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: a5bb8b2f-e30b-4bb4-cbb7-08d8cd184479
x-ms-traffictypediagnostic: LO2P123MB2671:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <LO2P123MB2671A9E8CAA84AF80F20448EC48E9@LO2P123MB2671.GBRP123.PROD.OUTLOOK.COM>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: sL6bUHXBIoJ88z7oTrmqMMUw7Q6uyfqgetMzbI/irf0c3D3TqzVFHH1JtKkNLjVCz0c97VT/q8Lz4mDwO7zEJ2qEy71aC7IqshARVIlbOCgozghRQhjZkyh9mIYHTiwngTrYCnkHXA3Uj9mjk2DRQAdlaC2a+jA5U2ySzjw3Ox//vQ1cr6QVnylX19L/iNdd+W3vLQqcPZ1NOFUTA1QTsRp5wTWVBtMj0nb3lQKQjRPbB0gGPlAyCEg6bfTyy4zdq3FT2zwiiWi1U5bb6KjMtEvp3lTUXfgmU8PEzE7y39IC3gjxncV6MaXDzygAR0QzkLImxmIcmrAy9HsAncnazjMigGt4Bp6piYHwscvaHH/CqR6IZVAShNH3yNStRuABBWI1o+YcSKN4ckachBN7MWF3U6zgNABXtbkYa/2H+osThImQYJSiLAdJlA/eE4uVUEtv/dSqjSo1+C1HPK+/FzxIRyuRVtJekQ6JaISUQlZTJJ6NNrPzPNWZQ4fQto8fM+m/6Gr+ym9B5LEw2zbyfxN+yV8AhvtZmBdIBiYOSWHn0GERWRF7eu/ymyWILaHR34W6ysbiKjF9YTvT6mqyZw==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:LO2P123MB4110.GBRP123.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(396003)(366004)(39830400003)(376002)(136003)(346002)(86362001)(6512007)(186003)(2616005)(83380400001)(33656002)(6506007)(478600001)(66574015)(8676002)(6486002)(54906003)(6916009)(8936002)(36756003)(76116006)(64756008)(66556008)(66476007)(66946007)(66446008)(2906002)(4326008)(5660300002)(316002)(71200400001)(53546011)(45980500001); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: =?utf-8?B?ZTBjZDkzM1RXUFZBMzFWSXVHeW1aQXlsU2VwUzFndFhIV0tLMWV4Z1NZOE5M?= =?utf-8?B?RmFURUhqb0pMamoxZFY4M29UN0lKMS9NYS9FM0hwVDJMN3NlNWkzdzI0WDAx?= =?utf-8?B?T09kOHhPMHdxbEY0RVdIUEE4dVNzVzFISjhiTHNFMDRQTkpIUnAvSEIxTnNT?= =?utf-8?B?dk1YdkM1QXFHanhxMnpBZ3ZsYXZZSjAvMnlWQW9NVG5kZ3J0QTFrWTV5OUJm?= =?utf-8?B?ZGt2bmIwMEhBaUN1QkloNjU5UnhUNWNmQ2tza0QrQlNKb0xPWENQZ3lEci9S?= =?utf-8?B?SXBLVU80akpDdVhlSGxGY21qOERUaUZqZVhxUm40KzVyL0x6clB4QUoxQmtK?= =?utf-8?B?OVFtL3hmUXNBdENaVis1QnVVRVpkd0tMMCtVY2laV1BzTUYvdUZFd1dvSVoz?= =?utf-8?B?Zit5NnQ5Q1pkUmpCWTBZK2dtVnNYT2JHbXdMM2NZRTZGdU16cXRxQzBacVRP?= =?utf-8?B?Rjlac2NZWHJyZXllWTB6Y1N6SVBNbThic2hoRVpzeDFZbXFqK1JialM3ZEl6?= =?utf-8?B?ZnRVVU9kc0JKUitIcHFucS9td1Uxc2lwMXJWZGY5dlFrRlUwV3RCQzVYK2FP?= =?utf-8?B?WEZ3Um1UY2hLSGtCVUlhSzBqTUs1UFNtTnJvTU5OVlZPWi9TUlpQLzZJR2dJ?= =?utf-8?B?UkRiMWJrSVpwNnpZaGhkTFhuM0syWmxJOTIzNGVRalYyeUxRSHhHS0c3WGRn?= =?utf-8?B?QW44RVpNblJDaElKajFqUnJtSlRWYjV5MGx0TU1SUHFnbGZiSWl1b216N2Rx?= =?utf-8?B?cnRLM25pbGFRTTM5MDJZdW53TDFqblpuR2VJakQ3aENqM2gwVDFpMzFxclZt?= =?utf-8?B?R1I4cjdNdkZ2cXR4YzFsTHlKVnRHRSt1UE92dk1WWXVmdXdJcHNJdGdkNkVD?= =?utf-8?B?MHJYVjRIeGl2V1hEMEVwUi9YN0piR2RTVnBEQlZnTWdtNGNtTDVFeUFJYXRK?= =?utf-8?B?WGlmU0FTUUZmZUNzU2hiZmtRSGJFU3d2WUtjZTIvSXhIbHFZeWtXaDZwTVNI?= =?utf-8?B?VysySUFCYnFKVDdZMGxnU1JoTGJmWU93MXhqckJOQ3J1ekd6eUVKNURoaDFV?= =?utf-8?B?TTZUNUwrc3VXY2xuenkxYlZXL2RVVHErZXRsUmhjWGZNR0FhQWVYbjQ3MU9y?= =?utf-8?B?Um85N2srZ3NJa2V2R2Q5K0NrR1pjdC8ySUJ1S2hpR3J2Zi9HN2dDOVpwNHpT?= =?utf-8?B?aWRyVjN3amdEY2ZyMkFlcEFVY3A5N1NHRU9RTjg1RHdTNlNNa2JqbjdQZjEv?= =?utf-8?B?NlArc0VIOTVETHlUZTdxekhyNXk5QVVvZGxDOEZJQ08vM0dnclo4V3VjRUJk?= =?utf-8?B?eVJ2cEdFczVCOEhVV1N6bFJIQ2dISFYwc1JvRnlENnRFaHpkU3hoQSszenYr?= =?utf-8?B?NlhoZVVyV2wva0oySnc3eU5sRllqeXhRd2NWeE8xR1FOS3BqZWtSZkZKTlpK?= =?utf-8?B?U3JEYk9tMUl3Q0NyYS9sTDhLcjZmR3N0V0hnV1FOc1NUOEV2YkNuK2pUS2ZZ?= =?utf-8?B?TTZhMm80eDJxNm1ybUtJdEFnbENlOVU4eDNkOXNQQVNVYTJGbUhnNGlSeDFB?= =?utf-8?B?Q2tpbkZoVWR2d3lHWEF0L3FMWWJ3U3Rud1p2YjZRR1NFb2ZqQ2RjTnJOZW43?= =?utf-8?B?WUFoczN4ak1vWHpLQXoySFIycDZNaUdrK2ZZd1RYTzRwbUgrbk0zSHczSWxh?= =?utf-8?B?MXdrMHJSK2xoSlFFd2RiVUtIcDhjRE4rcGhmTkdUWldaOWc3Q2hMZWwzVXIv?= =?utf-8?B?SXRsS3F5N0V3d2Q1eVpkK01rd1MzY1MrYXBMemNoV2Nad3dwYk9tam8wTSti?= =?utf-8?B?SFRnaExpWjRxTmE2am9zZGtyd2lqRm50R3BERFFqYy9GQjdOK2dtTkFNOTlV?= =?utf-8?Q?4qphl57sZHg0y?=
Content-Type: text/plain; charset="utf-8"
Content-ID: <850BE7F3AE710E43BEC9789E43DD45D2@GBRP123.PROD.OUTLOOK.COM>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: stewartbryant.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: LO2P123MB4110.GBRP123.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: a5bb8b2f-e30b-4bb4-cbb7-08d8cd184479
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Feb 2021 16:32:18.3870 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 08ac3335-91dd-4a2d-90a7-b5e825494a75
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: QFv7Mv7SZ3giaTvjr7KVhPVN1bT28m8+rN6qmcS6Upqv/XH32S3o6RjZrAtwEs+lNxmVPEbxks6ktgJc/03kqQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LO2P123MB2671
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/-S8XWywa7fYsuRSfWI6qIycJmO0>
X-Mailman-Approved-At: Tue, 09 Feb 2021 09:26:25 -0800
Subject: Re: [Tsv-art] Tsvart last call review of draft-ietf-mpls-rfc6374-sfl-08
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 09 Feb 2021 16:32:27 -0000

Mirja, thank you for your review.

I have actioned the comments as described below and they are all in the markdown ready for compile and upload when I have addressed the other review comments.

> On 25 Jan 2021, at 14:07, Mirja Kühlewind via Datatracker <noreply@ietf.org> wrote:
> 
> Reviewer: Mirja Kühlewind
> Review result: On the Right Track
> 
> E.g. section 7 contains the following text:
> 
> "A number of methods are described.  The expectation is that the MPLS
>   WG possibly with the assistance of the IPPM WG will select one or
>   maybe more than one of these methods for standardization."
> 
> Is the plan still to select one? If not, why are these different formats needed?

I have changed the text as follows:

A number of methods are described. Each of these methods has different
characteristics and different processing demands on the packet forwader.
The choice of method will depend on the type of diagnostic that the operator seeks. 


> 
> Also there is this editorial note in Sec 9.1:
> 
> "Editor's Note we need to review the following in the light of further
>   thoughts on the associated signaling protocol(s).  I am fairly
>   confident that we need all the fields other than SFL Batch and SFL
>   Index.  The Index is useful in order to map between the label and
>   information associated with the FEC.  The batch is part of the
>   lifetime management process."

That a comment from an old discussion. I have moved the text.

> 
> And in this context also the purpose of section 10 is unclear to me:
> 
> "A future version of the this document will discuss the applicability
>   of the various methods to pro-active and on-demand Measurement."

Again an old comment. I have removed the text.

> 
> One procedural comment: I had to take a look at draft-ietf-mpls-sfl-framework
> to look up the concept of SFL which suggests to me that this document should
> probably be a normative reference.

That was because the framework was originally Informational and we want to
Avoid the downref. It is now published as an ST RFC so have moved the ref to normative,

> 
> Now regarding RFC8321: The marking scheme shown in section 5 seems to assume a
> RFC832-like marking. RFC8321 is mentioned (only) in section 8, however, the
> discussion there is rather unclear to me. To me it seem that RFC8321 marking is
> assumed and as such this should be stated clearly in the introduction,
> potentially with a normative reference to RFC8321.

It says in the introduction 

"An approach to mitigating these synchronization issue is described in
{{RFC8321}} in which packets are
batched by the sender and each batch is marked in some way such that
adjacent batches can be easily recognized by the receiver.”<

> 
> Some nits:
> - Sec 5: "acting as proxy data service packets

Done

> Section 4" -> ... as described
> in Section 4...? -

I have checked each of the references and I think a pointer to the parent section is fine.

> Sec 5: "This it is proposed" -> "Thus it is proposed”

Done

> - Sec
> 5: "that in the time interval being analyzed, 10 packets always flow." -> Maybe
> "that 10 packets are used in each flow in the time interval being analyzed" ...?
>   packets always flow.
> 
> 
Done

- Stewart