[GROW] Genart Telechat review: draft-ietf-grow-filtering-threats-07
Robert Sparks <rjsparks@nostrum.com> Mon, 17 August 2015 17:48 UTC
Return-Path: <rjsparks@nostrum.com>
X-Original-To: grow@ietfa.amsl.com
Delivered-To: grow@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D3D201ACDE4; Mon, 17 Aug 2015 10:48:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.909
X-Spam-Level:
X-Spam-Status: No, score=-1.909 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=ham
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 9Gp2j-ZjKm7h; Mon, 17 Aug 2015 10:48:12 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (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 A8ACF1ACDBC; Mon, 17 Aug 2015 10:48:12 -0700 (PDT)
Received: from unnumerable.local (pool-71-170-237-80.dllstx.fios.verizon.net [71.170.237.80]) (authenticated bits=0) by nostrum.com (8.15.2/8.14.9) with ESMTPSA id t7HHmBLF080428 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=OK); Mon, 17 Aug 2015 12:48:12 -0500 (CDT) (envelope-from rjsparks@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host pool-71-170-237-80.dllstx.fios.verizon.net [71.170.237.80] claimed to be unnumerable.local
Message-ID: <55D21E56.7040608@nostrum.com>
Date: Mon, 17 Aug 2015 12:48:06 -0500
From: Robert Sparks <rjsparks@nostrum.com>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
MIME-Version: 1.0
To: General Area Review Team <gen-art@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, grow@ietf.org, draft-ietf-grow-filtering-threats.all@ietf.org
References: <558B3AEE.2010009@nostrum.com>
In-Reply-To: <558B3AEE.2010009@nostrum.com>
Content-Type: multipart/alternative; boundary="------------020800060909050209070503"
Archived-At: <http://mailarchive.ietf.org/arch/msg/grow/11KbF6bwd27oTXKTtsjiVUuv7uU>
Subject: [GROW] Genart Telechat review: draft-ietf-grow-filtering-threats-07
X-BeenThere: grow@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Grow Working Group Mailing List <grow.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/grow>, <mailto:grow-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/grow/>
List-Post: <mailto:grow@ietf.org>
List-Help: <mailto:grow-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/grow>, <mailto:grow-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 17 Aug 2015 17:48:16 -0000
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 wait for direction from your document shepherd or AD before posting a new version of the draft. For more information, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Document: draft-ietf-grow-filtering-threats-07 Reviewer: Robert Sparks Review Date: 17 Aug 2015 IETF LC End Date: 2 Jul 2015 IESG Telechat date: 20 Aug 2015 Summary: Ready with nits (but these nits may be discuss worthy) Nits/editorial comments: Thanks for removing the text suggesting when operators might charge each other. I still find the document ambiguous about what it's stating as the consensus of the IETF. In the places you changed how you used "we", you did the very thing I asked to to resist - you say things now like "The authors recommend". What is the IETF saying? Is it that the IETF agrees that's what the authors recommend? Or is the IETF recommending this. This needs to be edited to speak _only_ in terms of what the IETF recommends. I'm classifying this as a NIT since it's editorial, but I'm worried it's more than that. I encourage the IESG to consider whether this is a bigger issue. You still draw this conclusion: "The authors observe that proactive approaches can be complex to implement and can lead to undesired effects, and thus conclude that the reactive approach is the more reasonable recommendation to deal with unexpected flows." What is the IETF consensus position on what is more reasonable, and is it even necessary for the IETF to recommend one approach over the other. Why is it not sufficient to simply document the considerations with each approach and stop there? Not much was done with my last suggestion. I still think the document needs a strong editorial revision along the lines suggested below. On 6/24/15 6:19 PM, Robert Sparks wrote: > I am the assigned Gen-ART reviewer for this draft. For background on > Gen-ART, please see the FAQ at > > <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. > > Please resolve these comments along with any other Last Call comments > you may receive. > > Document: draft-ietf-grow-filtering-threats-06 > Reviewer: Robert Sparks > Review Date: 24-Jun-2015 > IETF LC End Date: 2-Jul-2015 > IESG Telechat date: not yet scheduled for a telechat > > Summary: Ready with nits > > From looking at the document history and list archives, this > document's been around for some time, and has had some editorial push > already. > The unintended consequences it highlights are interesting, and it will > be useful to operators to know these possible causes of unexpected > behavior. > > I encourage another strong editing pass before publication. > > This is being published as an IETF-stream document. When published it > reflects IETF consensus. > There are places in the text that I think are problematic given that > status. The issues are editorial, and I expect they will be easy to > address. > > The document uses "we" frequently. Originally, that meant the authors. > It's ambiguous what it means in an IETF-stream document. I suggest > editing out all occurrences. Try to avoid simply changing "we" to "the > authors" - find a way to reflect what the IETF is saying here. > > Is the last paragraph of 4.1 an IETF consensus position on how > operators might charge one another? It would be good to find a way to > word this that look more like statements of fact and less like > charging advice. > > The document draws some conclusions that I think are unnecessary. For > instance, "Therefore, we conclude that the reactive approach is the > more reasonable recommendation to deal with unexpected flows." Why > does the IETF need to say that (and is it an IETF consensus > statement)? It would be enough, I think, to reduce the discussion in > these sections to calling out the issues with each approach. > > Please simplify the sentences, and avoid passive construction. For > instance, "It can be considered problematic to be causing unexpected > traffic flows in other ASes." can be much shorter. After you do that, > I think you'll find it easier to identify and collapse sections of > redundant text. > > RjS >
- [GROW] Genart LC review: draft-ietf-grow-filterin… Robert Sparks
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Randy Bush
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Job Snijders
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Bill Woodcock
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Pierre Francois
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Pierre Francois
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Randy Bush
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Pierre Francois
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Randy Bush
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… bruno.decraene
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Christopher Morrow
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… juancamilo.cardona
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Christopher Morrow
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Camilo Cardona
- Re: [GROW] Genart LC review: draft-ietf-grow-filt… Christopher Morrow
- [GROW] Genart Telechat review: draft-ietf-grow-fi… Robert Sparks
- Re: [GROW] Genart Telechat review: draft-ietf-gro… Jari Arkko
- Re: [GROW] Genart Telechat review: draft-ietf-gro… Robert Sparks
- Re: [GROW] Genart Telechat review: draft-ietf-gro… Jari Arkko
- Re: [GROW] Genart Telechat review: draft-ietf-gro… Robert Sparks
- Re: [GROW] Genart Telechat review: draft-ietf-gro… Peter Schoenmaker