Re: [Gen-art] Gen-ART review of draft-atlas-icmp-unnumbered-08

Carlos Pignataro <cpignata@cisco.com> Mon, 28 December 2009 01:24 UTC

Return-Path: <cpignata@cisco.com>
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 039053A67C2 for <gen-art@core3.amsl.com>; Sun, 27 Dec 2009 17:24:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ln5Jgl7OvgMX for <gen-art@core3.amsl.com>; Sun, 27 Dec 2009 17:24:24 -0800 (PST)
Received: from av-tac-rtp.cisco.com (hen.cisco.com [64.102.19.198]) by core3.amsl.com (Postfix) with ESMTP id D11953A6783 for <gen-art@ietf.org>; Sun, 27 Dec 2009 17:24:23 -0800 (PST)
X-TACSUNS: Virus Scanned
Received: from rooster.cisco.com (localhost.cisco.com [127.0.0.1]) by av-tac-rtp.cisco.com (8.13.8+Sun/8.13.8) with ESMTP id nBS1O1Nx008326; Sun, 27 Dec 2009 20:24:01 -0500 (EST)
Received: from [10.116.85.235] (rtp-cpignata-87110.cisco.com [10.116.85.235]) by rooster.cisco.com (8.13.8+Sun/8.13.8) with ESMTP id nBS1O0LL025162; Sun, 27 Dec 2009 20:24:00 -0500 (EST)
Message-ID: <4B3808B0.2040501@cisco.com>
Date: Sun, 27 Dec 2009 20:24:00 -0500
From: Carlos Pignataro <cpignata@cisco.com>
Organization: cisco Systems, Inc.
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.0
MIME-Version: 1.0
To: Black_David@emc.com
References: <C2D311A6F086424F99E385949ECFEBCB012C42D7@CORPUSMX80B.corp.emc.com> <C2D311A6F086424F99E385949ECFEBCB012C42D8@CORPUSMX80B.corp.emc.com>
In-Reply-To: <C2D311A6F086424F99E385949ECFEBCB012C42D8@CORPUSMX80B.corp.emc.com>
X-Enigmail-Version: 0.96.0
X-Face: *3w8NvnQ|kS~V{&{U}$?G9U9EJQ8p9)O[1[1F'1i>XIc$5FR!hdAIf5}'Xu-3`^Z']h0J* ccB'fl/XJYR[+, Z+jj`4%06nd'y9[ln&ScJT5S+O18e^
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Cc: JR Rivers <jrrivers@alumni.calpoly.edu>, rbonica@juniper.net, gen-art@ietf.org, naiming@cisco.com, jari.arkko@piuha.net, alia.atlas@bt.com
Subject: Re: [Gen-art] Gen-ART review of draft-atlas-icmp-unnumbered-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/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: Mon, 28 Dec 2009 01:24:25 -0000

[adding JR to Cc:]

On 12/27/2009 6:02 PM, Black_David@emc.com wrote:
> And the draft needs a new email address for JR:

Yes, we will fix this before publication.

Thanks again,

-- Carlos.

> 
> Your message did not reach some or all of the intended recipients.
> 
>       Subject:	Gen-ART review of draft-atlas-icmp-unnumbered-08
>       Sent:	12/27/2009 5:53 PM
> 
> The following recipient(s) cannot be reached:
> 
>       jrrivers@cisco.com on 12/27/2009 5:54 PM
>             The e-mail system was unable to deliver the message, but did
> not report a specific reason.  Check the address and try again.  If it
> still fails, contact your system administrator.
>             < sj-inbound-d.cisco.com #5.0.0 smtp; 5.1.0 - Unknown
> address error 550-'5.1.1 <jrrivers@cisco.com>... User unknown' (delivery
> attempts: 0)>
> 
> 
> Thanks,
> --David
> 
> 
>> -----Original Message-----
>> From: Black, David
>> Sent: Sunday, December 27, 2009 5:53 PM
>> To: alia.atlas@bt.com; rbonica@juniper.net; cpignata@cisco.com;
> jrrivers@cisco.com; naiming@cisco.com;
>> 'gen-art@ietf.org'
>> Cc: Jari Arkko; Black, David
>> Subject: Gen-ART review of draft-atlas-icmp-unnumbered-08
>>
>> I have been selected as the General Area Review Team (Gen-ART)
>> reviewer for this draft (for background on Gen-ART, please see
>> http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html).
>>
>> Please resolve these comments along with any other Last Call
>> comments you may receive.
>>
>> Document: draft-atlas-icmp-unnumbered-08
>> Reviewer: David L. Black
>> Review Date: December 27, 2009
>> IETF LC End Date: January 1, 2010
>>
>> Summary:
>> This draft is on the right track, but has open issues, described
>> in the review.
>>
>> Comments:
>> The draft defines an extension to ICMP messages that allows additional
>> network interface information to be provided with some ICMP messages.
>> The information may be an ifIndex, an IP address, an Interface Name
>> and/or an MTU.
>>
>> The one open issue in this review involves the use of ifIndex to
>> identify an interface.  The motivating use cases in section 3
>> involve traceroute, ACL-caused-blockage, and the need to determine the
>> MTU - those use cases are "public" in the sense that they clearly
>> encompass use of the ICMP messages by other than the network operator
>> whose device generated those messages.
>>
>> In this context, IP address, Interface Name and MTU are all clearly
>> useful, but I am concerned about ifIndex.  An ifIndex is effectively
>> private to the SNMP management infrastructure of the network operator.
>> Determining what interface an ifIndex designates generally involves
>> SNMP access that a network operator may be reluctant to grant to
>> outsiders due to the level of detail that such access may expose.
>>
>> I suspect that the ifIndex could be *very* useful to an ifIndex-
>> enhanced traceroute issued from a network operator's management
> station
>> as alluded to in paragraphs 3-5 of Section 6 (Security
> Considerations).
>> My open issue is that I should have to go all the way down to the
> latter
>> part of the Security Considerations section in order to find the
>> motivations for and intended usage of one of the major features of
> this
>> extension.
>>
>> I suggest adding a Section 3.3 to discuss use of ifIndex with ICMP and
>> network management tools (e.g., enhanced traceroute plus an SNMP
> manager)
>> by a network operator to debug his/her own network.  This new section
>> should also contain some sort of warning that ifIndex information may
>> not be available to other than the network operator's management
>> applications (with a pointer to the Security Considerations section).
>>
>> Nits:
>> - Section 2: "o  that interface is numbered"  Please add a definition
>> 	of "numbered" for an interface.
>> - Section 4.1: Explain what a C-Type is.  A few words and a reference
>> 	to Section 8 of RFC 4884 should suffice.
>> - Section 5.4: This appears to be poorly stated:
>>    A single instance of IP Address information MAY be included only in
>>    the following circumstances:
>>
>>    Included in what?  I presume an Interface Information Object.
> Also,
>>    MAY ... only is poor usage of RFC 2119 terminology.  Here's an
> attempt
>>    at better phrasing:
>>
>>    When an Interface Information Object contains an IP Address, one of
>>    the following two conditions MUST be true:
>>
>> idnits 2.11.15 did not find any nits.
>>
>> Thanks,
>> --David
>> ----------------------------------------------------
>> David L. Black, Distinguished Engineer
>> EMC Corporation, 176 South St., Hopkinton, MA  01748
>> +1 (508) 293-7953             FAX: +1 (508) 293-7786
>> black_david@emc.com        Mobile: +1 (978) 394-7754
>> ----------------------------------------------------
>