Re: [CCAMP] I-D Action: draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext-09.txt

WG,
	Please review and comment on changes.  See
http://tools.ietf.org/rfcdiff?difftype=--hwdiff&url2=draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext-10.txt

Recall that this document is post WG LC.

Elisa,
	Thank you for the update.  Some comments.
- Please review IDNITs and ensure that the document is clean, see
http://tools.ietf.org/idnits?url=http://tools.ietf.org/id/draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext-10.txt

- Missing metrics:  line 684 (using lines numbers from nits URL),

- Threshold definition, lines 687/8:   I really don't think these
definitions are adequate to ensure consistent implementation. How about:
 Loss Threshold: indicates the MaxLMIntervalLoss threshold (measured in
 packets) as defined in [RFC6374].

- Line 742-4, Delay Threshold: is this one way or two way delay?  I
don't see how you can specify a default.

- line 784, Refresh Timer.  This should have a reference and perhaps
just say "MUST carry the same value used in the Refresh Timer field
defined in [rfc6427]."

You might want to discuss your proposed changes on the list and get
agreement before rev'ing the document again.  Of course, rev numbers are
cheap too...

Lou

On 10/11/2012 11:46 AM, Elisa Bellagamba wrote:
> 
> Hi Lou,
> 
> we have just uploaded version 10 which is inserting back the refresh timer field in accordance with RFC 6427 and fixing the TBD values to the ones recommended in RFC 6375.
> 
> Thanks,
> Elisa 
> 
> -----Original Message-----
> From: ccamp-bounces@ietf.org [mailto:ccamp-bounces@ietf.org] On Behalf Of Lou Berger
> Sent: den 8 oktober 2012 18:27
> To: draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext@tools.ietf.org
> Cc: ccamp@ietf.org
> Subject: Re: [CCAMP] I-D Action: draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext-09.txt
> 
> Authors,
> 	I see you still have some default values called out as TBD.
> Can you please send your proposed defaults to the WG so that we can ensure consensus on these? (Obviously, these need to be filled in before any publication request.)
> 
> Also, I see you removed the units from the Refresh Timer field.  Was this intentional?  I think some unit of measure is needed.
> 
> Thanks,
> Lou
> 
> On 10/7/2012 2:23 PM, internet-drafts@ietf.org wrote:
>>
>> 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           : Configuration of Pro-Active Operations, Administration, and Maintenance (OAM) Functions for MPLS-based Transport Networks using RSVP-TE
>> 	Author(s)       : Elisa Bellagamba
>>                           Loa Andersson
>>                           Pontus Skoldstrom
>>                           Dave Ward
>>                           Attila Takacs
>> 	Filename        : draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext-09.txt
>> 	Pages           : 22
>> 	Date            : 2012-10-07
>>
>> Abstract:
>>    This specification describes the configuration of pro-active MPLS-TP
>>    Operations, Administration, and Maintenance (OAM) Functions for a
>>    given LSP using a set of TLVs that are carried by the RSVP-TE
>>    protocol.
>>
>>    This document is a product of a joint Internet Engineering Task Force
>>    (IETF) / International Telecommunication Union Telecommunication
>>    Standardization Sector (ITU-T) effort to include an MPLS Transport
>>    Profile within the IETF MPLS and PWE3 architectures to support the
>>    capabilities and functionalities of a packet transport network.
>>
>>
>> The IETF datatracker status page for this draft is:
>> https://datatracker.ietf.org/doc/draft-ietf-ccamp-rsvp-te-mpls-tp-oam-
>> ext
>>
>> There's also a htmlized version available at:
>> http://tools.ietf.org/html/draft-ietf-ccamp-rsvp-te-mpls-tp-oam-ext-09
>>
>> A diff from the previous version is available at:
>> http://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-rsvp-te-mpls-tp-oam-
>> ext-09
>>
>>
>> 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
> 
> 
> 
> 