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

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