Re: [IPFIX] I-D Action: draft-ietf-ipfix-text-adt-08.txt

Paul Aitken <paitken@cisco.com> Mon, 04 August 2014 08:22 UTC

Return-Path: <paitken@cisco.com>
X-Original-To: ipfix@ietfa.amsl.com
Delivered-To: ipfix@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 373161B28B2 for <ipfix@ietfa.amsl.com>; Mon, 4 Aug 2014 01:22:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.502
X-Spam-Level:
X-Spam-Status: No, score=-14.502 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, USER_IN_DEF_DKIM_WL=-7.5] 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 fY0ZE4_5FLF7 for <ipfix@ietfa.amsl.com>; Mon, 4 Aug 2014 01:21:58 -0700 (PDT)
Received: from aer-iport-2.cisco.com (aer-iport-2.cisco.com [173.38.203.52]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2F6671B28AE for <ipfix@ietf.org>; Mon, 4 Aug 2014 01:21:58 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=1395; q=dns/txt; s=iport; t=1407140518; x=1408350118; h=message-id:date:from:mime-version:to:cc:subject: references:in-reply-to:content-transfer-encoding; bh=q/fgG+hCcM3zFGRUMF58sycE15YfeQ5CF+TcI8oWZoU=; b=IMOcNwLHRGSd+d0NmXRdIGcBnbs47MgJ+o2Be9syyuZqdE8gkujDXtjq XU3XCa9lMHATHPMGA7XPhQkQTDwUWzIkjqj2taCdkJZYH8SiB3+v7dV59 OEje6+BsweQbXIUtzTYeiODaNKUPFSjkriUrWHbC6Iddrnn6zWXdgsDa/ E=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: AqoEAJZB31OtJssW/2dsb2JhbABbhDbTcAGBLXeEAwEBAQMBOEABBQsLDgoJFg8JAwIBAgFFBg0BBQIBAYg2CMRCF45qEQFQB4RLAQScBocjjTyDTjwvgQ0
X-IronPort-AV: E=Sophos;i="5.01,796,1400025600"; d="scan'208";a="131859186"
Received: from aer-iport-nat.cisco.com (HELO aer-core-2.cisco.com) ([173.38.203.22]) by aer-iport-2.cisco.com with ESMTP; 04 Aug 2014 08:21:56 +0000
Received: from [10.61.109.110] (dhcp-10-61-109-110.cisco.com [10.61.109.110]) by aer-core-2.cisco.com (8.14.5/8.14.5) with ESMTP id s748Lup8008217; Mon, 4 Aug 2014 08:21:56 GMT
Message-ID: <53DF42A4.1090506@cisco.com>
Date: Mon, 04 Aug 2014 09:21:56 +0100
From: Paul Aitken <paitken@cisco.com>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: Brian Trammell <ietf@trammell.ch>
References: <20140804050737.28858.12663.idtracker@ietfa.amsl.com> <0CBA48F7-719E-482C-968C-10C00CF57882@trammell.ch> <53DF3E8E.5050905@cisco.com> <A9D47C27-897B-4090-A6DA-AF330A05DE67@trammell.ch>
In-Reply-To: <A9D47C27-897B-4090-A6DA-AF330A05DE67@trammell.ch>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: http://mailarchive.ietf.org/arch/msg/ipfix/9x87R0NXwyHgEGYts48J5DXfruA
Cc: "ipfix@ietf.org Group" <ipfix@ietf.org>
Subject: Re: [IPFIX] I-D Action: draft-ietf-ipfix-text-adt-08.txt
X-BeenThere: ipfix@ietf.org
X-Mailman-Version: 2.1.15
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: Mon, 04 Aug 2014 08:22:01 -0000

Brian,

> On 04 Aug 2014, at 10:04, Paul Aitken <paitken@cisco.com> wrote:
>
>> Brian,
>>
>>> This revision addresses final IETF LC comments on extensibility and copy/paste errors in the examples (thanks, Paul)!
>> You missed tcpControlBits: since it's defined as unsigned16 in [IANA], you should either use unsigned16 or explain why unsigned8 is being used (eg, say that reduced-size encoding is being used).
>>
>> Recall:
>>
>>>> And tcpControlBits:
>>>>
>>>>           tcpControlBits(6)<unsigned8>[1]
>>>>
>>>> - has been revised to unsigned16 [RFC7125]. Changing this would also make Figure 2 align more neatly.
> Note that in a textual encoding, alignment is irrelevant.

That was cut-n-paste from an earlier email. I can't immediately see what 
it was referring to.

>>> It'll probably still be exported as 1 byte everywhere, but the type is indeed unsigned16
> Ah, indeed. Rev -09 then.
>
>> It would be useful to discuss reduced size encoding and show a specific example. eg, if I'm collecting ingressInterface which is nominally a u32, but I'm a small device so I only export a u8, should I export:
>>
>> ingressInterface(10)<unsigned32>[1]
>>
>> or
>>
>> ingressInterface(10)<unsigned8>[1]
> This is an interesting question but it would not appear to be in scope at all for this document.

Where would such a clarification be given if not here?

P.