Re: [Isis-wg] I-D Action:draft-ietf-ccamp-isis-interas-te-extension-03.txt

"Les Ginsberg (ginsberg)" <ginsberg@cisco.com> Wed, 27 August 2008 08:37 UTC

Return-Path: <isis-wg-bounces@ietf.org>
X-Original-To: isis-wg-archive@megatron.ietf.org
Delivered-To: ietfarch-isis-wg-archive@core3.amsl.com
Received: from [127.0.0.1] (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 3175F3A69BD; Wed, 27 Aug 2008 01:37:34 -0700 (PDT)
X-Original-To: isis-wg@core3.amsl.com
Delivered-To: isis-wg@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 2F8083A6BFB for <isis-wg@core3.amsl.com>; Wed, 27 Aug 2008 01:37:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id lO1U9Afc7hoH for <isis-wg@core3.amsl.com>; Wed, 27 Aug 2008 01:37:30 -0700 (PDT)
Received: from sj-iport-2.cisco.com (sj-iport-2.cisco.com [171.71.176.71]) by core3.amsl.com (Postfix) with ESMTP id E50183A69BD for <isis-wg@ietf.org>; Wed, 27 Aug 2008 01:37:30 -0700 (PDT)
X-IronPort-AV: E=Sophos;i="4.32,277,1217808000"; d="scan'208";a="78478696"
Received: from sj-dkim-4.cisco.com ([171.71.179.196]) by sj-iport-2.cisco.com with ESMTP; 27 Aug 2008 08:36:45 +0000
Received: from sj-core-2.cisco.com (sj-core-2.cisco.com [171.71.177.254]) by sj-dkim-4.cisco.com (8.12.11/8.12.11) with ESMTP id m7R8agQg027746; Wed, 27 Aug 2008 01:36:42 -0700
Received: from xbh-sjc-211.amer.cisco.com (xbh-sjc-211.cisco.com [171.70.151.144]) by sj-core-2.cisco.com (8.13.8/8.13.8) with ESMTP id m7R8agLE001562; Wed, 27 Aug 2008 08:36:42 GMT
Received: from xmb-sjc-222.amer.cisco.com ([128.107.191.106]) by xbh-sjc-211.amer.cisco.com with Microsoft SMTPSVC(6.0.3790.1830); Wed, 27 Aug 2008 01:36:41 -0700
X-MimeOLE: Produced By Microsoft Exchange V6.5
Content-class: urn:content-classes:message
MIME-Version: 1.0
Date: Wed, 27 Aug 2008 01:38:15 -0700
Message-ID: <AE36820147909644AD2A7CA014B1FB52064AC7C1@xmb-sjc-222.amer.cisco.com>
In-Reply-To: <6DEB3C8154434B10BE83D8D8B66053AE@M55527>
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
Thread-Topic: I-D Action:draft-ietf-ccamp-isis-interas-te-extension-03.txt
Thread-Index: AckHIhk4lJ0bxX0BSj6xXiavgQHVHAA+6HLQ
References: <6DEB3C8154434B10BE83D8D8B66053AE@M55527>
From: "Les Ginsberg (ginsberg)" <ginsberg@cisco.com>
To: Mach Chen <mach@huawei.com>, isis mailing list <isis-wg@ietf.org>
X-OriginalArrivalTime: 27 Aug 2008 08:36:41.0780 (UTC) FILETIME=[07B13740:01C90820]
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; l=4859; t=1219826205; x=1220690205; c=relaxed/simple; s=sjdkim4002; h=Content-Type:From:Subject:Content-Transfer-Encoding:MIME-Version; d=cisco.com; i=ginsberg@cisco.com; z=From:=20=22Les=20Ginsberg=20(ginsberg)=22=20<ginsberg@cisc o.com> |Subject:=20RE=3A=20I-D=20Action=3Adraft-ietf-ccamp-isis-in teras-te-extension-03.txt |Sender:=20; bh=YdVN08OzS6o7HJMasMVhIW7DqwCLw66q28jzbkmduFE=; b=abgl0xW+hwyEnnamo0rqVXRUVeKMBrja7Rbugyt0fqIUqG/NDYRX7QR0W7 oMbJSwZ0IWhhBv8bHyU/0L/w2axDb/K6fduEmp+MG0SGhN++muuZ9A8RQdp1 SsA9eURByL;
Authentication-Results: sj-dkim-4; header.From=ginsberg@cisco.com; dkim=pass ( sig from cisco.com/sjdkim4002 verified; );
Cc: Hannes Gredler <hannes@juniper.net>
Subject: Re: [Isis-wg] I-D Action:draft-ietf-ccamp-isis-interas-te-extension-03.txt
X-BeenThere: isis-wg@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: IETF IS-IS working group <isis-wg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/pipermail/isis-wg>
List-Post: <mailto:isis-wg@ietf.org>
List-Help: <mailto:isis-wg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=subscribe>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: isis-wg-bounces@ietf.org
Errors-To: isis-wg-bounces@ietf.org

Mach -

One editorial nit:

In Section 3.1 2nd paragraph page 8:

s/GNINFO/GENINFO

On a more substantive note, the discussion of sub-TLV allocation that
you added into Section 6.2 is clearly an improvement, but it still gives
me cause for concern. You say:

" the sub-TLVs which are defined in 
   [ISIS-TE], [ISIS-TE-V3] and other documents for describing the TE 
   properties of an TE link are applicable to describe an inter-AS TE 
   link and MAY be included in the Inter-AS Reachability TLV when 
   adverting inter-AS TE links. So, these sub-TLVs need to be 
   registered in the ISIS sub-TLV registry for TLV 141. And in order to 
   simplify the registration, we suggest using the same registry value 
   as they are registered in the ISIS sub-TLV registry for TLV 22."

This suggests that:

1)Every subTLV in the TLV22 registry needs to be added to the TLV141
subTLV registry - and any future definitions have to be added in both
places regardless of whether they are for TLV22 or TLV141 (or both).

2)It may be acceptable for a subTLV which is used in both TLV22 and
TLV141 to have different number assignments in the two TLV registries
(though you recommend against this).

I would prefer that we insist upon a single shared registry so that the
issues of duplicate registrations and/or different numerical assignments
never arises.

This would then eliminate the need to repeat the existing TLV22 subTLV
assignments in this document (as you do immediately below the quoted
text) - and it would eliminate confusion between the statement above and
the statement later in the same section:

"...the new sub-TLVs MUST be defined from a sub-
   TLV registry which is shared by these two TLVs."

   Les

> -----Original Message-----
> From: Mach Chen [mailto:mach@huawei.com]
> Sent: Monday, August 25, 2008 7:19 PM
> To: isis mailing list
> Cc: Hannes Gredler; Les Ginsberg (ginsberg)
> Subject: Fw: I-D
Action:draft-ietf-ccamp-isis-interas-te-extension-03.txt
> 
> Hi,
> 
> We have already updated the draft according to the comments received
> during
> the ISIS WG Last Call from Hannes Gredler and Les Ginsberg.
> 
> Hannes and Les, could you please confirm whether this revision
addresses
> your comments?
> 
> 
> Best regards,
> Mach
> 
> --------------------------------------------------
> From: <Internet-Drafts@ietf.org>
> Sent: Monday, August 25, 2008 11:30 AM
> To: <i-d-announce@ietf.org>
> Cc: <ccamp@ops.ietf.org>
> Subject: I-D Action:draft-ietf-ccamp-isis-interas-te-extension-03.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           : ISIS Extensions in Support of Inter-AS
Multiprotocol
> > Label Switching (MPLS) and Generalized MPLS (GMPLS) Traffic
Engineering
> > Author(s)       : M. Chen, R. Zhang
> > Filename        : draft-ietf-ccamp-isis-interas-te-extension-03.txt
> > Pages           : 20
> > Date            : 2008-08-24
> >
> > This document describes extensions to the ISIS (ISIS) protocol to
> > support Multiprotocol Label Switching (MPLS) and Generalized MPLS
> > (GMPLS) Traffic Engineering (TE) for multiple Autonomous Systems
> > (ASes). It defines ISIS-TE extensions for the flooding of TE
> > information about inter-AS links which can be used to perform inter-
> > AS TE path computation.
> >
> >
> >
> > No support for flooding information from within one AS to another AS
> > is proposed or defined in this document.
> >
> > Conventions used in this document
> >
> > The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT",
> > "SHOULD", "SHOULD NOT", "RECOMMENDED", "MAY", and "OPTIONAL" in this
> > document are to be interpreted as described in RFC-2119 [RFC2119].
> >
> > A URL for this Internet-Draft is:
> >
http://www.ietf.org/internet-drafts/draft-ietf-ccamp-isis-interas-te-
> extension-03.txt
> >
> > Internet-Drafts are also available by anonymous FTP at:
> > ftp://ftp.ietf.org/internet-drafts/
> >
> > Below is the data which will enable a MIME compliant mail reader
> > implementation to automatically retrieve the ASCII version of the
> > Internet-Draft.
> >
> 
> 
> 
> > _______________________________________________
> > I-D-Announce mailing list
> > I-D-Announce@ietf.org
> > https://www.ietf.org/mailman/listinfo/i-d-announce
> > Internet-Draft directories: http://www.ietf.org/shadow.html
> > or ftp://ftp.ietf.org/ietf/1shadow-sites.txt
> >
> 
> begin 666 draft-ietf-ccamp-isis-interas-te-extension-03.txt
> M0V]N=&5N="U4>7!E.B!T97AT+W!L86EN#0I#;VYT96YT+4E$.B \,C P."TP
> <."TR-#(P,C,T.2Y)+41 :65T9BYO<F<^#0H-"@``
> `
> end

_______________________________________________
Isis-wg mailing list
Isis-wg@ietf.org
https://www.ietf.org/mailman/listinfo/isis-wg