Re: [Last-Call] Genart last call review of draft-ietf-dots-telemetry-use-cases-11
Lars Eggert <lars@eggert.org> Thu, 20 October 2022 12:25 UTC
Return-Path: <lars@eggert.org>
X-Original-To: last-call@ietfa.amsl.com
Delivered-To: last-call@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AF88AC14F73E; Thu, 20 Oct 2022 05:25:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.107
X-Spam-Level:
X-Spam-Status: No, score=-7.107 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=eggert.org
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 O61nyeN95i_m; Thu, 20 Oct 2022 05:25:31 -0700 (PDT)
Received: from mail.eggert.org (mail.eggert.org [IPv6:2a00:ac00:4000:400:211:32ff:fe22:186f]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DE5ADC14F73F; Thu, 20 Oct 2022 05:25:30 -0700 (PDT)
Received: from smtpclient.apple (unknown [IPv6:2a00:ac00:4000:400:f1db:6c8d:4bb0:23a7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.eggert.org (Postfix) with ESMTPSA id CBE831DC2CD; Thu, 20 Oct 2022 15:25:13 +0300 (EEST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=eggert.org; s=dkim; t=1666268713; bh=8KT6JmrjQfVWml1Gjfoej/2cqb/NTxBVvB7Bypg9c9M=; h=Subject:From:In-Reply-To:Date:Cc:References:To; b=swjbpVtAtmmJzA3EJPNldpiGlafl0h1Fdg7wqujMVDC2C9Z58rE2UdxV3QM5j8J1Y PIFgZYpoQaOVcKEMITtPvDnKMkeNuZ1GjJbDdWxHCvOxwJeCrrcf+XZhFcl1Uow31G MzdYAPfOSi33ZDrEn+cGabxqKDaW5qbglVH9Nk5Q=
Content-Type: multipart/signed; boundary="Apple-Mail=_1B61423A-76E1-483D-B281-386CAF108D20"; protocol="application/pgp-signature"; micalg="pgp-sha512"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3696.120.41.1.1\))
From: Lars Eggert <lars@eggert.org>
In-Reply-To: <166373969727.11523.6129559868974729364@ietfa.amsl.com>
Date: Thu, 20 Oct 2022 15:25:13 +0300
Cc: gen-art@ietf.org, dots@ietf.org, draft-ietf-dots-telemetry-use-cases.all@ietf.org, last-call@ietf.org
Message-Id: <A0F13881-AF00-4A8C-B431-D030A994F610@eggert.org>
References: <166373969727.11523.6129559868974729364@ietfa.amsl.com>
To: Peter Yee <peter@akayla.com>
X-MailScanner-ID: CBE831DC2CD.A85A8
X-MailScanner: Not scanned: please contact your Internet E-Mail Service Provider for details
X-MailScanner-From: lars@eggert.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/last-call/iMEPMvxe7_C-7kELpz7d1kS8rqo>
Subject: Re: [Last-Call] Genart last call review of draft-ietf-dots-telemetry-use-cases-11
X-BeenThere: last-call@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: IETF Last Calls <last-call.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/last-call>, <mailto:last-call-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/last-call/>
List-Post: <mailto:last-call@ietf.org>
List-Help: <mailto:last-call-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/last-call>, <mailto:last-call-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 20 Oct 2022 12:25:35 -0000
Peter, thank you for your review. I have entered a No Objection ballot for this document. Lars > On 2022-9-21, at 8:54, Peter Yee via Datatracker <noreply@ietf.org> wrote: > > Reviewer: Peter Yee > Review result: Ready with Nits > > I am the assigned Gen-ART reviewer for this draft. The General Area > Review Team (Gen-ART) reviews all IETF documents being processed > by the IESG for the IETF Chair. Please treat these comments just > like any other last call comments. > > For more information, please see the FAQ at > > <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. > > Document: draft-ietf-dots-telemetry-use-cases-11 > Reviewer: Peter Yee > Review Date: 2022-09-20 > IETF LC End Date: 2022-09-20 > IESG Telechat date: Not scheduled for a telechat > > Summary: This document gives use cases showing how RFC 9244 can be employed for > to convey DOTS telemetry. It seems perfectly fine as an informational adjunct > to RFC 9244, giving more involved examples. > > Major issues: None. > > Minor issues: None. > > Nits/editorial comments: > > Page 3, section 3, 1st paragraph: insert “the” before “DOTS telemetry > specifications” and change “specifications” to “specification”. > > Page 4, section 3.1.1, 1st paragraph, 1st sentence: delete “such”. Change “is” > to “are”. > > Page 4, section 3.1.1, 1st paragraph, 2nd sentence: change “recent” to > “recently”. Change “Tps” to “Tbps”, unless you believe that 1 transaction per > second is a lot of traffic. One tablespoon might be. ;-) > > Page 7, 1st paragraph, 2nd sentence: change “identifies” to “identify”. Change > “of” to “about”. > > Page 7, 1st paragraph, 3rd sentence: delete the first two commas (bracketing > “then”). > > Page 7, section 3.1.2, 1st paragraph, 2nd sentence: change “under attack time” > to “at the time of an attack”. > > Page 9, 1st paragraph, 2nd sentence: change “of” to “on”. > > Page 9, 1st paragraph, 4th sentence: delete “each”. > > Page 9, 1st paragraph, 5th sentence: change “atribute” to “attribute”. > > Page 10, section 3.1.3, 1st paragraph, 1st sentence: delete “an”. > > Page 12, 1st paragraph, 2nd sentence: change “of” to “about”. Delete “a” after > “using”. > > Page 12, 1st paragraph, 3rd sentence: delete “On the other hands,” and > capitalize the ‘t’ in “the”. Insert “the” before “volume”. > > Page 12, section 3.1.4, 1st paragraph, 1st sentence: delete the comma after > “Short”. Change “internet” to “Internet”. > > Page 12, section 3.1.4, 1st paragraph, 2nd sentence: considering inserting > “salient” before “feature”. Insert “it” before “start” and change “start” to > “starts”. Change “go” to “goes” in both occurrences in the sentence. Insert > “then” before “ back to maximum”. > > Page 12, section 3.1.4, 1st paragraph, 3rd sentence: delete “for them”. Insert > “such” before “an attack”. Change “by” to “using a”. Change “it” to “this”. > > Page 12, section 3.1.4, 2nd paragraph, 2nd sentence: insert “the” before > “attack traffic”. > > Page 14, 1st paragraph, 3rd sentence: change “rate-limit” to “rate-limiting of”. > > Page 14, 2nd paragraph: change “gatherd” to “gathered”. > > Page 15, section 3.1.5, 1st paragraph, 3rd sentence: change “suspecious” to > “suspicious”. > > Page 15, section 3.1.5, 1st paragraph, 4th sentence: delete “an”. > > Page 15, 3rd paragraph, 2nd sentence: change “as below” to “that”. > > Page 15, 3rd paragraph, 3rd sentence: add a comma after “total attack > connection”. > > Page 17, Figure 11: change “vulnerbilities” to “vulnerabilities” twice. > > Page 19, 1st paragraph, 4th sentence: delete the two commas bracketing “then”. > Change the first space after “e.g.” to a comma. > > Page 19, section 3.2, 1st paragraph, 2nd sentence: change “counter measure” to > “countermeasure”. > > Page 19, section 3.2, 2nd paragraph, 1st sentence: change “counter measure” to > “countermeasures” (note the plural). > > Page 19, section 3.2, 2nd paragraph, last sentence: append a comma after > “”total attack traffic percentiles”. Change “detail” to “details”. > > Page 21, 1st paragraph, 2nd sentence: insert “the” before > “total-pipe-capacity”. Insert “the” before “DOTS”. > > Page 22, 1st paragraph, 1st sentence: change “mitugation” to “mitigation”. > > Page 22, 1st paragraph, 1st sentence: change “reports” to “report”. > > Page 22, 1st paragraph, 2nd sentence: change “system” to “systems”, presuming > that there can be more than one DMS reporting to the orchestrator. For the > words “sends them”, what does “them” mean? I’m assuming that the orchestrator > creates a single, integrated, deduplicated report, so I’m not sure what “them” > is. > > Page 22, section 3.3.1, 1st paragraph, 1st sentence: change “internet” to > “Internet”. > > Page 22, section 3.3.1, 1st paragraph, 2nd sentence: delete “On the other hand,” > > Page 22, section 3.3.1, 2nd paragraph, 1st sentence: on the assumption that > multiple flow connectors are allowed, as shown in Figure 16: change > “collector’s” to “collectors’”. > > Page 22, section 3.3.1, 2nd paragraph, 2nd sentence: change “collector” to > “collectors” on the same assumption as the previous comment. If that assumption > is incorrect, ignore both comments. Also considering changing the figure to > have a single flow collector shown in that case. > > Page 25, 1st paragraph, 3rd sentence: delete the two commas bracketing “then”. > > Page 25, section 3.3.2, 2nd sentence: insert “a” or “the” before “baseline”. > Insert “the” before “DMSes”. > > Page 25, section 3.3.2, 3rd sentence: change “collector” to “collectors”. > > Page 25, Figure 18: I’m not sure what “[ Dst ]” means. It doesn’t appear > elsewhere in this specification or in RFC 9244. Perhaps “destination”? > > Page 26, 1st paragraph, 1st sentence: I can’t parse this sentence. Consider > rewriting it. > > > > -- > last-call mailing list > last-call@ietf.org > https://www.ietf.org/mailman/listinfo/last-call
- [Last-Call] Genart last call review of draft-ietf… Peter Yee via Datatracker
- Re: [Last-Call] Genart last call review of draft-… H Y
- Re: [Last-Call] Genart last call review of draft-… Lars Eggert