Re: Gen-ART LC review of draft-ietf-pcp-description-option-03

"Reinaldo Penno (repenno)" <repenno@cisco.com> Mon, 27 January 2014 09:38 UTC

Return-Path: <repenno@cisco.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 932C91A011C; Mon, 27 Jan 2014 01:38:51 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -10.035
X-Spam-Level:
X-Spam-Status: No, score=-10.035 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RP_MATCHES_RCVD=-0.535, SPF_PASS=-0.001, USER_IN_DEF_DKIM_WL=-7.5] 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 VV0_slMA765A; Mon, 27 Jan 2014 01:38:49 -0800 (PST)
Received: from alln-iport-5.cisco.com (alln-iport-5.cisco.com [173.37.142.92]) by ietfa.amsl.com (Postfix) with ESMTP id B64441A0117; Mon, 27 Jan 2014 01:38:48 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=23400; q=dns/txt; s=iport; t=1390815527; x=1392025127; h=from:to:cc:subject:date:message-id:in-reply-to: mime-version; bh=/PDkheEyah9yV21wccpxSltr10EWTKycXqUxebM/oVM=; b=ckcklDLGlDCWokqRwx/yAx9Cz1jk2eUwI532ccUbQffRA2FPThNCl8Pz Us77TfjggPL+9S1y2jx8LLyZaCtYVYboKvpxroIqRQqiiJcX0JJt6VYN8 QC48NA4P7NC9dyCEf9xPITWODoOsirFgHYn3EIwMboJL8U5asJDExqA8T w=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: AkAHABwo5lKtJV2b/2dsb2JhbABZgkhEOFa0AIhTgQoWdIIlAQEBBC1MEgEIDgMDAQEBIQcoERQJCAEBBAENBQmHaAMRDcI8DYVWF4x3I4EqOBEGAYQ4BIVYkGOBbIEyiyuFQYMtgWlB
X-IronPort-AV: E=Sophos; i="4.95,728,1384300800"; d="scan'208,217"; a="15747634"
Received: from rcdn-core-4.cisco.com ([173.37.93.155]) by alln-iport-5.cisco.com with ESMTP; 27 Jan 2014 09:38:46 +0000
Received: from xhc-aln-x01.cisco.com (xhc-aln-x01.cisco.com [173.36.12.75]) by rcdn-core-4.cisco.com (8.14.5/8.14.5) with ESMTP id s0R9cjRb019913 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Mon, 27 Jan 2014 09:38:46 GMT
Received: from xmb-rcd-x04.cisco.com ([169.254.8.37]) by xhc-aln-x01.cisco.com ([173.36.12.75]) with mapi id 14.03.0123.003; Mon, 27 Jan 2014 03:38:45 -0600
From: "Reinaldo Penno (repenno)" <repenno@cisco.com>
To: Roni Even <ron.even.tlv@gmail.com>, "mohamed.boucadair@orange.com" <mohamed.boucadair@orange.com>, "draft-ietf-pcp-description-option.all@tools.ietf.org" <draft-ietf-pcp-description-option.all@tools.ietf.org>
Subject: Re: Gen-ART LC review of draft-ietf-pcp-description-option-03
Thread-Topic: Gen-ART LC review of draft-ietf-pcp-description-option-03
Thread-Index: Ac8ablfn3UbFfKwfSEyn6Pcjjhog/QAyx66AAA7VfwD//3wCgA==
Date: Mon, 27 Jan 2014 09:38:44 +0000
Message-ID: <CF0B68F0.8395%repenno@cisco.com>
In-Reply-To: <033d01cf1b42$854c10e0$8fe432a0$@gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.2.3.120616
x-originating-ip: [10.21.91.128]
Content-Type: multipart/alternative; boundary="_000_CF0B68F08395repennociscocom_"
MIME-Version: 1.0
X-Mailman-Approved-At: Mon, 27 Jan 2014 07:23:56 -0800
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 27 Jan 2014 09:38:51 -0000

Hi,

The idea is that it can be seen by the entity operating the PCP Server.

Thanks,

From: Roni Even <ron.even.tlv@gmail.com<mailto:ron.even.tlv@gmail.com>>
Date: Monday, January 27, 2014 at 1:31 AM
To: "mohamed.boucadair@orange.com<mailto:mohamed.boucadair@orange.com>" <mohamed.boucadair@orange.com<mailto:mohamed.boucadair@orange.com>>, "draft-ietf-pcp-description-option.all@tools.ietf.org<mailto:draft-ietf-pcp-description-option.all@tools.ietf.org>" <draft-ietf-pcp-description-option.all@tools.ietf.org<mailto:draft-ietf-pcp-description-option.all@tools.ietf.org>>
Cc: "ietf@ietf.org<mailto:ietf@ietf.org>" <ietf@ietf.org<mailto:ietf@ietf.org>>, "gen-art@ietf.org<mailto:gen-art@ietf.org>" <gen-art@ietf.org<mailto:gen-art@ietf.org>>
Subject: RE: Gen-ART LC review of draft-ietf-pcp-description-option-03
Resent-From: <draft-alias-bounces@tools.ietf.org<mailto:draft-alias-bounces@tools.ietf.org>>
Resent-To: <dthaler@microsoft.com<mailto:dthaler@microsoft.com>>, <dwing@cisco.com<mailto:dwing@cisco.com>>, "mohamed.boucadair@orange.com<mailto:mohamed.boucadair@orange.com>" <mohamed.boucadair@orange.com<mailto:mohamed.boucadair@orange.com>>, Cisco Employee <repenno@cisco.com<mailto:repenno@cisco.com>>, <ted.lemon@nominum.com<mailto:ted.lemon@nominum.com>>
Resent-Date: Monday, January 27, 2014 at 1:35 AM

Hi Med,
Inline
Roni

From: mohamed.boucadair@orange.com<mailto:mohamed.boucadair@orange.com> [mailto:mohamed.boucadair@orange.com]
Sent: 27 January, 2014 10:31 AM
To: Roni Even; draft-ietf-pcp-description-option.all@tools.ietf.org<mailto:draft-ietf-pcp-description-option.all@tools.ietf.org>
Cc: ietf@ietf.org<mailto:ietf@ietf.org>; gen-art@ietf.org<mailto:gen-art@ietf.org>
Subject: RE: Gen-ART LC review of draft-ietf-pcp-description-option-03

Dear Roni,
Thank you for the review.
Please see inline.
Cheers,
Med
De : Roni Even [mailto:ron.even.tlv@gmail.com]
Envoyé : dimanche 26 janvier 2014 09:13
À : draft-ietf-pcp-description-option.all@tools.ietf.org<mailto:draft-ietf-pcp-description-option.all@tools.ietf.org>
Cc : ietf@ietf.org<mailto:ietf@ietf.org>; gen-art@ietf.org<mailto:gen-art@ietf.org>
Objet : Gen-ART LC review of draft-ietf-pcp-description-option-03

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-pcp-description-option-03

Reviewer: Roni Even

Review Date:2014–1-26

IETF LC End Date: 2014–2-4

IESG Telechat date:



Summary: This draft is almost ready for publication as a standard track RFC.





Major issues:



Minor issues:



1.       I am not sure what the use of the “description” option is. I can see how to create one but as for usage the only text I can see is “Querying the PCP server to get the description text of an existing mapping is out of scope.” I think that the document should provide information about the purpose of this option.

[Med] The main usage is for a user to associate a textual description with a mapping. This is captured by this text “   This option can be used by a user (or an application) to indicate a
   description associated with a given mapping such as "FTP server", "My
   remote access to my CP router", "Camera", "Network attached storage
   serve", etc.

”

[Roni Even] This is motivation but it is not clear why the user does it if no one can see it or does anything with it. It is kept on the server but not used by anyone?


Nits/editorial comments:

  1.  The IANA section should provide enough information to fill the registry http://www.iana.org/assignments/pcp-parameters/pcp-parameters.xml#options
[Med] Required information as per RFC6887 is provided in http://tools.ietf.org/html/draft-ietf-pcp-description-option-03#section-2.
[Roni Even] So section 5 should point to the definition is section 2