Re: [Gen-art] Genart last call review of draft-ietf-ippm-initial-registry-12

Alissa Cooper <alissa@cooperw.in> Wed, 04 December 2019 16:30 UTC

Return-Path: <alissa@cooperw.in>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6C15212002E; Wed, 4 Dec 2019 08:30:33 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.773
X-Spam-Level:
X-Spam-Status: No, score=-2.773 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_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.073, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=cooperw.in header.b=GfI+V2WI; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=RFcgzvsu
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 8XarqBrdmUKD; Wed, 4 Dec 2019 08:30:31 -0800 (PST)
Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 9FB37120073; Wed, 4 Dec 2019 08:30:31 -0800 (PST)
Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id D1FB42262A; Wed, 4 Dec 2019 11:30:30 -0500 (EST)
Received: from mailfrontend1 ([10.202.2.162]) by compute7.internal (MEProxy); Wed, 04 Dec 2019 11:30:30 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cooperw.in; h= content-type:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; s=fm1; bh=l X2U24KzCqNLE+WhlsHJrAVn0mFuLB010geRZ5nyIOA=; b=GfI+V2WIAoyibI2kj b2LSV2SsznLjf4HzDwjgK4DclrPfVXI8A2pjKsVGTd0E0mnJWn3egFmtvqbUtoft Uz3olZtO1kj+/GMyh1doYUKs7MuYFvV3DmzDEzlET3sYVez+zly6wmO9QIxJvPgn ST5Ke1o1LgvWdu5VUizKoGNm/6yGtZekJHhYgdtbCTr9bkFyBl/QEmOSGA/IqZgX 41Uk3R78OVwcOhu9jZQtneB8KQaCLx+7OYvE0fh9dQZiZWOoA/ITepLecMlbC4yg fcHHYU3Kvglc4XHXvC/sqgkIH67GYK8XB/tAKWIKJ4LGp3lSwLvsuR1vZVoCu1sB 3nDNg==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=lX2U24KzCqNLE+WhlsHJrAVn0mFuLB010geRZ5nyI OA=; b=RFcgzvsuV/9xejoLlBwnR4MzQFV23/EJZhxyEvVtPd6AXdBg/7UCdwy6Z 24UNSaIiU3CCvWUEs9vNjnCjJta6+m+VWwzjXOCdqNGeaEpl+fmTpFryqyswjrsK SPUCRxuI0henoG8llriHkFWj5IyK1FdjgpuedvnGGZrWEHksq9cUCQgfHTE+nAPN L6oI9BH1SxoscO/QIuUb5EQC4zL0wbLwQZZqeSa8Hw/aidLyOLL2nautPAfauwUu 2xwmhAOO/Syhi5l/z/kL7as3tKAUMz/P1sW9UGNUmyen/vfOzKDfzaruRrkQLMXc XqNhUypNcEtpkcn401AbNGFvKDZnw==
X-ME-Sender: <xms:Jt_nXd7N4aiyzHF--Kl9uJOP6RqxPttXnjQyPPL17jvDzYHUUgBBTg>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedufedrudejledgkeejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurheptggguffhjgffgffkfhfvofesthhqmhdthhdtvdenucfhrhhomheptehlihhs shgrucevohhophgvrhcuoegrlhhishhsrgestghoohhpvghrfidrihhnqeenucffohhmrg hinhepihgvthhfrdhorhhgpdhprhhoohhfphhoihhnthdrtghomhenucfkphepuddtkedr hedurddutddurdelkeenucfrrghrrghmpehmrghilhhfrhhomheprghlihhsshgrsegtoh hophgvrhifrdhinhenucevlhhushhtvghrufhiiigvpedt
X-ME-Proxy: <xmx:Jt_nXcdYcEb9clFhMt2SZSbjbSmwr3O3BMNYibGNyRCCcrcBizIDSg> <xmx:Jt_nXTstl518c7KmgihSuxWy5ML4aZvpadJdGp50t9LTZ8WZRgmbtQ> <xmx:Jt_nXckVZPMOtZbSrfTWmJLfxBWYjK-JJQTWpHtt_HmKsOG7ZdK-6w> <xmx:Jt_nXTRBgQ1XSvIMT06XS09CN-81l69f1-3gqBmZ7u-fIhq6JHr4mQ>
Received: from alcoop-m-c46z.fios-router.home (pool-108-51-101-98.washdc.fios.verizon.net [108.51.101.98]) by mail.messagingengine.com (Postfix) with ESMTPA id 0D11680059; Wed, 4 Dec 2019 11:30:30 -0500 (EST)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\))
From: Alissa Cooper <alissa@cooperw.in>
In-Reply-To: <e1ab718e-cc0b-2e05-b3f9-e696ed8df5fd@joelhalpern.com>
Date: Wed, 04 Dec 2019 11:30:28 -0500
Cc: "MORTON, ALFRED C (AL)" <acm@research.att.com>, "gen-art@ietf.org" <gen-art@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "draft-ietf-ippm-initial-registry.all@ietf.org" <draft-ietf-ippm-initial-registry.all@ietf.org>, "ippm@ietf.org" <ippm@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <505CFB6D-D492-40E4-A340-F27A83DE1227@cooperw.in>
References: <157262730194.31927.5524715609259531175@ietfa.amsl.com> <4D7F4AD313D3FC43A053B309F97543CFA0B6B28E@njmtexg5.research.att.com> <e1ab718e-cc0b-2e05-b3f9-e696ed8df5fd@joelhalpern.com>
To: Joel Halpern Direct <jmh.direct@joelhalpern.com>
X-Mailer: Apple Mail (2.3445.9.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/znlqEi1L6dUqJXsX8zTYfGgGT-M>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-ippm-initial-registry-12
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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: Wed, 04 Dec 2019 16:30:33 -0000

Joel, thanks for your review. Al, thanks for your response. I entered a No Objection ballot.

Alissa


> On Nov 3, 2019, at 4:00 PM, Joel Halpern Direct <jmh.direct@joelhalpern.com> wrote:
> 
> Thanks Al.  I presumed all the ducks were in a row, but thought I should ask to be certain.
> 
> Yours,
> Joel
> 
> On 11/3/2019 3:14 PM, MORTON, ALFRED C (AL) wrote:
>> Hi Joel,
>> Thanks for your review, please see replies below.
>> Al
>>> -----Original Message-----
>>> From: Joel Halpern via Datatracker [mailto:noreply@ietf.org]
>>> Sent: Friday, November 1, 2019 12:55 PM
>>> To: gen-art@ietf.org
>>> Cc: last-call@ietf.org; draft-ietf-ippm-initial-registry.all@ietf.org;
>>> ippm@ietf.org
>>> Subject: Genart last call review of draft-ietf-ippm-initial-registry-12
>>> 
>>> Reviewer: Joel Halpern
>>> Review result: Ready
>>> 
>>> I am the assigned Gen-ART reviewer for this draft. The General Area
>>> Review Team (Gen-ART) reviews all IETF documents being processed
>>> by the IESG for the IETF Chair.  Please treat these comments just
>>> like any other last call comments.
>>> 
>>> For more information, please see the FAQ at
>>> 
>>> <https://urldefense.proofpoint.com/v2/url?u=https-
>>> 3A__trac.ietf.org_trac_gen_wiki_GenArtfaq&d=DwICaQ&c=LFYZ-
>>> o9_HUMeMTSQicvjIg&r=OfsSu8kTIltVyD1oL72cBw&m=GTgiUHp01_savOvQS49iOt8XRHfRw
>>> hPgZj-TNotgKGk&s=M0ib3zYg2qffmRujLJv2h_WHQ16W9fOYat9hNtBqcFk&e=>.
>>> 
>>> Document: draft-ietf-ippm-initial-registry-12
>>> Reviewer: Joel Halpern
>>> Review Date: 2019-11-01
>>> IETF LC End Date: 2019-11-06
>>> IESG Telechat date: Not scheduled for a telechat
>>> 
>>> Summary: This document is ready for publication as a Proposed Standard
>>> 
>>> Side note: I presume that as part of the process for
>>> draft-ietf-ippm-metric-registry (the normative reference the defines the
>>> structure used in this document) there has been discussion with IANA explicitly
>>> about the fact that this registry has an extremely large number of
>>> columns, some with extremely verbose content, and it will likely take some work for
>>> IANA to determine how to present this in a human-readable fashion?  And the
>>> lesser point that is probably covered by existing procedures, but I wanted to
>>> check, that IANA is prepared to fill in the URLs scattered throughout the
>>> document?
>> [acm]
>> Yes and Yes. We prepared a mock-up of the new Registry at various stages
>> of development. Humbly, it was my idea to make the registry entries both
>> readable and useful. The IANA reps suggested the mock-up early-on, and we have
>> shared the different versions with the IPPM WG.  We/IANA plan to make the
>> mock-up more widely available (but we failed to do that in time for Last Call).
>>> 
>>> Second note:  I did not review the accuracy of the descriptions of the metrics,
>>> but only looked for clarity.  This is material well known to the WG, and mostly
>>> derived from other documents this or closely related working groups have
>>> produced.
>>> 
>>> Major issues: N/A
>>> 
>>> Minor issues:
>>>     For those entries that are defining two (or more) closely related metrics,
>>>     should the document actually have two (or more) lines for URL, since the
>>>     text says that IANA is to assign two URLs.  (And the list of differing
>>>     fields should presumably include URL?)
>> [acm]
>> There are sections of the document that define more than one registry entry,
>> so yes, there will be >1 URLs, etc. in the corresponding rows.
>>> 
>>>     In the first part of section 5, there is a note about potentially splitting
>>>     the registry entry into two registry entries.  I can not understand the
>>>     note.  The registry is either defined with one entry or defined with two
>>>     entries.  Is this still an open item?  (If so, my "ready" above clearly
>>>     should be "Ready with issues.")  I think it is just an erroneous retention
>>>     of text from earlier?
>> [acm]
>> Exactly, it is a note left-stranded by editing later in the section,
>> thanks for catching it - deleted in the working version.
>>> 
>>> Nits/editorial comments:
>>>     If there are no roles to define in 5.3.6, shouldn't it say "N/A"
>> [acm]
>> Actually, it should define the Roles (Src and Dst) as with other Metrics.
>> Something went wrong with formatting here - the text below the
>> section header disappeared... thanks for catching that!
>>>     Some comments and remarks say "None" which makes sense.  Some say
>>>     "Additional (Informational) details for this entry" which seems to be
>>>     text left over from the template that should say "None"?
>> [acm]
>> Right,  found and replaced in working copy.
>>> 
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art