Re: [IPFIX] New WG Last Call fordraft-ietf-ipfix-flow-selection-tech-06.txt

Gerhard Muenz <muenz@net.in.tum.de> Sun, 19 June 2011 10:40 UTC

Return-Path: <muenz@net.in.tum.de>
X-Original-To: ipfix@ietfa.amsl.com
Delivered-To: ipfix@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 52D9A11E810C for <ipfix@ietfa.amsl.com>; Sun, 19 Jun 2011 03:40:03 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.249
X-Spam-Level:
X-Spam-Status: No, score=-2.249 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HELO_EQ_DE=0.35]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0OG2ZU7dZZ0h for <ipfix@ietfa.amsl.com>; Sun, 19 Jun 2011 03:40:02 -0700 (PDT)
Received: from mail-out1.informatik.tu-muenchen.de (mail-out1.informatik.tu-muenchen.de [131.159.0.8]) by ietfa.amsl.com (Postfix) with ESMTP id 1D34811E80D8 for <ipfix@ietf.org>; Sun, 19 Jun 2011 03:40:01 -0700 (PDT)
Received: from [192.168.2.152] (p4FCFDD59.dip.t-dialin.net [79.207.221.89]) by mail.net.in.tum.de (Postfix) with ESMTPSA id CEB412255BA7; Sun, 19 Jun 2011 12:40:29 +0200 (CEST)
Message-ID: <4DFDD218.2000501@net.in.tum.de>
Date: Sun, 19 Jun 2011 12:40:24 +0200
From: Gerhard Muenz <muenz@net.in.tum.de>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.2.17) Gecko/20110414 Thunderbird/3.1.10
MIME-Version: 1.0
To: Brian Trammell <trammell@tik.ee.ethz.ch>
References: <20110523092128.18082.9981.idtracker@ietfa.amsl.com><804B13F8F3D94A4AB18B9B01ACB68FA1044F55B0@EXCHSRV.fokus.fraunhofer.de><4DDC22C7.6070704@auckland.ac.nz><1E3C6A6E-83BC-4ABC-861E-EF46FC3F0624@tik.ee.ethz.ch> <FAE7D0F5-4B75-4ABE-AC3D-07AE763AB1A2@tik.ee.ethz.ch> <804B13F8F3D94A4AB18B9B01ACB68FA1044F5AA2@EXCHSRV.fokus.fraunhofer.de> <E9CB7DD2-3387-4668-B21F-2225C84F816C@tik.ee.ethz.ch>
In-Reply-To: <E9CB7DD2-3387-4668-B21F-2225C84F816C@tik.ee.ethz.ch>
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Cc: IPFIX Working Group <ipfix@ietf.org>
Subject: Re: [IPFIX] New WG Last Call fordraft-ietf-ipfix-flow-selection-tech-06.txt
X-BeenThere: ipfix@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IPFIX WG discussion list <ipfix.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ipfix>, <mailto:ipfix-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ipfix>
List-Post: <mailto:ipfix@ietf.org>
List-Help: <mailto:ipfix-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ipfix>, <mailto:ipfix-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 19 Jun 2011 10:40:03 -0000

>>> 5. Within section 6, it is not clear how to apply this
>>> configuration data model. Should this be harmonized with
>>> ipfix-configuration-model?
>> 
>> Yes, it shoudl be possible to use this as basis, like PASMP-TECH.
>> Should we just reference to the config-model or  do you suggest any
>> further alignment?
> 
> Since configuration per the configuration-model draft is done with
> YANG for NETCONF-compatibility, I think this should be handled as a
> model extension, with YANG defined for it. I'm neither a YANG nor a
> NETCONF expert; perhaps someone with more experience could pop in and
> say whether my suggestion makes any sense. :)

I had a brief look at section 6 without having read the entire draft.
It seems that FS_SELECTOR_ID is an enumerator for FS_TYPE. If this is
true, the following sentence is imprecise because only one of
FS_SELECTOR_ID and FS_TYPE is required for the configuration:
"A flow selection configuration consists of FS_SELECTOR_ID, FS_TYPE,
FS_SELECTOR PARAMETERS."

My recommendation is not to define any enumeration or names such as
FS_SELECTOR_ID and FS_TYPE. This should be done in the data model (i.e.
MIB or YANG draft) if needed and according to the corresponding
enumeration or naming style.

However, you should provide a description for every single parameter
that you propose. This text can be later copied into the data model draft.

Regards,
Gerhard