Re: Last Call: <draft-ietf-behave-ipfix-nat-logging-06.txt> (IPFIX Information Elements for logging NAT Events)

Paul Aitken <paitken@brocade.com> Wed, 02 March 2016 13:59 UTC

Return-Path: <paitken@Brocade.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 85EBD1B2AA8; Wed, 2 Mar 2016 05:59:06 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, SPF_PASS=-0.001] 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 hAh7yvQ8oi8S; Wed, 2 Mar 2016 05:59:05 -0800 (PST)
Received: from mx0a-000f0801.pphosted.com (mx0a-000f0801.pphosted.com [IPv6:2620:100:9001:7a::1]) (using TLSv1.2 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 005441B2AA6; Wed, 2 Mar 2016 05:59:04 -0800 (PST)
Received: from pps.filterd (m0000542.ppops.net [127.0.0.1]) by mx0a-000f0801.pphosted.com (8.15.0.59/8.15.0.59) with SMTP id u22DtMlu020289; Wed, 2 Mar 2016 05:59:03 -0800
Received: from brmwp-exmb12.corp.brocade.com ([208.47.132.227]) by mx0a-000f0801.pphosted.com with ESMTP id 21bc3145bs-3 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NOT); Wed, 02 Mar 2016 05:59:02 -0800
Received: from EMEAWP-EXMB11.corp.brocade.com (172.29.11.85) by BRMWP-EXMB12.corp.brocade.com (172.16.59.130) with Microsoft SMTP Server (TLS) id 15.0.1104.5; Wed, 2 Mar 2016 06:58:56 -0700
Received: from [192.168.1.68] (172.16.181.50) by EMEAWP-EXMB11.corp.brocade.com (172.29.11.85) with Microsoft SMTP Server (TLS) id 15.0.1104.5; Wed, 2 Mar 2016 14:58:53 +0100
Subject: Re: Last Call: <draft-ietf-behave-ipfix-nat-logging-06.txt> (IPFIX Information Elements for logging NAT Events)
To: "Senthil Sivakumar (ssenthil)" <ssenthil@cisco.com>, "draft-ietf-behave-ipfix-nat-logging@ietf.org" <draft-ietf-behave-ipfix-nat-logging@ietf.org>
References: <56BC9C63.3080404@brocade.com> <D2E3BF4A.168243%ssenthil@cisco.com> <56BF1786.5090607@brocade.com> <D2E764D8.168A91%ssenthil@cisco.com>
From: Paul Aitken <paitken@brocade.com>
Message-ID: <56D6F19D.6020102@brocade.com>
Date: Wed, 02 Mar 2016 13:58:53 +0000
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:38.0) Gecko/20100101 Thunderbird/38.5.1
MIME-Version: 1.0
In-Reply-To: <D2E764D8.168A91%ssenthil@cisco.com>
Content-Type: multipart/alternative; boundary="------------020406030704080600010302"
X-Originating-IP: [172.16.181.50]
X-ClientProxiedBy: hq1wp-excas12.corp.brocade.com (10.70.38.22) To EMEAWP-EXMB11.corp.brocade.com (172.29.11.85)
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:, , definitions=2016-03-02_05:, , signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1601100000 definitions=main-1603020252
Archived-At: <http://mailarchive.ietf.org/arch/msg/ietf/SeYRSi6PyxLFBxXgmf10uMmI4i0>
X-Mailman-Approved-At: Wed, 02 Mar 2016 15:57:11 -0800
Cc: "ietf@ietf.org" <ietf@ietf.org>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 02 Mar 2016 13:59:06 -0000

Senthil, I hadn't realised you'd published a new version of the draft. 
Please CC me if/when you update it again.

I've quickly reviewed the diffs between -06 and -07 :


1. Terminology

" /Any non-IPFIX terminology used to convey NAT events are described in 
/ /this section./"

-> which section is "/this/" referring to? Since this paragraph seems 
only to serve as an introduction to the third paragraph which only 
contains a single exception, would it be better to remove these two 
lines and go directly to the third paragraph? :

    However, that causes
    confusion in terminology used in NAT specific terms and IPFIX IEs.
    Any non-IPFIX terminology used to convey NAT events are described in
    this section.



2. Introduction

"This document details the IPFIX Information Elements(IEs)"

-> No need to repeat "(IEs)" since this was already explained in the 
Terminology section.

-> Remove the duplicated text:

    The IPFIX Protocol [RFC7011] defines a generic push mechanism for
    exporting information and events.  The IPFIX Information Model
    [IPFIX-IANA] defines a set of standard IEs which can be carried by
    the IPFIX protocol.This document details the IPFIX Information Elements(IEs) that MUST be 
logged by a NAT device that supports NAT logging using IPFIX.   This document details the IPFIX Information
    Elements(IEs) that MUST be logged by a NAT device that supports NAT
    logging using IPFIX, and all the optional fields.  The fields
    specified in this document are gleaned from [RFC4787] and [RFC5382].



5.4. Quota exceeded Event types

-> In the "/The events that can be reported are .../", I'd like to see 
the text be identical to the items listed in table 3 to remove any 
possible ambiguity.



5.6.8.4.Global Address mapping high threshold reached

-> Extra whitespace at the period: "paired address pooling behavior ."



8.1. New Information Elements / natLimitEvent, natThresholdEvent,

-> typo: " describer in Table below."

-> you should remove the "Table 22" and "Table 23" descriptions under 
those tables, because these won't make any sense when the text is 
transcribed into IANA's registry. E



8.2. Modified Information Elements / natEvent

-> Again, you can't modify the definitions of the existing values.


P.