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

"Roni Even" <ron.even.tlv@gmail.com> Mon, 27 January 2014 11:34 UTC

Return-Path: <ron.even.tlv@gmail.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 427031A01E8; Mon, 27 Jan 2014 03:34:22 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.999
X-Spam-Level:
X-Spam-Status: No, score=-1.999 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, SPF_PASS=-0.001] 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 ZC22RVCiZgjA; Mon, 27 Jan 2014 03:34:18 -0800 (PST)
Received: from mail-ee0-x233.google.com (mail-ee0-x233.google.com [IPv6:2a00:1450:4013:c00::233]) by ietfa.amsl.com (Postfix) with ESMTP id 8D5831A01E4; Mon, 27 Jan 2014 03:34:17 -0800 (PST)
Received: by mail-ee0-f51.google.com with SMTP id b57so2199343eek.38 for <multiple recipients>; Mon, 27 Jan 2014 03:34:15 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:references:in-reply-to:subject:date:message-id :mime-version:content-type:thread-index:content-language; bh=pHb8s3pVsDXVxNNIHuXFf9iYCvfCD+jjspbwRHMZrNE=; b=WeLp9V62jEOKqmn0gKjcMGdjcqkwopiZjyAFRWqygdcu4AKJLhDW3rWOjHro3tqOWW 9kJlD13+OoRZufKwgFfBFUkFfwIeUXPRNaDhwzix4j8vHrmZPsvl/PHi85hG93S/dMEn RUcPbGuPAvpBm4oxwVZZIIGRZ9ievC/fyryZTMfQWsGQgYPEOKs3QnwaFt4K/sUI0i9Y ygYt1KHdjV7Ji2hstgmaXFtKwWlpW6mFDSBGpRlDVZgoj56X5Sj3STbsKV09puDUHbs5 ZTBe+dl53tcJAd5HHnGZ/yWzVeF6ZxTxYbfhvx1YWw2taS8u5H2Xu4FioTokmAjq0ryl ITVQ==
X-Received: by 10.14.6.129 with SMTP id 1mr274364een.99.1390822454872; Mon, 27 Jan 2014 03:34:14 -0800 (PST)
Received: from RoniE ([109.66.49.55]) by mx.google.com with ESMTPSA id 46sm41351623ees.4.2014.01.27.03.34.10 for <multiple recipients> (version=TLSv1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 27 Jan 2014 03:34:13 -0800 (PST)
From: Roni Even <ron.even.tlv@gmail.com>
To: mohamed.boucadair@orange.com, draft-ietf-pcp-description-option.all@tools.ietf.org
References: <02db01cf1a6e$62eb8280$28c28780$@gmail.com> <94C682931C08B048B7A8645303FDC9F36F4736082C@PUEXCB1B.nanterre.francetelecom.fr> <033d01cf1b42$854c10e0$8fe432a0$@gmail.com> <94C682931C08B048B7A8645303FDC9F36F473608DF@PUEXCB1B.nanterre.francetelecom.fr>
In-Reply-To: <94C682931C08B048B7A8645303FDC9F36F473608DF@PUEXCB1B.nanterre.francetelecom.fr>
Subject: RE: Gen-ART LC review of draft-ietf-pcp-description-option-03
Date: Mon, 27 Jan 2014 13:30:24 +0200
Message-ID: <036401cf1b53$2ec6cdd0$8c546970$@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="----=_NextPart_000_0365_01CF1B63.F2517290"
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQF6/p6mIwEjavahOcoGuw0nlckk9wI8szcDAx3Ucj0Btw/0U5sIHEGg
Content-Language: en-us
Cc: gen-art@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 11:34:22 -0000

Hi Med,

I think some text will be useful

Roni

 

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

 

Re-,

The description is used by the entity managing the PCP server. The document
does not elaborate on such usage as this is deployment-specific: e.g.,  (1)
it can be used as  helper to clean the mapping table, (2) in some
deployments that make use of a portal to instruct pcp mappings, this option
can be used to store a subscriber-identifier, etc. We can add some text if
you think this is useful to be mentioned in the document.

For the second point, I updated Section 5 with a pointer to Section 2.
Thanks.

Cheers,

Med

De : Roni Even [mailto:ron.even.tlv@gmail.com] 
Envoyé : lundi 27 janvier 2014 10:31
À : BOUCADAIR Mohamed IMT/OLN;
draft-ietf-pcp-description-option.all@tools.ietf.org
Cc : ietf@ietf.org; gen-art@ietf.org
Objet : RE: Gen-ART LC review of draft-ietf-pcp-description-option-03

 

Hi Med, 

Inline

Roni

 

From: 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
Cc: ietf@ietf.org; 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
Cc : ietf@ietf.org; 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