[Gen-art] Review: draft-ietf-ipfix-flow-selection-tech-10.txt
"Joel M. Halpern" <jmh@joelhalpern.com> Fri, 23 March 2012 17:47 UTC
Return-Path: <jmh@joelhalpern.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EB82321F8609 for <gen-art@ietfa.amsl.com>; Fri, 23 Mar 2012 10:47:53 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.265
X-Spam-Level:
X-Spam-Status: No, score=-102.265 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, IP_NOT_FRIENDLY=0.334, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id HursTo5oQMe8 for <gen-art@ietfa.amsl.com>; Fri, 23 Mar 2012 10:47:50 -0700 (PDT)
Received: from morbo.mail.tigertech.net (morbo.mail.tigertech.net [67.131.251.54]) by ietfa.amsl.com (Postfix) with ESMTP id 22DE521F8602 for <gen-art@ietf.org>; Fri, 23 Mar 2012 10:47:50 -0700 (PDT)
Received: from mailc2.tigertech.net (mailc2.tigertech.net [208.80.4.156]) by morbo.tigertech.net (Postfix) with ESMTP id 8127B55805E for <gen-art@ietf.org>; Fri, 23 Mar 2012 10:47:49 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by mailc2.tigertech.net (Postfix) with ESMTP id 111F618035C; Fri, 23 Mar 2012 10:47:48 -0700 (PDT)
X-Virus-Scanned: Debian amavisd-new at c2.tigertech.net
Received: from [10.216.6.78] (unknown [93.158.43.3]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mailc2.tigertech.net (Postfix) with ESMTPSA id 27B1218035A; Fri, 23 Mar 2012 10:47:46 -0700 (PDT)
Message-ID: <4F6CB741.804@joelhalpern.com>
Date: Fri, 23 Mar 2012 13:47:45 -0400
From: "Joel M. Halpern" <jmh@joelhalpern.com>
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:10.0.2) Gecko/20120216 Thunderbird/10.0.2
MIME-Version: 1.0
To: "A. Jean Mahoney" <mahoney@nostrum.com>, draft-ietf-ipfix-flow-selection-tech@tools.ietf.org
References: <4F6BB53A.3070609@nostrum.com>
In-Reply-To: <4F6BB53A.3070609@nostrum.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: "Romascanu, Dan (Dan)" <dromasca@avaya.com>, gen-art@ietf.org
Subject: [Gen-art] Review: draft-ietf-ipfix-flow-selection-tech-10.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 23 Mar 2012 17:47:54 -0000
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-ipfix-flow-selection-tech-10.txt Flow Selection Techniques Reviewer: Joel M. Halpern Review Date: 24-March-2012 IETF LC End Date: 11-April-2012 IESG Telechat date: N/A Summary: This document is nearly ready for publication as a proposed standard. Major issues: Should the hash functions identified in 6.1 for hash based filtering (BOB, IPSX, CRC-323) have references to their definitions? Minor issues: The tracker does not indicate a document shepherd for this document. If compliance is a big issue, then this document is under-specified. FOr example, it says in section 5.1 "In order to be compliant with this document, at least the Property Match Filtering MUST be implemented." However, Property Match Filtering as defined in section 5.1.1 is a range of behaviors, so it is unclear which property tests must be supported (any, all possible?) in order to be compliant. Nits/editorial comments:
- [Gen-art] A *new* batch of IETF LC reviews - 2012… A. Jean Mahoney
- [Gen-art] Review: draft-ietf-ipfix-flow-selection… Joel M. Halpern
- [Gen-art] Review: draft-ietf-ipfix-flow-selection… Joel M. Halpern
- Re: [Gen-art] Review: draft-ietf-ipfix-flow-selec… Benoit Claise
- Re: [Gen-art] Review: draft-ietf-ipfix-flow-selec… Jari Arkko