Re: [CCAMP] I-D Action: draft-ietf-ccamp-rsvp-te-li-lb-04.txt

"Dongjie (Jimmy)" <jie.dong@huawei.com> Sat, 25 October 2014 06:11 UTC

Return-Path: <jie.dong@huawei.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2C9891A6FC2 for <ccamp@ietfa.amsl.com>; Fri, 24 Oct 2014 23:11:53 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.211
X-Spam-Level:
X-Spam-Status: No, score=-4.211 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] 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 WX1AfSmkViHM for <ccamp@ietfa.amsl.com>; Fri, 24 Oct 2014 23:11:51 -0700 (PDT)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 916FF1A1B30 for <ccamp@ietf.org>; Fri, 24 Oct 2014 23:11:50 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml403-hub.china.huawei.com) ([172.18.7.190]) by lhrrg01-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id BOA35404; Sat, 25 Oct 2014 06:11:49 +0000 (GMT)
Received: from nkgeml405-hub.china.huawei.com (10.98.56.36) by lhreml403-hub.china.huawei.com (10.201.5.217) with Microsoft SMTP Server (TLS) id 14.3.158.1; Sat, 25 Oct 2014 07:11:48 +0100
Received: from NKGEML512-MBX.china.huawei.com ([169.254.7.22]) by nkgeml405-hub.china.huawei.com ([10.98.56.36]) with mapi id 14.03.0158.001; Sat, 25 Oct 2014 14:11:44 +0800
From: "Dongjie (Jimmy)" <jie.dong@huawei.com>
To: Lou Berger <lberger@labn.net>, "draft-ietf-ccamp-rsvp-te-li-lb@tools.ietf.org" <draft-ietf-ccamp-rsvp-te-li-lb@tools.ietf.org>
Thread-Topic: [CCAMP] I-D Action: draft-ietf-ccamp-rsvp-te-li-lb-04.txt
Thread-Index: AQHP7DmxZe4di09DEUSAb36hmAw2LZw4nFYQgANiRwCAAQp7EIAAGJKAgAM2mPA=
Date: Sat, 25 Oct 2014 06:11:43 +0000
Message-ID: <76CD132C3ADEF848BD84D028D243C9273375C339@nkgeml512-mbx.china.huawei.com>
References: <20141020074350.21488.53873.idtracker@ietfa.amsl.com> <76CD132C3ADEF848BD84D028D243C92733758033@nkgeml512-mbx.china.huawei.com> <544805C0.8060103@labn.net> <76CD132C3ADEF848BD84D028D243C92733759B2E@nkgeml512-mbx.china.huawei.com> <5448F9E7.6050703@labn.net>
In-Reply-To: <5448F9E7.6050703@labn.net>
Accept-Language: en-US, zh-CN
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.111.96.76]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: http://mailarchive.ietf.org/arch/msg/ccamp/ugSqoOqh05WRas5xj9jRLyQHujk
Cc: "ccamp@ietf.org" <ccamp@ietf.org>
Subject: Re: [CCAMP] I-D Action: draft-ietf-ccamp-rsvp-te-li-lb-04.txt
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 25 Oct 2014 06:11:53 -0000

Hi Lou, 

Thank you for your comments and suggestions. We will submit a new revision to reflect them. Please see my replies inline:

> Jie,
>     Thank you for the quick response.  See below for responses.
> 
> It probably would also be worthwhile to add some words on when this
> extension should be used versus draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext
> <http://tools.ietf.org/wg/ccamp/draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext/>.

In some earlier version there used to be some words talking about draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext and this draft, we will add some words back in the new revision. 

Basically draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext defines extensions for configuring pro-active MPLS-TP OAMs, while draft-ietf-ccamp-rsvp-te-li-lb is about LI & LB provisioning, which belong to on-demand OAM functions.

> >> - Section 3.1,
> >> I think intermediate node behavior related to processing of the A is
> >> unchanged by this document so think you should drop the 3 lines that contain
> the phrase "...
> >> intermediate nodes ..."
> > Agree the behavior of intermediate node is unchanged, will remove the
> descriptions about intermediate node behavior in a new revision.
> >
> >> - Section 3.2
> >> The section is not clear if the A bit processing occurs / completes
> >> before Loopback related processing begins or occurs coincidentally.
> >> (I read the last sentence of the first paragraph saying the former
> >> and the last sentence of the 2nd paragraph saying the latter.)  Either way it
> should be clarified.
> > The A bit (lock) processing needs to be completed before initiating loopback
> request. The last sentence of the 2nd paragraph was to emphasize that when
> signaling loopback, the A bit should be kept set to indicate the LSP is still in lock
> mode.
> > We will rephrase the sentence to make it clearer.
> Great.  Please include appropriate conformance language.

Will fix this in the new revision.

> >> Where is the Loopback (B) bit / Attribute Flag defined? This document, right?
> >> Then the document, needs to state so in this section and in the IANA section.
> > The Loopback bit was defined in the end of section 2 "Extensions to RSVP-TE".
> To make it clearer, will use subsections for each extension in the new revision.
> 
> This is a good change to section 2. You might want to rename that section
> encodings or formats as the whole document is about rsvp-te extensions. You
> can take a look at section 4 of
> draft-ietf-ccamp-mpls-tp-rsvpte-ext-associated-lsp as a recent example.
> 
> This said, you still need to mention where it is defined (e.g., in section 3.1, add
> "defined above" the first time you mention the bit) and add it to the IANA
> section.
>

Will fix this in the new revision.

Many thanks, 
Jie

> >> Section 4, needs to be updated to follow RFC2360, See
> >> http://tools.ietf.org/html/draft-ietf-ccamp-mpls-tp-rsvpte-ext-associ
> >> ated-lsp-1
> >> 0
> >> for a recent example.
> > Thanks for pointing to this example. Will update the IANA section accordingly.
> >
> >> - Also please cleanup your id-nits, see
> >> http://tools.ietf.org/idnits?url=https://tools.ietf.org/id/draft-ietf
> >> -ccamp-rsvp-t
> >> e-li-lb-04.txt
> > Will clean this in the new revision.
> 
> Thanks,
> Lou
> 
> > Best regards,
> > Jie
> >
> >> Lou
> >>
> >> On 10/20/2014 3:58 AM, Dongjie (Jimmy) wrote:
> >>> Dear all,
> >>>
> >>> The only change in this new version is:
> >>>
> >>> - Update the references (one has become RFC)
> >>>
> >>> Your review and comments are highly appreciated.
> >>>
> >>> Many thanks,
> >>> Jie
> >>>
> >>>> -----Original Message-----
> >>>> From: CCAMP [mailto:ccamp-bounces@ietf.org] On Behalf Of
> >>>> internet-drafts@ietf.org
> >>>> Sent: Monday, October 20, 2014 3:44 PM
> >>>> To: i-d-announce@ietf.org
> >>>> Cc: ccamp@ietf.org
> >>>> Subject: [CCAMP] I-D Action: draft-ietf-ccamp-rsvp-te-li-lb-04.txt
> >>>>
> >>>>
> >>>> A New Internet-Draft is available from the on-line Internet-Drafts
> >> directories.
> >>>>  This draft is a work item of the Common Control and Measurement
> >>>> Plane Working Group of the IETF.
> >>>>
> >>>>         Title           : GMPLS RSVP-TE Extensions for Lock Instruct
> >> and
> >>>> Loopback
> >>>>         Authors         : Jie Dong
> >>>>                           Mach(Guoyi) Chen
> >>>>                           Zhenqiang Li
> >>>>                           Daniele Ceccarelli
> >>>> 	Filename        : draft-ietf-ccamp-rsvp-te-li-lb-04.txt
> >>>> 	Pages           : 8
> >>>> 	Date            : 2014-10-20
> >>>>
> >>>> Abstract:
> >>>>    This document specifies extensions to Resource Reservation Protocol
> -
> >>>>    Traffic Engineering (RSVP-TE) to support Lock Instruct (LI) and
> >>>>    Loopback (LB) mechanisms for Label Switched Paths (LSPs).  These
> >>>>    mechanisms are applicable to technologies which use Generalized
> >>>>    Multi-Protocol Label Switching (GMPLS) as control plane.
> >>>>
> >>>>
> >>>>
> >>>> The IETF datatracker status page for this draft is:
> >>>> https://datatracker.ietf.org/doc/draft-ietf-ccamp-rsvp-te-li-lb/
> >>>>
> >>>> There's also a htmlized version available at:
> >>>> http://tools.ietf.org/html/draft-ietf-ccamp-rsvp-te-li-lb-04
> >>>>
> >>>> A diff from the previous version is available at:
> >>>> http://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-rsvp-te-li-lb-04
> >>>>
> >>>>
> >>>> Please note that it may take a couple of minutes from the time of
> >>>> submission until the htmlized version and diff are available at
> tools.ietf.org.
> >>>>
> >>>> Internet-Drafts are also available by anonymous FTP at:
> >>>> ftp://ftp.ietf.org/internet-drafts/
> >>>>
> >>>> _______________________________________________
> >>>> CCAMP mailing list
> >>>> CCAMP@ietf.org
> >>>> https://www.ietf.org/mailman/listinfo/ccamp
> >>> _______________________________________________
> >>> CCAMP mailing list
> >>> CCAMP@ietf.org
> >>> https://www.ietf.org/mailman/listinfo/ccamp
> >>>
> >
> 
> 
> _______________________________________________
> CCAMP mailing list
> CCAMP@ietf.org
> https://www.ietf.org/mailman/listinfo/ccamp