Re: [RTG-DIR] RtgDir review: draft-ietf-ccamp-attribute-bnf-02.txt
Leeyoung <leeyoung@huawei.com> Mon, 10 October 2011 17:32 UTC
Return-Path: <leeyoung@huawei.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6407621F8C6B for <rtg-dir@ietfa.amsl.com>; Mon, 10 Oct 2011 10:32:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.165
X-Spam-Level:
X-Spam-Status: No, score=-6.165 tagged_above=-999 required=5 tests=[AWL=0.434, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 21fHNinOQjiK for <rtg-dir@ietfa.amsl.com>; Mon, 10 Oct 2011 10:32:37 -0700 (PDT)
Received: from usaga02-in.huawei.com (usaga02-in.huawei.com [206.16.17.70]) by ietfa.amsl.com (Postfix) with ESMTP id 10BFC21F8C6A for <rtg-dir@ietf.org>; Mon, 10 Oct 2011 10:32:35 -0700 (PDT)
Received: from huawei.com (localhost [127.0.0.1]) by usaga02-in.huawei.com (iPlanet Messaging Server 5.2 HotFix 2.14 (built Aug 8 2006)) with ESMTP id <0LSV00GD5228C0@usaga02-in.huawei.com> for rtg-dir@ietf.org; Mon, 10 Oct 2011 12:32:33 -0500 (CDT)
Received: from dfweml201-edg.china.huawei.com ([172.18.4.104]) by usaga02-in.huawei.com (iPlanet Messaging Server 5.2 HotFix 2.14 (built Aug 8 2006)) with ESMTPS id <0LSV00IF2228ES@usaga02-in.huawei.com> for rtg-dir@ietf.org; Mon, 10 Oct 2011 12:32:32 -0500 (CDT)
Received: from DFWEML404-HUB.china.huawei.com (10.193.5.203) by dfweml201-edg.china.huawei.com (172.18.9.107) with Microsoft SMTP Server (TLS) id 14.1.270.1; Mon, 10 Oct 2011 10:32:30 -0700
Received: from DFWEML501-MBX.china.huawei.com ([10.124.31.87]) by dfweml404-hub.china.huawei.com ([10.193.5.203]) with mapi id 14.01.0270.001; Mon, 10 Oct 2011 10:32:25 -0700
Date: Mon, 10 Oct 2011 17:32:25 +0000
From: Leeyoung <leeyoung@huawei.com>
In-reply-to: <4E92F104.6050808@labn.net>
X-Originating-IP: [10.47.128.252]
To: Lou Berger <lberger@labn.net>
Message-id: <7AEB3D6833318045B4AE71C2C87E8E171817F71C@DFWEML501-MBX.china.huawei.com>
MIME-version: 1.0
Content-type: text/plain; charset="us-ascii"
Content-language: en-US
Content-transfer-encoding: 7bit
Accept-Language: en-US, zh-CN
Thread-topic: [RTG-DIR] RtgDir review: draft-ietf-ccamp-attribute-bnf-02.txt
Thread-index: AQHMh09XiLds9zA5bkGk/Tu3a3mad5V11fHg
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
References: <7AEB3D6833318045B4AE71C2C87E8E171817F447@DFWEML501-MBX.china.huawei.com> <4E92F104.6050808@labn.net>
Cc: "draft-ietf-ccamp-attribute-bnf.all@tools.ietf.org" <draft-ietf-ccamp-attribute-bnf.all@tools.ietf.org>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "rtg-ads@tools.ietf.org" <rtg-ads@tools.ietf.org>
Subject: Re: [RTG-DIR] RtgDir review: draft-ietf-ccamp-attribute-bnf-02.txt
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/rtg-dir>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 10 Oct 2011 17:32:39 -0000
Lou, That's better. I have no more issue with this document. Young -----Original Message----- From: Lou Berger [mailto:lberger@labn.net] Sent: Monday, October 10, 2011 8:20 AM To: Leeyoung Cc: rtg-ads@tools.ietf.org; draft-ietf-ccamp-attribute-bnf.all@tools.ietf.org; rtg-dir@ietf.org Subject: Re: [RTG-DIR] RtgDir review: draft-ietf-ccamp-attribute-bnf-02.txt Young, Thank you for the comments. Please see below for in-line responses. Lou On 10/7/2011 5:26 PM, Leeyoung wrote: > Hello, > > I have been selected as the Routing Directorate reviewer for this draft. > The Routing Directorate seeks to review all routing or routing-related > drafts as they pass through IETF last call and IESG review. The purpose > of the review is to provide assistance to the Routing ADs. For more > information about the Routing Directorate, please see > http://www.ietf.org/iesg/directorate/routing.html > > Although these comments are primarily for the use of the Routing ADs, it > would be helpful if you could consider them along with any other IETF > Last Call comments that you receive, and strive to resolve them through > discussion or by updating the draft. > > Document: draft-ietf-ccamp-attribute-bnf-02.txt > > Reviewer: Young Lee > Review Date: 7 October 2011 > IETF LC End Date: 10 October 2011 > Intended Status: Standard track > > *Summary:* > I have no major concerns about this document that I think should be > resolved before publication. > > *Comments:* > This document is clearly written and easy to understand. > > *Major Issues:* > No major issues found. > > *Minor Issues:* > Section 3.2.1 > > > I am not sure if I understand this compatibility section written as follows: > > A node that does not support the LSP Attribute object formatting as > defined in this section will interpret the first present LSP > Attribute object as representing LSP operational status even when it > is intended to represent S2L sub-LSP status. It is unclear if this > is a significant issue as the LSP Attribute object is currently > considered to be an unsuitable mechanism for reporting operational > status of P2MP LSPs, for example see Section 2.1 of [NO-PHP-OOB]. > The intent of this document is to correct this limitation and it is > expected that networks that wish to make use of such operational > reporting will deploy this extension. > > If the node that does not support this new LSP attribute object, how can > it interprets the first > Present LSP Attribute object as LSP operational status if the LSP > attribute object is not a suitable > mechanism currently for reporting operational status of P2MP LSPs? > > It sounds like a contradictory statement as I read. Please clarify this > section if needed. Does the following revision clarify the paragraph? OLD: > A node that does not support the LSP Attribute object formatting as > defined in this section will interpret the first present LSP > Attribute object as representing LSP operational status even when it > is intended to represent S2L sub-LSP status. NEW: A node that supports [RFC4875] and [RFC5420], but not this document, will interpret the first LSP Attribute object present in a received message, which is formatted as described in this document, as representing LSP operational status rather than S2L sub-LSP status. Lou > > > *Nits:* > None identified > > >