Re: [Gen-art] Last Call Review: draft-ietf-mile-enum-reference-format-10

Tom Taylor <tom.taylor.stds@gmail.com> Tue, 09 December 2014 23:08 UTC

Return-Path: <tom.taylor.stds@gmail.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DBB891A8896 for <gen-art@ietfa.amsl.com>; Tue, 9 Dec 2014 15:08:46 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2
X-Spam-Level:
X-Spam-Status: No, score=-2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=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 FtWM4RcKgUvx for <gen-art@ietfa.amsl.com>; Tue, 9 Dec 2014 15:08:44 -0800 (PST)
Received: from mail-ig0-x230.google.com (mail-ig0-x230.google.com [IPv6:2607:f8b0:4001:c05::230]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2FF901A0074 for <gen-art@ietf.org>; Tue, 9 Dec 2014 15:08:44 -0800 (PST)
Received: by mail-ig0-f176.google.com with SMTP id l13so5831154iga.3 for <gen-art@ietf.org>; Tue, 09 Dec 2014 15:08:43 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:subject:references :in-reply-to:content-type:content-transfer-encoding; bh=aVJ0jyBIFxeSJZPbWiqywTBjhc/XnZ/OZ9/i1i4XN+s=; b=gUzyGmBxggjp/TnW6ne2liW1U8vg/ROpIE+GZ9bBKBHwE9vPSareD038Gw4y28c2ha oo6zzPEuJ/dG+yN+zNDzCwpnXbB5GJ0uZsUxZ3814x5sqommCrpCjfDNQYc9NifbMpby +/gydWAZ2Gt06ohjtwy7VTFJoDHrR95c/GPKsLuIX17ydcN6V2aruuCrbWsQYzqxScEh emdh51Qb6eoY7HEqLQOYWQWGdDSHf/3Gouf/dU3/W7rAWR/xbXfVPN+s0T9mi7am6KBU /xZUjVSrI01v54o5W4advb4W4ZvainxNdnjI1rjF0oMGG3e8jywnajyPUUvti/4aQnfa vaMw==
X-Received: by 10.107.170.98 with SMTP id t95mr994744ioe.7.1418166523324; Tue, 09 Dec 2014 15:08:43 -0800 (PST)
Received: from [192.168.0.101] ([216.254.167.19]) by mx.google.com with ESMTPSA id v83sm1283086iov.30.2014.12.09.15.08.42 for <multiple recipients> (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Tue, 09 Dec 2014 15:08:43 -0800 (PST)
Message-ID: <548780F9.3010600@gmail.com>
Date: Tue, 09 Dec 2014 18:08:41 -0500
From: Tom Taylor <tom.taylor.stds@gmail.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: "Black, David" <david.black@emc.com>, Gen Art <gen-art@ietf.org>, "draft-ietf-mile-enum-reference-format.all@tools.ietf.org" <draft-ietf-mile-enum-reference-format.all@tools.ietf.org>
References: <54873E73.8000101@gmail.com> <CE03DB3D7B45C245BCA0D243277949362AEF0D@MX104CL02.corp.emc.com>
In-Reply-To: <CE03DB3D7B45C245BCA0D243277949362AEF0D@MX104CL02.corp.emc.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: http://mailarchive.ietf.org/arch/msg/gen-art/GOClCoI5Wf2V2DewrUoY95yKuk4
Subject: Re: [Gen-art] Last Call Review: draft-ietf-mile-enum-reference-format-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
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: Tue, 09 Dec 2014 23:08:47 -0000

Seems like you need a reference to 5070-bis rather than 5070, then.

Tom

On 09/12/2014 4:40 PM, Black, David wrote:
> Tom,
>
> Thanks for reviewing this draft.
>
>> Major issues: I am having a hard time reconciling the extension
>> procedures specified in Section 5 of RFC 5070 [IODEF] with the content
>> of the draft.
>
> No surprise there, as those extension procedures are not being used,
> although one would have to be a mile WG participant to understand why ...
>
>> As I see it, you have added an attribute to ReferenceName,
>> and this is actually not covered by RFC 5070. As I understand it, 5.1
>> covers ENUMs and 5.2 covers new classes. My conclusion is that this
>> document should update RFC 5070, describing how to add new attributes --
>> or is that the equivalent of adding a new class? Even if it were a
>> simple matter of adding ENUMs, where are the ext- declarations called
>> for by Section 5.1 of RFC 5070?
>
> The mile WG is in the process of replacing RFC 5070 with a new IODEF v2
> including a new XML schema that will use the enum ref format schema in
> the draft that you reviewed.  Here's the 5070bis draft:
>
> http://datatracker.ietf.org/doc/draft-ietf-mile-rfc5070-bis/
>
> This topic was discussed in the mile WG meeting in Honolulu, and the
> course of action that resulted is to not update RFC 5070 or provide
> extension definitions for it because RFC 5070 will be replaced soon.
>
> Thanks,
> --David
>
>> -----Original Message-----
>> From: Tom Taylor [mailto:tom.taylor.stds@gmail.com]
>> Sent: Tuesday, December 09, 2014 1:25 PM
>> To: Gen Art; draft-ietf-mile-enum-reference-format.all@tools.ietf.org
>> Subject: Last Call Review: draft-ietf-mile-enum-reference-format-10
>>
>> 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-mile-enum-reference-format-10
>> Reviewer: Tom Taylor
>> Review Date: 9/12/2014
>> IETF LC End Date: 16/12/2014
>> IESG Telechat date: (if known)
>>
>> Summary: Basically a well-written document with tiny nits. The "major
>> issue" may simply be a matter of my inexperience with XML schemas.
>>
>> Major issues: I am having a hard time reconciling the extension
>> procedures specified in Section 5 of RFC 5070 [IODEF] with the content
>> of the draft. As I see it, you have added an attribute to ReferenceName,
>> and this is actually not covered by RFC 5070. As I understand it, 5.1
>> covers ENUMs and 5.2 covers new classes. My conclusion is that this
>> document should update RFC 5070, describing how to add new attributes --
>> or is that the equivalent of adding a new class? Even if it were a
>> simple matter of adding ENUMs, where are the ext- declarations called
>> for by Section 5.1 of RFC 5070?
>>
>> Minor issues:
>>
>> Nits/editorial comments:
>>
>> Tiny nit, third paragraph of Security Considerations:
>>       s/third-party/third party/ (three times)
>> Former is an adjective, but contecxt requires a noun.
>>
>> The last sentence of the IANA Considerations section has a forward
>> reference to Section 6 which should instead be Section 5.
>
>