ietf
[Top] [All Lists]

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

2014-01-27 04:06:12
Hi,

This should be stated and I assume that it is not for standardization but
not clear why the entity operating the PCP server needs it, is it described
in another document.

I am curious why querying the PCP server is ?out of scope? and not is ?for
further work?.

Roni

 

From: Reinaldo Penno (repenno) [mailto:repenno(_at_)cisco(_dot_)com] 
Sent: 27 January, 2014 11:39 AM
To: Roni Even; mohamed(_dot_)boucadair(_at_)orange(_dot_)com;
draft-ietf-pcp-description-option(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org
Cc: ietf(_at_)ietf(_dot_)org; gen-art(_at_)ietf(_dot_)org
Subject: Re: Gen-ART LC review of draft-ietf-pcp-description-option-03

 

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>
Date: Monday, January 27, 2014 at 1:31 AM
To: "mohamed(_dot_)boucadair(_at_)orange(_dot_)com" 
<mohamed(_dot_)boucadair(_at_)orange(_dot_)com>,
"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>
Cc: "ietf(_at_)ietf(_dot_)org" <ietf(_at_)ietf(_dot_)org>, 
"gen-art(_at_)ietf(_dot_)org" <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>
Resent-To: <dthaler(_at_)microsoft(_dot_)com>, <dwing(_at_)cisco(_dot_)com>,
"mohamed(_dot_)boucadair(_at_)orange(_dot_)com" 
<mohamed(_dot_)boucadair(_at_)orange(_dot_)com>, Cisco
Employee <repenno(_at_)cisco(_dot_)com>, 
<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] 
Sent: 27 January, 2014 10:31 AM
To: Roni Even; 
draft-ietf-pcp-description-option(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org
Cc: ietf(_at_)ietf(_dot_)org; 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
Cc : ietf(_at_)ietf(_dot_)org; 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