ietf
[Top] [All Lists]

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

2014-01-27 09:25:20
Hi,

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

Thanks,

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

Hi Med,
Inline
Roni

From: 
mohamed(_dot_)boucadair(_at_)orange(_dot_)com<mailto:mohamed(_dot_)boucadair(_at_)orange(_dot_)com>
 [mailto:mohamed(_dot_)boucadair(_at_)orange(_dot_)com]
Sent: 27 January, 2014 10:31 AM
To: Roni Even; 
draft-ietf-pcp-description-option(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org<mailto:draft-ietf-pcp-description-option(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org>
Cc: ietf(_at_)ietf(_dot_)org<mailto:ietf(_at_)ietf(_dot_)org>; 
gen-art(_at_)ietf(_dot_)org<mailto:gen-art(_at_)ietf(_dot_)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(_dot_)even(_dot_)tlv(_at_)gmail(_dot_)com]
Envoyé : dimanche 26 janvier 2014 09:13
À : 
draft-ietf-pcp-description-option(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org<mailto:draft-ietf-pcp-description-option(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org>
Cc : ietf(_at_)ietf(_dot_)org<mailto:ietf(_at_)ietf(_dot_)org>; 
gen-art(_at_)ietf(_dot_)org<mailto:gen-art(_at_)ietf(_dot_)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