Re: [mpls] Ben Campbell's No Objection on draft-ietf-mpls-app-aware-tldp-08: (with COMMENT)

Ben Campbell <ben@nostrum.com> Sat, 24 June 2017 05:02 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0C5DF129B4F; Fri, 23 Jun 2017 22:02:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.881
X-Spam-Level:
X-Spam-Status: No, score=-1.881 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-0.001, T_SPF_HELO_PERMERROR=0.01, T_SPF_PERMERROR=0.01] autolearn=ham autolearn_force=no
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 nY6AH5yDdOWl; Fri, 23 Jun 2017 22:02:44 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A2ECD129B4D; Fri, 23 Jun 2017 22:02:44 -0700 (PDT)
Received: from [10.0.1.63] (cpe-66-25-7-22.tx.res.rr.com [66.25.7.22]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id v5O52fd3005652 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Sat, 24 Jun 2017 00:02:42 -0500 (CDT) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host cpe-66-25-7-22.tx.res.rr.com [66.25.7.22] claimed to be [10.0.1.63]
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
From: Ben Campbell <ben@nostrum.com>
In-Reply-To: <D572A0E8.F0CC1%sesale@juniper.net>
Date: Sat, 24 Jun 2017 00:02:40 -0500
Cc: The IESG <iesg@ietf.org>, "draft-ietf-mpls-app-aware-tldp@ietf.org" <draft-ietf-mpls-app-aware-tldp@ietf.org>, "mpls-chairs@ietf.org" <mpls-chairs@ietf.org>, "loa@pi.nu" <loa@pi.nu>, "mpls@ietf.org" <mpls@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <7280838F-4BC6-473A-AF72-28B3C0B05785@nostrum.com>
References: <149814291691.30632.16984845315293045969.idtracker@ietfa.amsl.com> <D572A0E8.F0CC1%sesale@juniper.net>
To: Santosh Esale <sesale@juniper.net>
X-Mailer: Apple Mail (2.3273)
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/cwpGSViyBzPc-e3vE0rMAT4ALRg>
Subject: Re: [mpls] Ben Campbell's No Objection on draft-ietf-mpls-app-aware-tldp-08: (with COMMENT)
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 24 Jun 2017 05:02:47 -0000

Hi,

Your responses resolve all of my comments.

Thanks!

Ben.

> On Jun 23, 2017, at 4:08 PM, Santosh Esale <sesale@juniper.net> wrote:
> 
> Hello Ben, Please see inline for answers.
> 
> 
> On 6/22/17, 7:48 AM, "Ben Campbell" <ben@nostrum.com> wrote:
> 
>> Ben Campbell has entered the following ballot position for
>> draft-ietf-mpls-app-aware-tldp-08: No Objection
>> 
>> When responding, please keep the subject line intact and reply to all
>> email addresses included in the To and CC lines. (Feel free to cut this
>> introductory paragraph, however.)
>> 
>> 
>> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
>> for more information about IESG DISCUSS and COMMENT positions.
>> 
>> 
>> The document, along with other ballot positions, can be found here:
>> https://datatracker.ietf.org/doc/draft-ietf-mpls-app-aware-tldp/
>> 
>> 
>> 
>> ----------------------------------------------------------------------
>> COMMENT:
>> ----------------------------------------------------------------------
>> 
>> (I cleared my discuss. Alexey pointed out that the registration policies
>> for
>> unassigned ranges are explicitly mentioned in the table itself.
>> Apologies, I
>> missed that.)
>> 
>> -General:  When you use the word "application", is a person skilled in
>> MPLS
>> automatically going to think in terms of the sort of "applications" listed
>> here? If not, it would be good to have a paragraph early in the
>> introduction
>> that describes what you mean by "applications". As an application-layer
>> person,
>> I tend to think of it in terms of application-layer applications. Am I
>> correct
>> to assume that it would not make sense to register, for example, HTTP in
>> the
>> the LDP Targeted Application Identifier registry?
> Correct. 
> Also added the following definition of application to first paragraph -
> Targeted 
> LDP application is an application that uses tLDP session to exchange
> information 
> such as FEC-Label bindings with a peer LSR in the network.
> 
> 
>> 
>> - 1.1: I noticed a number instances of at least "may" in lower case. If
>> your
>> intent is that lower case words not be treated as keywords, please
>> consider
>> using the new boilerplate from RFC 8174
> 
> Updated.
> 
>> 
>> Nits:
>> 
>> -1, first paragraph: "LDP uses extended discovery mechanism..."
>> Missing article before "discovery".
> 
> Updated.
>> 
>> -2.2, paragraph 4 and 5, "... a maximum value, as 0xffff.": should "as" be
>> "such as"?
> 
> Yes. Updated.
> 
>> 
>> -2.2, paragraphs 6 and 7: the "it" in the opening sentence of each
>> paragraph
>> has an unclear antecedent.
> 
> ‘it’ was referring to an antecedent from the previous paragraph.
> Can ‘it' jump over to refer to the noun in the previous paragraph ? :)
> Updated to replace ‘it’ with the ‘noun’ .
>> 
> Thanks,
> Santosh on behalf of Authors
>