Re: [IPFIX] [irsg] Review of draft-irtf-nmrg-location-ipfix-07.txt

"Eggert, Lars" <lars@netapp.com> Wed, 29 March 2017 18:29 UTC

Return-Path: <lars@netapp.com>
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 1AEB9126B72; Wed, 29 Mar 2017 11:29:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.921
X-Spam-Level:
X-Spam-Status: No, score=-6.921 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=netapp.onmicrosoft.com
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 0Cb7I_8ZUE2i; Wed, 29 Mar 2017 11:29:30 -0700 (PDT)
Received: from mx142.netapp.com (mx142.netapp.com [216.240.21.19]) (using TLSv1.2 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id BB3E712025C; Wed, 29 Mar 2017 11:29:30 -0700 (PDT)
X-IronPort-AV: E=Sophos;i="5.36,242,1486454400"; d="asc'?scan'208";a="179701701"
Received: from hioexcmbx05-prd.hq.netapp.com ([10.122.105.38]) by mx142-out.netapp.com with ESMTP; 29 Mar 2017 11:17:05 -0700
Received: from VMWEXCCAS01-PRD.hq.netapp.com (10.122.105.11) by hioexcmbx05-prd.hq.netapp.com (10.122.105.38) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Wed, 29 Mar 2017 11:28:28 -0700
Received: from NAM03-BY2-obe.outbound.protection.outlook.com (10.120.60.153) by VMWEXCCAS01-PRD.hq.netapp.com (10.122.105.11) with Microsoft SMTP Server (TLS) id 15.0.1210.3 via Frontend Transport; Wed, 29 Mar 2017 11:28:28 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=netapp.onmicrosoft.com; s=selector1-netapp-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=4evFL2l7xH2riW7aSeb4SCLwIpL501QT7vl4DAgLTYw=; b=C5Irl68Zih92FHo66OqmvZS8EQXK4PRNvU9FY3qjGqgsvXuxypkUhYdDKNB8oolNj8QDpuTd17cRbGOXoTN6xvWml+PzbECcnqLCEyktad5SA5mDTHcUfryls58bpDSBWtTQDVrVZGk4mX0v31Dk2hOg0pcT/SFijMFzm7i6Jr4=
Received: from BN3PR0601MB1153.namprd06.prod.outlook.com (10.160.157.18) by BN3PR0601MB1155.namprd06.prod.outlook.com (10.160.157.20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.991.14; Wed, 29 Mar 2017 18:28:27 +0000
Received: from BN3PR0601MB1153.namprd06.prod.outlook.com ([10.160.157.18]) by BN3PR0601MB1153.namprd06.prod.outlook.com ([10.160.157.18]) with mapi id 15.01.0991.021; Wed, 29 Mar 2017 18:28:27 +0000
From: "Eggert, Lars" <lars@netapp.com>
To: Abdelkader Lahmadi <abdelkader.lahmadi@loria.fr>
CC: "IPFIX@ietf.org" <IPFIX@ietf.org>, "nmrg-chairs@ietf.org" <nmrg-chairs@ietf.org>, "Internet Research Steering Group (irsg@irtf.org)" <irsg@irtf.org>, "draft-irtf-nmrg-location-ipfix.authors@ietf.org" <draft-irtf-nmrg-location-ipfix.authors@ietf.org>, "opsawg-chairs@ietf.org" <opsawg-chairs@ietf.org>, PJ Aitken <pjaitken@brocade.com>, Rick Hofstede <mail@rickhofstede.nl>
Thread-Topic: [irsg] [IPFIX] Review of draft-irtf-nmrg-location-ipfix-07.txt
Thread-Index: AQHSlA2rRN1dx6gTP0qxqcp8ZIxXXaGC9I4AgAAC9ACAAAL6AIApSViAgAAGkACAAAH/gA==
Date: Wed, 29 Mar 2017 18:28:27 +0000
Message-ID: <4ECEFCC5-683E-4F53-AF42-FFDC7A48FE05@netapp.com>
References: <BBA82579FD347748BEADC4C445EA0F21A2260FE9@NKGEML515-MBX.china.huawei.com> <318bf874-2700-640e-e0c1-0ea7953b448f@gmail.com> <64CC7F98-8868-4BE5-ABE3-F6F01BF2FFF6@loria.fr> <cbc29c8a-5e11-7918-0afe-dfebafe0cd2c@gmail.com> <808e677f-f0ed-37de-2084-9f2074359658@brocade.com> <9F2B4045-D838-4D94-8B55-1599879537E2@loria.fr> <c056c266-5206-e87c-3535-d765e94b58ed@brocade.com> <24A6EB4E-2AED-4A31-A066-8D6EDB8E7C73@loria.fr> <6AA4BDC1-D371-4576-9C5D-BBC49391095B@netapp.com> <7DBCA7BE-C819-4107-AC7A-A5CA08680B18@loria.fr> <FEF04314-8E1D-4026-BFCF-41FDB56F4A40@netapp.com>
In-Reply-To: <FEF04314-8E1D-4026-BFCF-41FDB56F4A40@netapp.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator:
x-mailer: Apple Mail (2.3273)
authentication-results: loria.fr; dkim=none (message not signed) header.d=none;loria.fr; dmarc=none action=none header.from=netapp.com;
x-ms-exchange-messagesentrepresentingtype: 1
x-originating-ip: [2001:67c:370:128:4520:3ece:ac25:8bd5]
x-microsoft-exchange-diagnostics: 1; BN3PR0601MB1155; 7:sEr7IwonCtFG/TnYteM7lOVKPi7zgA8eXcfcw3izqpHGgFrUb+adnaifFqfyFygvV+bMB2mrypNHIxaRm+9aUCdM8Jduu8l25SFvyCRTDINYQjga18JdNl3nGyTQZ7NS4xxYlXdt+uhrvIYlckE1QrJtc0EPJuRqew+yyurI+sSwZDl75GQuUjX3Cj/izsl7Yba1B++9g1YN5s0VrcZmi5A/hhH3HZjw5CGYr5i4vPqFfujNNAOPkcrLvBqwQC6Oad8lpUlNcrivPMP6StuHUPw83bXj0ThkIf051zXdqAthVZsjzVBHsEThYui61+Cf4u03X4CJVxe9FN5Wf+aaCQ==
x-ms-office365-filtering-correlation-id: 57a86cd0-8e66-4874-fc29-08d476d164c2
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254075)(201703131423075)(201703031133081); SRVR:BN3PR0601MB1155;
x-microsoft-antispam-prvs: <BN3PR0601MB1155407C1516498A085A3C07A7350@BN3PR0601MB1155.namprd06.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(120809045254105)(131327999870524);
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(102415395)(6040450)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6055026)(6041248)(20161123560025)(20161123564025)(20161123555025)(201703131423075)(201702281528075)(201703061421075)(201703061406075)(20161123558025)(20161123562025)(6072148); SRVR:BN3PR0601MB1155; BCL:0; PCL:0; RULEID:; SRVR:BN3PR0601MB1155;
x-forefront-prvs: 0261CCEEDF
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(6009001)(39400400002)(39840400002)(39410400002)(39450400003)(39850400002)(45074003)(24454002)(377424004)(6306002)(8936002)(6512007)(81166006)(345774005)(99286003)(6436002)(6486002)(6506006)(77096006)(99936001)(54906002)(36756003)(8676002)(93886004)(230783001)(5660300001)(53936002)(50986999)(38730400002)(229853002)(6116002)(110136004)(6246003)(102836003)(76176999)(25786009)(3280700002)(4326008)(2900100001)(2906002)(122556002)(50226002)(2950100002)(6916009)(33656002)(305945005)(86362001)(7736002)(82746002)(83716003)(3660700001)(57306001)(53546009)(189998001); DIR:OUT; SFP:1101; SCL:1; SRVR:BN3PR0601MB1155; H:BN3PR0601MB1153.namprd06.prod.outlook.com; FPR:; SPF:None; MLV:sfv; LANG:en;
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: multipart/signed; boundary="Apple-Mail=_57B32BB9-5C34-454C-B509-7806F5CC2CA7"; protocol="application/pgp-signature"; micalg=pgp-sha512
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Mar 2017 18:28:27.5013 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 4b0911a0-929b-4715-944b-c03745165b3a
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR0601MB1155
X-OriginatorOrg: netapp.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/ipfix/nFm7AqmX6mznqkedHV1m12DOpzo>
Subject: Re: [IPFIX] [irsg] Review of draft-irtf-nmrg-location-ipfix-07.txt
X-BeenThere: ipfix@ietf.org
X-Mailman-Version: 2.1.22
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: <https://mailarchive.ietf.org/arch/browse/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: Wed, 29 Mar 2017 18:29:34 -0000

Correction: My spy on the IESG informs me that you linked to an outdated version of the review; the current one being https://datatracker.ietf.org/doc/conflict-review-irtf-nmrg-location-ipfix/

The current review *does* call out a problem with the draft, but has not concluded yet.

Lars


> On 2017-3-29, at 13:21, Eggert, Lars <lars@netapp.com>; wrote:
> 
> Hi,
> 
> On 2017-3-29, at 12:57, Abdelkader Lahmadi <abdelkader.lahmadi@loria.fr>; wrote:
>> Dear Lars, and NMRG chairs,
> 
> Allison has taken over as chair, so this is a question for her. However:
> 
>> How we can proceed now with the draft?
>> 
>> According to this: https://datatracker.ietf.org/doc/conflict-review-irtf-nmrg-location-ipfix/00/, It seems that the draft is not in conflict with IETF work.
> 
> The IESG has not yet concluded its review - that link at the moment captures the current status of individual ADs positions. They will send a formal response when their review has concluded.
> 
>> But, it seems that an IETF review and IESG approval are required for registering the location method tokens according to this:
>> https://datatracker.ietf.org/doc/conflict-review-irtf-nmrg-location-ipfix/
> 
> All IANA actions coming from the IETF require IESG approval. Typically, that approval is implicit in a review that results in a document being publishable.
> 
>> I suppose that we can now update the draft with the IANA recommendation and also the review provided IPFIX expert (Paul).
> 
> I'd wait until the IESG review has concluded.
> 
> Lars
> 
> 
> 
> 
>> Thank you.
>> Best regards.
>>> On 03 Mar 2017, at 12:28, Eggert, Lars <lars@netapp.com>; wrote:
>>> 
>>> On 2017-3-3, at 12:17, Abdelkader Lahmadi <abdelkader.lahmadi@loria.fr>; wrote:
>>>> So, we work on a new text to resolve ALL the raised issues and send you a version.
>>> 
>>> Hang on. Your RG chair should have explained how the process works.
>>> 
>>> This IRTF document is now being reviewed (per RFC5742) by the IESG. That review is *not* the same review that happens for IETF documents. Specifically, the IESG can really only say one of these five things:
>>> 
>>>>  1. The IESG has concluded that there is no conflict between this
>>>>     document and IETF work.
>>>> 
>>>>  2. The IESG has concluded that this work is related to IETF work done
>>>>     in WG <X>, but this relationship does not prevent publishing.
>>>> 
>>>>  3. The IESG has concluded that publication could potentially disrupt
>>>>     the IETF work done in WG <X> and recommends not publishing the
>>>>     document at this time.
>>>> 
>>>>  4. The IESG has concluded that this document violates IETF procedures
>>>>     for <Y> and should therefore not be published without IETF review
>>>>     and IESG approval.
>>>> 
>>>>  5. The IESG has concluded that this document extends an IETF protocol
>>>>     in a way that requires IETF review and should therefore not be
>>>>     published without IETF review and IESG approval.
>>> 
>>> 
>>> At the moment, you are getting individual comments from IPFIX experts as part of the IANA review process of the registry actions your document wants to make happen. Those are very valuable (thank you!), but before you make any changes to the document, please wait until an Area Director says that the issues raised are at a significance where they will ask for an IESG response other than #1 or #2 above.
>>> 
>>> It sounds like this is likely going to be the case here, but please wait until the ADs have caught up on the discussion. And please wait with submitting a new revision until that happens as well.
>>> 
>>> Lars
>>> 
>>> 
>>>> 
>>>> Best,
>>>>> On 03 Mar 2017, at 12:07, PJ Aitken <pjaitken@brocade.com>; wrote:
>>>>> 
>>>>> Abdelkader, it was me who did the IE-doctors review. That's only concerned with the IANA request; it's not an IPFIX review of the document.
>>>>> 
>>>>> P.
>>>>> 
>>>>> 
>>>>> On 03/03/17 11:01, Abdelkader Lahmadi wrote:
>>>>>> Hello,
>>>>>> We haven’t really get a "proper review" of the document by IPFIX experts. Recently, we had a discussion with IANA and they asked IE-doctors to make a review, since that we received some points to be fixed regarding the proposed IE. I can forward to you the other comments from IE-doctors that we have received by IANA.
>>>>>> 
>>>>>> Thank you for your comments, Ok we will fix the raised issues in the document.
>>>>>> Best regards.
>>>>>> 
>>>>>> 
>>>>>>> On 03 Mar 2017, at 11:42, PJ Aitken <pjaitken@brocade.com>; wrote:
>>>>>>> 
>>>>>>> Authors, has this document been reviewed by any IPFIX experts?
>>>>>>> 
>>>>>>> I see a request on November 23rd, but no reviews. So let me sign up for that.
>>>>>>> 
>>>>>>> 
>>>>>>> First, I took a quick look at the Figures in Appendix B:
>>>>>>> 
>>>>>>> Figure 1: the Field Count should be 5, not 2.
>>>>>>> 
>>>>>>> Figure 2: the size of the optional Padding field is wrong: the figure shows 9 bits rather than 8.
>>>>>>> 
>>>>>>> Figure 4: the Length of 32 should be 28. The "geospatialLocationPosLat" Information Element isn't defined.
>>>>>>> 
>>>>>>> Figure 5: the Field Count of 2 should be 3.
>>>>>>> 
>>>>>>> Figure 7: The "geospatialLocationPostLng" and "geospatialLocationtLng" Information Elements aren't defined.
>>>>>>> 
>>>>>>> Figure 9: The sizes of the "CivicValue" data fields are not shown correctly. eg, "Inria Nancy-Grand Est" is depicted in 6 octets when it should contain 21. Therefore the Figure is misleading and difficult to understand; it is not a good example. Please redraw the figure correctly. Please mark the variable-lengths eg "vlen = 21".
>>>>>>> 
>>>>>>> Figure 11:
>>>>>>> The Set IDs (311, 312, 313) do not correspond to the Template IDs in Figure 10 (306, 307, 308).
>>>>>>> Again, the "Inria Nancy-Grand Grand Est" field is depicted in 6 octets rather than the requisite 27. Without the repeated "Grand", the 21 would be correct. Please write "vlen=21"
>>>>>>> The "Civic location Attr length" of 25 seems wrong.
>>>>>>> 
>>>>>>> 
>>>>>>> This document is not ready for publication. Please post an updated version so I can check that all the IPFIX details are correct.
>>>>>>> 
>>>>>>> Thanks,
>>>>>>> P.
>>>>> 
>>>>> _______________________________________________
>>>>> IPFIX mailing list
>>>>> IPFIX@ietf.org
>>>>> https://www.ietf.org/mailman/listinfo/ipfix
>>>> 
>>> 
>> 
>