Re: [Gen-art] Re: LC Review: draft-ietf-disman-remops-mib-v2-09.txt
Brian E Carpenter <brc@zurich.ibm.com> Wed, 15 March 2006 09:58 UTC
Received: from [127.0.0.1] (helo=stiedprmman1.va.neustar.com) by megatron.ietf.org with esmtp (Exim 4.43) id 1FJSli-00070l-2p; Wed, 15 Mar 2006 04:58:14 -0500
Received: from [10.91.34.44] (helo=ietf-mx.ietf.org) by megatron.ietf.org with esmtp (Exim 4.43) id 1FJSlf-0006v6-F9; Wed, 15 Mar 2006 04:58:11 -0500
Received: from mtagate1.uk.ibm.com ([195.212.29.134]) by ietf-mx.ietf.org with esmtp (Exim 4.43) id 1FJSld-0002vD-QE; Wed, 15 Mar 2006 04:58:11 -0500
Received: from d06nrmr1407.portsmouth.uk.ibm.com (d06nrmr1407.portsmouth.uk.ibm.com [9.149.38.185]) by mtagate1.uk.ibm.com (8.12.10/8.12.10) with ESMTP id k2F9w8nk114468; Wed, 15 Mar 2006 09:58:08 GMT
Received: from d06av02.portsmouth.uk.ibm.com (d06av02.portsmouth.uk.ibm.com [9.149.37.228]) by d06nrmr1407.portsmouth.uk.ibm.com (8.12.10/NCO/VER6.8) with ESMTP id k2F9wSOd222808; Wed, 15 Mar 2006 09:58:28 GMT
Received: from d06av02.portsmouth.uk.ibm.com (loopback [127.0.0.1]) by d06av02.portsmouth.uk.ibm.com (8.12.11/8.13.3) with ESMTP id k2F9w8Nw012101; Wed, 15 Mar 2006 09:58:08 GMT
Received: from mail-gw3.hursley.ibm.com (mail-gw3.hursley.ibm.com [9.20.131.251]) by d06av02.portsmouth.uk.ibm.com (8.12.11/8.12.11) with ESMTP id k2F9w8sU012098; Wed, 15 Mar 2006 09:58:08 GMT
Received: from [9.20.136.27] (helo=sp15en17.hursley.ibm.com) by mail-gw3.hursley.ibm.com with esmtp (Exim 4.50) id 1FJSlc-0001gw-5S; Wed, 15 Mar 2006 09:58:08 +0000
Received: from zurich.ibm.com (sig-9-145-254-246.de.ibm.com [9.145.254.246]) by sp15en17.hursley.ibm.com (AIX5.1/8.11.6p2/8.11.0) with SMTP id k2F9w7V168520; Wed, 15 Mar 2006 09:58:07 GMT
Message-ID: <4417E525.6000302@zurich.ibm.com>
Date: Wed, 15 Mar 2006 10:57:57 +0100
From: Brian E Carpenter <brc@zurich.ibm.com>
Organization: IBM
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6) Gecko/20040113
X-Accept-Language: en, fr, de
MIME-Version: 1.0
To: Juergen Quittek <quittek@netlab.nec.de>
Subject: Re: [Gen-art] Re: LC Review: draft-ietf-disman-remops-mib-v2-09.txt
References: <E3F9D87C63E2774390FE67C924EC99BB0AB3D6E8@zrc2hxm1.corp.nortel.com> <7.0.1.0.0.20060306171715.0329f8c0@stevecrocker.com> <69669820960C84F7A53340CC@[192.168.1.128]>
In-Reply-To: <69669820960C84F7A53340CC@[192.168.1.128]>
Content-Type: text/plain; charset="us-ascii"; format="flowed"
Content-Transfer-Encoding: 7bit
X-Spam-Score: 0.0 (/)
X-Scan-Signature: d2b46e3b2dfbff2088e0b72a54104985
Cc: gen-art@ietf.org, quittek@ccrle.nec.de, Randy Presuhn <randy_presuhn@mindspring.com>, Bert Wijnen <bwijnen@lucent.com>, disman@ietf.org
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www1.ietf.org/pipermail/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
Errors-To: gen-art-bounces@ietf.org
Thanks. I have captured all this as a DISCUSS comment but it will be a simple cut and paste to make it into an RFC Editor note, at which point I can clear. Brian Juergen Quittek wrote: > Joel, > > Thank you for your comments. > Please see replies inline. > > --On 3/6/06 6:10 PM -0500 Joel M. Halpern wrote: > >> I was selected as General Area Review Team reviewer for this >> specification >> (for background on Gen-ART, please see >> http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html). >> >> This document is nearly ready for publication as a Proposed Standard. >> I have one concern, described below, and a few minor comments. >> >> The definition of the lookup tables bothers me. >> The short form: >> The name of the MIB is NSLOOKUP. The name of the operation is dns. > > > The MIB module specification does not necessarily require an implementation > to use DNS. So the operation is name lookup. > >> But the definition is not "perform a dns A or AAAA lookup." It is, >> "do the equivalent of some API" that the host may or may not have. >> This is not a good functional definition. > > > Indeed. But this is the intention of the MIB module. Usually, there > is more value in finding out with what result an actual name is resolved > at a remote host than what DNS delivers to this host as resolution. > >> The function is defined in very general terms. It performs either a >> symbolic lookup to get an IP address, with further definition only in >> terms of the name of common API functions, or a reverser lookup to get >> from IP address to name, again with the functional requirement given in >> terms of an API name. Given that these APIs are not standardized 9and >> no particular implementation is referenced) this is not a clear, >> implementable, interoperable definition. I suspect that the intent here >> is to allow whatever the host would do with a typical application request >> to get this kind of information. And that it is specifically vague so as >> to allow reference to caches or local configuration tables, as the host >> would perform. >> However, the text does not say "Do what you would do for a local >> application." >> It says "do something like these APIs." > > > I see the problem and agree that it should be fixed. > >> If the intent is explicitly to cause local caching / config tables to be >> visible to the MIB, then we MUST say that. If the intent is to >> perform a DNS >> lookup from that host (forward or reverse) then we need to say that. > > > As stated above, my view is that the MIB module should return whatever an > application would receive. This should be clarified in section 3.3.3. > What about > > OLD: > 3.3.3. lookupResultsTable > > The lookupResultsTable is used to store the results of lookup > operations. The lookupResultsTable is initially indexed by the same > index elements that the lookupCtlTable contains (lookupCtlOwnerIndex > and lookupCtlOperationName) but has a third index element, > lookupResultsIndex (Unsigned32 textual convention), in order to > associate multiple results with the same lookupCtlEntry. > > NEW: > 3.3.3. lookupResultsTable > > The lookupResultsTable is used to store the results of lookup > operations. Results to be reported here SHOULD be results of > a lookup function that is commonly used by applications at the > managed node. This implies that results are not necessarily > consistent with the results of a pure DNS lookup at the > managed node, but may be influenced by local lookup tables or > other source of information, depending on the configuration of > the managed node. > > The lookupResultsTable is initially indexed by the same > index elements that the lookupCtlTable contains (lookupCtlOwnerIndex > and lookupCtlOperationName) but has a third index element, > lookupResultsIndex (Unsigned32 textual convention), in order to > associate multiple results with the same lookupCtlEntry. > > ? > >> Minor: The description of the pingResultsTable refers to the >> pingPastProbeTable. There is no such table. Presumably, the >> pingProbeHistoryTable is intended. > > > That's right. Thanks for this catch. > >> Minor: Could the first sentence of 3.1.4 on pingProbeHistoryTable be: >> The results of each past ping probe is stored in this table on a >> per pingCtlEntry basis. >> ? > > > I would be fine with this change if we replace "is" with "are". > > Thanks, > > Juergen > >> Minor: What is the purpose of traceRouteCtlByPassRouteTable? I can >> understand with ping needing to be able to say "ping this neighbor, >> even if routing is confused. However, why is there a similar entry >> in the traceroute control table? Given that the restriction is that >> one is talking to a neighbor, what is the point of sending a traceroute? >> >> >> >> At 04:50 PM 3/1/2006, Mary Barnes wrote: >> >>> --------------------------- >>> Reviewer: Joel Halpern >>> >>> - 'Definitions of Managed Objects for Remote Ping, Traceroute, and >>> Lookup >>> Operations ' >>> <draft-ietf-disman-remops-mib-v2-09.txt> as a Proposed Standard >>> >>> IETF LC ends on 2006-03-09. >>> >>> The file can be obtained via >>> <http://www.ietf.org/internet-drafts/draft-ietf-disman-remops-mib-v2-09.txt>http://www.ietf.org/internet-drafts/draft-ietf-disman-remops-mib-v2-09.txt >>> >>> >> > > > > _______________________________________________ > Gen-art mailing list > Gen-art@ietf.org > https://www1.ietf.org/mailman/listinfo/gen-art > _______________________________________________ Gen-art mailing list Gen-art@ietf.org https://www1.ietf.org/mailman/listinfo/gen-art
- [Gen-art] A *new* batch of IETF LC reviews - Marc… Mary Barnes
- [Gen-art] Re: LC Review: draft-ietf-disman-remops… Joel M. Halpern
- [Gen-art] Re: LC Review: draft-ietf-disman-remops… Joel M. Halpern
- [Gen-art] Re: LC Review: draft-ietf-disman-remops… Joel M. Halpern
- [Gen-art] Re: LC Review: draft-ietf-disman-remops… Juergen Quittek
- [Gen-art] Re: LC Review: draft-ietf-disman-remops… Juergen Quittek
- Re: [Gen-art] Re: LC Review: draft-ietf-disman-re… Brian E Carpenter
- [Gen-art] Re: LC Review: draft-ietf-disman-remops… Juergen Quittek
- [Gen-art] Re: LC Review: draft-ietf-disman-remops… Joel M. Halpern