Re: [ippm] Erik Kline's No Objection on draft-ietf-ippm-capacity-metric-method-06: (with COMMENT)

"MORTON, ALFRED C (AL)" <acm@research.att.com> Wed, 24 February 2021 14:12 UTC

Return-Path: <acm@research.att.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 791BB3A1625; Wed, 24 Feb 2021 06:12:19 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.896
X-Spam-Level:
X-Spam-Status: No, score=-1.896 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_BLOCKED=0.001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 i8dJUU5kdEdc; Wed, 24 Feb 2021 06:12:18 -0800 (PST)
Received: from mx0a-00191d01.pphosted.com (mx0b-00191d01.pphosted.com [67.231.157.136]) (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 5EC083A1624; Wed, 24 Feb 2021 06:12:17 -0800 (PST)
Received: from pps.filterd (m0049462.ppops.net [127.0.0.1]) by m0049462.ppops.net-00191d01. (8.16.0.43/8.16.0.43) with SMTP id 11OE6CsU022367; Wed, 24 Feb 2021 09:12:11 -0500
Received: from tlpd255.enaf.dadc.sbc.com (sbcsmtp3.sbc.com [144.160.112.28]) by m0049462.ppops.net-00191d01. with ESMTP id 36w7ebxb99-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 24 Feb 2021 09:12:11 -0500
Received: from enaf.dadc.sbc.com (localhost [127.0.0.1]) by tlpd255.enaf.dadc.sbc.com (8.14.5/8.14.5) with ESMTP id 11OEC97d127316; Wed, 24 Feb 2021 08:12:10 -0600
Received: from zlp30494.vci.att.com (zlp30494.vci.att.com [135.46.181.159]) by tlpd255.enaf.dadc.sbc.com (8.14.5/8.14.5) with ESMTP id 11OEC2aM127149 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Wed, 24 Feb 2021 08:12:03 -0600
Received: from zlp30494.vci.att.com (zlp30494.vci.att.com [127.0.0.1]) by zlp30494.vci.att.com (Service) with ESMTP id D307C4009E75; Wed, 24 Feb 2021 14:12:02 +0000 (GMT)
Received: from clph811.sldc.sbc.com (unknown [135.41.107.12]) by zlp30494.vci.att.com (Service) with ESMTP id 9D0844009E77; Wed, 24 Feb 2021 14:12:02 +0000 (GMT)
Received: from sldc.sbc.com (localhost [127.0.0.1]) by clph811.sldc.sbc.com (8.14.5/8.14.5) with ESMTP id 11OEC2R8115877; Wed, 24 Feb 2021 08:12:02 -0600
Received: from mail-azure.research.att.com (mail-azure.research.att.com [135.207.255.18]) by clph811.sldc.sbc.com (8.14.5/8.14.5) with ESMTP id 11OEBtNv114442; Wed, 24 Feb 2021 08:11:55 -0600
Received: from exchange.research.att.com (njbdcas1.research.att.com [135.197.255.61]) by mail-azure.research.att.com (Postfix) with ESMTP id 1EB6310A18D6; Wed, 24 Feb 2021 09:11:55 -0500 (EST)
Received: from njmtexg5.research.att.com ([fe80::b09c:ff13:4487:78b6]) by njbdcas1.research.att.com ([fe80::8c6b:4b77:618f:9a01%11]) with mapi id 14.03.0468.000; Wed, 24 Feb 2021 09:12:12 -0500
From: "MORTON, ALFRED C (AL)" <acm@research.att.com>
To: Erik Kline <ek.ietf@gmail.com>, The IESG <iesg@ietf.org>
CC: "draft-ietf-ippm-capacity-metric-method@ietf.org" <draft-ietf-ippm-capacity-metric-method@ietf.org>, "ippm-chairs@ietf.org" <ippm-chairs@ietf.org>, "ippm@ietf.org" <ippm@ietf.org>, Ian Swett <ianswett@google.com>, "tpauly@apple.com" <tpauly@apple.com>
Thread-Topic: Erik Kline's No Objection on draft-ietf-ippm-capacity-metric-method-06: (with COMMENT)
Thread-Index: AQHXCneFYUzVXwie1Eytg1RL/4aBG6pnTeXA
Date: Wed, 24 Feb 2021 14:12:11 +0000
Message-ID: <4D7F4AD313D3FC43A053B309F97543CF014769FD76@njmtexg5.research.att.com>
References: <161414861268.8652.15719785883972669316@ietfa.amsl.com>
In-Reply-To: <161414861268.8652.15719785883972669316@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [24.148.42.167]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.369, 18.0.761 definitions=2021-02-24_04:2021-02-24, 2021-02-24 signatures=0
X-Proofpoint-Spam-Details: rule=outbound_policy_notspam policy=outbound_policy score=0 malwarescore=0 bulkscore=0 clxscore=1011 mlxscore=0 suspectscore=0 adultscore=0 impostorscore=0 lowpriorityscore=0 priorityscore=1501 mlxlogscore=999 spamscore=0 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000 definitions=main-2102240109
Archived-At: <https://mailarchive.ietf.org/arch/msg/ippm/5bLRt4DZTFGYCd9nCPlwzPLETPI>
Subject: Re: [ippm] Erik Kline's No Objection on draft-ietf-ippm-capacity-metric-method-06: (with COMMENT)
X-BeenThere: ippm@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 24 Feb 2021 14:12:20 -0000

Hi Erik, thanks for your review and comments.
Please see replies below [acm], all changes have been implemented in the working version.

regards,
Al

> -----Original Message-----
> From: Erik Kline via Datatracker [mailto:noreply@ietf.org]
> Sent: Wednesday, February 24, 2021 1:37 AM
> To: The IESG <iesg@ietf.org>
> Cc: draft-ietf-ippm-capacity-metric-method@ietf.org; ippm-chairs@ietf.org;
> ippm@ietf.org; Ian Swett <ianswett@google.com>om>; tpauly@apple.com;
> tpauly@apple.com
> Subject: Erik Kline's No Objection on draft-ietf-ippm-capacity-metric-
> method-06: (with COMMENT)
> 
> Erik Kline has entered the following ballot position for
> draft-ietf-ippm-capacity-metric-method-06: No Objection
> 
...
> 
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
> 
> [[ comments ]]
> 
> [ section 4 ]
> 
> * RFC 6438 isn't only about flow label treatment at Tunnel End Points,
> since
>   other devices in the network can do ECMP where the flow label is part of
>   the flow.
> 
>   Maybe just a full stop after "when routers have complied with RFC6438
>   guidelines."?
[acm] 
This change is fine with me.

> 
> 
> [[ questions ]]
> 
> [ sections 5.6/6.6/7.4 ]
> 
> * Should the "megabit = 1,000,000 bits" text from section 6.6 be also used
>   in sections 5.6 and 7.4, or even called out separately earlier on?
[acm] 
Thanks for your suggestion.
I added " (which is 1,000,000 bits per second to avoid any confusion)"
in 5.6 and 7.4. (this may have been an earlier comment intended to apply everywhere).

> 
>   (Maybe it's just my own experience, but I've found that, while "mibi" 100%
>   of the time means base 2, "mega" only mostly means base 10 and occasionally
>   can still be interpreted as base 2.)
[acm] 
We share that experience. Others might ask, "what's a kibibit?"

> 
> 
> [[ nits ]]
> 
> [ section 2 ]
> 
> * "Also, to foster the development..."
> 
>   This appears to be a sentence fragment rather than a grammatically
> correct
>   sentence.  I assume the point is to say that fostering the development
> ...
>   is also goal of the document.
[acm] 
Thanks, I re-wrote that sentence as follows:

Fostering the development of protocol support for this metric and method of measurement is also a goal of this memo (all active testing protocols currently defined by the IPPM WG are UDP-based, meeting a key requirement of these methods).


> 
> * "The supporting" -> "Supporting the", might read more naturally?
[acm] 
It might, except we would then have two sentences in a row (with the change above) beginning:
Fostering the...
Supporting the...

I left "The supporting" as-is.

> 
> [ section 5.4 ]
> 
> * "The statistics used to to summarize" -> "The statistics used to summarize"
[acm] 
Thanks

> 
> [ section 6.5 ]
> 
> * "so that is" -> "so that it is"?
[acm] 
Thanks

> 
>