Re: [ippm] John Scudder's No Objection on draft-ietf-ippm-explicit-flow-measurements-03: (with COMMENT)
Giuseppe Fioccola <giuseppe.fioccola@huawei.com> Fri, 26 May 2023 08:00 UTC
Return-Path: <giuseppe.fioccola@huawei.com>
X-Original-To: ippm@ietfa.amsl.com
Delivered-To: ippm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B0C79C14CE4C; Fri, 26 May 2023 01:00:37 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.195
X-Spam-Level:
X-Spam-Status: No, score=-4.195 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H5=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id RISoHR4Dxi8O; Fri, 26 May 2023 01:00:35 -0700 (PDT)
Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D3865C15108C; Fri, 26 May 2023 01:00:34 -0700 (PDT)
Received: from frapeml100006.china.huawei.com (unknown [172.18.147.207]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4QSHKf3RCHz6J6hc; Fri, 26 May 2023 15:55:50 +0800 (CST)
Received: from frapeml500006.china.huawei.com (7.182.85.219) by frapeml100006.china.huawei.com (7.182.85.201) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Fri, 26 May 2023 10:00:31 +0200
Received: from frapeml500006.china.huawei.com ([7.182.85.219]) by frapeml500006.china.huawei.com ([7.182.85.219]) with mapi id 15.01.2507.023; Fri, 26 May 2023 10:00:31 +0200
From: Giuseppe Fioccola <giuseppe.fioccola@huawei.com>
To: John Scudder <jgs@juniper.net>, The IESG <iesg@ietf.org>
CC: "draft-ietf-ippm-explicit-flow-measurements@ietf.org" <draft-ietf-ippm-explicit-flow-measurements@ietf.org>, "ippm-chairs@ietf.org" <ippm-chairs@ietf.org>, "ippm@ietf.org" <ippm@ietf.org>, "marcus.ihlar@ericsson.com" <marcus.ihlar@ericsson.com>
Thread-Topic: John Scudder's No Objection on draft-ietf-ippm-explicit-flow-measurements-03: (with COMMENT)
Thread-Index: AQHZjoeGeZnomeAYcEyi4Efja1Mwgq9rIF0A
Date: Fri, 26 May 2023 08:00:31 +0000
Message-ID: <f9ce0f2ea03547ee94f7ef68375766fd@huawei.com>
References: <168496404944.21984.14328246653038787134@ietfa.amsl.com>
In-Reply-To: <168496404944.21984.14328246653038787134@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.81.200.79]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/ippm/AryWXYlx_pPwWstmBhuQa2eJZlg>
Subject: Re: [ippm] John Scudder's No Objection on draft-ietf-ippm-explicit-flow-measurements-03: (with COMMENT)
X-BeenThere: ippm@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: IETF IP Performance Metrics Working Group <ippm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ippm>, <mailto:ippm-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ippm/>
List-Post: <mailto:ippm@ietf.org>
List-Help: <mailto:ippm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ippm>, <mailto:ippm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 26 May 2023 08:00:37 -0000
Hi John, Thank you for your review. Please see inline [GF]. Regards, Giuseppe -----Original Message----- From: John Scudder via Datatracker <noreply@ietf.org> Sent: Wednesday, May 24, 2023 11:34 PM To: The IESG <iesg@ietf.org> Cc: draft-ietf-ippm-explicit-flow-measurements@ietf.org; ippm-chairs@ietf.org; ippm@ietf.org; marcus.ihlar@ericsson.com; marcus.ihlar@ericsson.com Subject: John Scudder's No Objection on draft-ietf-ippm-explicit-flow-measurements-03: (with COMMENT) John Scudder has entered the following ballot position for draft-ietf-ippm-explicit-flow-measurements-03: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ for more information about how to handle DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-ippm-explicit-flow-measurements/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- # John Scudder, RTG AD, comments for draft-ietf-ippm-explicit-flow-measurements-03 CC @jgscudder Thanks for this document. While I am not sufficiently expert in the subject area to give it a deep review, I trust this has been done by others. I do have a few questions and comments that I hope may help, below. I support Jim Guichard's DISCUSS position. ## COMMENT ### Section 1 the encrypted transport-layer headers (e.g. QUIC, TCP). Since when are TCP headers encrypted? [GF]: We will revise this sentence. We meant the possible use of TLS with TCP. ### Section 3.1 There is a list of bullet items that talk about "by setting the T bit", e.g. * the client selects, generates and consequently transmits a first train of packets, by setting the T bit to 1; The way the sentence is constructed, it literally means "the client causes the train of packets to be generated and transmitted by means of setting the T bit to 1". I doubt that's what you mean. Possibly you mean something like "it sets the T bit to 1 to identify packets in this train"? [GF]: Thank you for the suggestion. We will change these statements. ### Section 3.1.2 The reflection counter is first introduced by mentioning that the reflection counter is unlocked to start counting incoming marked packets that will be reflected later; This makes for a rather bumpy experience for the first-time reader. I suggest introducing this counter somehow before referencing it. You might also mention that as part of initialization, the reflection counter is locked. [GF]: Ok we can introduce the reflection counter before. It makes sense in section 3.1. I also have a hard time being sure I've understood this correctly: The generation token counter should be capped to limit the effects of a subsequent sudden reduction in the other endpoint's packet rate that could prevent that endpoint from reflecting collected packets. The most conservative cap value is 1. I presume "capped" is used in the normal sense of not being allowed to exceed a certain value. I guess yes, 1 is a conservative value in a way, after all the only lower value is zero and that wouldn't make a lot of sense. As written the text has the odor of recommending the value of 1 without actually doing so, it's just that "conservative" sounds like approval in this context. Is all of that intentional? [GF]: I think we can also explicitly recommend the value of 1. ### Section 3.3 Surely "IP/IPv6" is the wrong terminology -- should be either "IP" (meaning, IPv4 or IPv6) or "IPv4/IPv6"? [GF]: Ok we will replace it. ### Section 3.3 Concerning the Unreported Loss counter you have, If the protocol is able to rescind the loss determination later, a positive Unreported Loss counter may be decremented due to the rescission, but it should not become negative due to the rescission. and later, (so Unreported Loss counter may become negative when a packet with L=1 is sent after a partial packet has been lost) Is my conclusion correct, that negative values are permitted and should be supported, but only in the latter case and not the former? [GF]: Yes, but I will modify the first sentence to avoid misunderstanding. We can say that in general it should not become negative due to the rescission, but it can happen in few cases. ## NITS ### Section 3.5, Section 4 s/this draft/this document/ [GF]: Ok ### Section 7 s/gleamed/gleaned/ [GF]: Ok ## Notes This review is in the ["IETF Comments" Markdown format][ICMF], You can use the [`ietf-comments` tool][ICT] to automatically convert this review into individual GitHub issues. [ICMF]: https://github.com/mnot/ietf-comments/blob/main/format.md [ICT]: https://github.com/mnot/ietf-comments
- [ippm] John Scudder's No Objection on draft-ietf-… John Scudder via Datatracker
- Re: [ippm] John Scudder's No Objection on draft-i… Giuseppe Fioccola