Re: [CCAMP] FW: I-D Action: draft-ietf-ccamp-gmpls-general-constraints-ospf-te-06.txt

Leeyoung <leeyoung@huawei.com> Fri, 31 January 2014 19:24 UTC

Return-Path: <leeyoung@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 0FF9D1A0292 for <ccamp@ietfa.amsl.com>; Fri, 31 Jan 2014 11:24:47 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.725
X-Spam-Level:
X-Spam-Status: No, score=-3.725 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HK_RANDOM_ENVFROM=0.001, HK_RANDOM_FROM=0.999, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RP_MATCHES_RCVD=-0.535, SPF_PASS=-0.001, T_HTML_ATTACH=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 vhCilMwbNQTn for <ccamp@ietfa.amsl.com>; Fri, 31 Jan 2014 11:24:41 -0800 (PST)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) by ietfa.amsl.com (Postfix) with ESMTP id 2FA641A0286 for <ccamp@ietf.org>; Fri, 31 Jan 2014 11:24:40 -0800 (PST)
Received: from 172.18.7.190 (EHLO lhreml204-edg.china.huawei.com) ([172.18.7.190]) by lhrrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id BAR21578; Fri, 31 Jan 2014 19:24:36 +0000 (GMT)
Received: from LHREML406-HUB.china.huawei.com (10.201.5.243) by lhreml204-edg.china.huawei.com (172.18.7.223) with Microsoft SMTP Server (TLS) id 14.3.158.1; Fri, 31 Jan 2014 19:24:08 +0000
Received: from DFWEML701-CHM.china.huawei.com (10.193.5.50) by lhreml406-hub.china.huawei.com (10.201.5.243) with Microsoft SMTP Server (TLS) id 14.3.158.1; Fri, 31 Jan 2014 19:24:34 +0000
Received: from DFWEML706-CHM.china.huawei.com ([169.254.8.193]) by dfweml701-chm.china.huawei.com ([169.254.1.21]) with mapi id 14.03.0158.001; Fri, 31 Jan 2014 11:24:27 -0800
From: Leeyoung <leeyoung@huawei.com>
To: Lou Berger <lberger@labn.net>
Thread-Topic: [CCAMP] FW: I-D Action: draft-ietf-ccamp-gmpls-general-constraints-ospf-te-06.txt
Thread-Index: AQHPHq3Th/udhy2zekKfXLom0svhpZqfMc5A
Date: Fri, 31 Jan 2014 19:24:27 +0000
Message-ID: <7AEB3D6833318045B4AE71C2C87E8E1729BB569C@dfweml706-chm.china.huawei.com>
References: <7AEB3D6833318045B4AE71C2C87E8E1729BA5098@dfweml510-mbx.china.huawei.com> <143220693c8.2764.9b4188e636579690ba6c69f2c8a0f1fd@labn.net> <52DD7ECD.5070407@labn.net> <7AEB3D6833318045B4AE71C2C87E8E1729BB55EC@dfweml706-chm.china.huawei.com> <52EBE3D3.80300@labn.net>
In-Reply-To: <52EBE3D3.80300@labn.net>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator:
x-originating-ip: [10.47.130.110]
Content-Type: multipart/mixed; boundary="_003_7AEB3D6833318045B4AE71C2C87E8E1729BB569Cdfweml706chmchi_"
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Cc: CCAMP <ccamp@ietf.org>
Subject: Re: [CCAMP] FW: I-D Action: draft-ietf-ccamp-gmpls-general-constraints-ospf-te-06.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: Fri, 31 Jan 2014 19:24:47 -0000

Hi Lou,

All corrected into the document. Acee's the only comment unresolved is on the [GEN-Encode], which will be worked out there.

Here's the working version and the idnits results. I will submit the new version shortly. 

Young

-----Original Message-----
From: CCAMP [mailto:ccamp-bounces@ietf.org] On Behalf Of Lou Berger
Sent: Friday, January 31, 2014 11:57 AM
To: Leeyoung
Cc: CCAMP
Subject: Re: [CCAMP] FW: I-D Action: draft-ietf-ccamp-gmpls-general-constraints-ospf-te-06.txt

Young,
	I think you're alignment with the new node attribute TLV usage is fine, but the iana implications needs to reflect the change as well.

1) in the two places you replaced TDB with 14, I think you should have "TBA (by IANA)" or "14 (suggested, to be assigned by IANA)"

2) in the two places you replaced TDB with 26, I think you should have "TBA (by IANA)" or "26 (suggested, to be assigned by IANA)"

YOUNG>> Done. 

3) Also you need to specifically identify the registry:

OLD
   This document introduces the following sub-TLVs of Node Attribute
   TLV (Value 5):

NEW
   This document defines the a new sub-TLV of the Node Attribute
   TLV (Value 5). The assignment of the following new type in the
   "Types for sub-TLVs of TE Node Attribute TLV" portion of the
   "Open Shortest Path First (OSPF) Traffic Engineering TLVs"
   registry is needed:

and

OLD
   This document introduces the following sub-TLV of TE Link TLV (Value
   2):
NEW
   This document defines the a new sub-TLV of the TE Link TLV
   (Value 2). The assignment of the following new type in the
   "Types for sub-TLVs of TE Link TLV" portion of the
   "Open Shortest Path First (OSPF) Traffic Engineering TLVs"
   registry is needed:

YOUNG>> Done. 

Once these changes are made, please submit.

(I of course defer to Acee on his comments.)

Thanks,
Lou

On 1/31/2014 12:28 PM, Leeyoung wrote:
> Hi Lou,
> 
> Please see my comment inline.
> 
> Here's the working version (07) that has reflected all your comments and Acee's and the idnits results. 
> 
> Let me know if this is ready to publish.
> 
> Thanks.
> Young
> 
> -----Original Message-----
> From: Lou Berger [mailto:lberger@labn.net]
> Sent: Monday, January 20, 2014 1:54 PM
> To: Leeyoung
> Cc: CCAMP; Acee Lindem
> Subject: Re: [CCAMP] FW: I-D Action: 
> draft-ietf-ccamp-gmpls-general-constraints-ospf-te-06.txt
> 
> 
> Young, (all),
> 
> 
> There are few minor items in this document.
> 
> See
> http://tools.ietf.org/idnits?url=http://tools.ietf.org/id/draft-ietf-c
> camp-gmpls-general-constraints-ospf-te-06.txt
> for idnits
> 
>   == Line 273 has weird spacing: '...ictions  can...'
> 
>   == Line 488 has weird spacing: '...   of   these ...'
> 
>   == Line 489 has weird spacing: '...cluding   thos...'
> 
>   == Line 490 has weird spacing: '... not be   cons...'
> 
>   == Line 499 has weird spacing: '...   and   shall...'
> 
>   == (1 more instance...)
> 
>   == Missing Reference: 'Gen-Encode' is mentioned on line 256, but not 
> defined
> 
>   == Missing Reference: 'RFC 3630' is mentioned on line 308, but not 
> defined
> 
>   == Missing Reference: 'RFC 4203' is mentioned on line 308, but not 
> defined
> 
>   == Missing Reference: 'RFC5920' is mentioned on line 312, but not 
> defined
> 
> YOUNG>> Done.
> 
> - section 2.1
> s/ingress/input
> s/egress/output
> 
> YOUNG>> Done.
> 
> There a number of places where the "new" Generic Node Attribute is still mentioned (just search for it).  These need to be aligned with the change to use the Node Attribute TLV.
> 
> YOUNG>> cleaned up. 
> 
> The following comments remain unaddressed:
>>
>> - Section 4, last paragraph
>>   Text should be aligned with the preexisting TE procedures as defined
>>   in Section 3 of RFC3630.
> How about replace the paragraph with:
>    As with other TE information, an implementation typically
>    take measures to avoid rapid and frequent updates of routing
>    information that could cause the routing network to become swamped.
>    See [RFC3630] Section 3 for related details.
> 
> YOUNG>> Replaced. 
> 
> I didn't check to see if Acee's comments were addressed
> 
> YOUNG>> Addressed. See the email and the newer working version. 
> 
> That's it on this one.
> 
> Lou
> 
> On 12/23/2013 7:34 PM, Lou Berger wrote:
>> Thank you Young. I'm hoping we hear about the pending IPR disclosure 
>> sometime soon...
>>
>> Lou
>>
>>
>> On December 23, 2013 6:32:40 PM Leeyoung <leeyoung@huawei.com> wrote:
>>> Hi Lou,
>>>
>>> To avoid the expiration of the draft, we have updated this draft. In 
>>> doing so, I believe we resolved most of the pending issues you and 
>>> Acee raised during the WG LC process.
>>> Regards,
>>> Young
>>>
>>>
>>>
>>> -----Original Message-----
>>> From: CCAMP [mailto:ccamp-bounces@ietf.org] On Behalf Of 
>>> internet-drafts@ietf.org
>>> Sent: Monday, December 23, 2013 5:13 PM
>>> To: i-d-announce@ietf.org
>>> Cc: ccamp@ietf.org
>>> Subject: [CCAMP] I-D Action: 
>>> draft-ietf-ccamp-gmpls-general-constraints-ospf-te-06.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           : OSPF-TE Extensions for General Network Element Constraints
>>>         Authors         : Fatai Zhang
>>>                           Young Lee
>>>                           Jianrui Han
>>>                           Greg Bernstein
>>>                           Yunbin Xu
>>> 	Filename        : draft-ietf-ccamp-gmpls-general-constraints-ospf-te-06.txt
>>> 	Pages           : 13
>>> 	Date            : 2013-12-23
>>>
>>> Abstract:
>>>    Generalized Multiprotocol Label Switching (GMPLS) can be used to
>>>    control a wide variety of technologies including packet switching
>>>    (e.g., MPLS), time-division (e.g., SONET/SDH, Optical Transport
>>>    Network (OTN)), wavelength (lambdas), and spatial switching (e.g.,
>>>    incoming port or fiber to outgoing port or fiber). In some of these
>>>    technologies, network elements and links may impose additional
>>>    routing constraints such as asymmetric switch connectivity, non-
>>>    local label assignment, and label range limitations on links. This
>>>    document describes Open Shortest Path First (OSPF) routing protocol
>>>    extensions to support these kinds of constraints under the control
>>>    of GMPLS.
>>>
>>>
>>>
>>> The IETF datatracker status page for this draft is:
>>> https://datatracker.ietf.org/doc/draft-ietf-ccamp-gmpls-general-cons
>>> t
>>> raints-ospf-te/
>>>
>>> There's also a htmlized version available at:
>>> http://tools.ietf.org/html/draft-ietf-ccamp-gmpls-general-constraint
>>> s
>>> -ospf-te-06
>>>
>>> A diff from the previous version is available at:
>>> http://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-gmpls-general-cons
>>> t
>>> raints-ospf-te-06
>>>
>>>
>>> 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