Subject: RE: [mpls] RE: [Isis-wg] CCAMP Working group last callondraft-ietf-ccamp-te-node-cap-02.txt

From: "LE ROUX Jean-Louis RD-CORE-LAN" <[email-masked]>
Date: Thu, 2 Nov 2006 14:22:28 +0100

Hi Lee

Sorry for this late answer.
Thanks for your comments, please see inline, 

> -----Message d'origine-----
> De : Les Ginsberg (ginsberg) [mailto:[email-masked]] 
> Envoyé : dimanche 22 octobre 2006 08:37
> À : [email-masked]; [email-masked]; [email-masked]; 
> [email-masked]
> Cc : [email-masked]
> Objet : [mpls] RE: [Isis-wg] CCAMP Working group last 
> callondraft-ietf-ccamp-te-node-cap-02.txt 
> 
> In regards to the IS-IS definitions, there is (again - sorry 
> JP...) confusion in the use of the terms TLV and subTLV.
> 
> For example, in Section 2 it states:
> 
> <snip>
> A new TLV is defined for ISIS and OSPF: the TE Node Capability 
>    Descriptor TLV, to be carried within: 
>         - The ISIS Capability TLV ([ISIS-CAP]) for ISIS...
> <end snip>
> 
> If the information is carried within another TLV, then it is 
> clearly subTLV information - the difference between the two 
> is more than terminology since it impacts both the encoding 
> and the scope of the identifiers which are assigned.
> 
> Section 4.2 then goes on to discuss:
> 
> "The format of the IS-IS TE Node Capability TLV" - which is, 
> as pointed out above, actually a subTLV.
> 
> And then goes on to define "subsubTLVs" (which are 
> incorrectly called subTLVs).
> 
> As "subsubTLVs" are new to IS-IS encoding some discussion of 
> this added level of hierarchy is required. However, I think 
> it might also be fair to entertain the possibility that the 
> "TE Node Capability subTLV" be deleted and the "subsubTLVs" 
> defined in the subsections of 4.2 be defined more directly as 
> subTLVs of the IS-IS CAPs TLV. This is possible because as 
> currently defined, the "TE Node Capability subTLV" has no 
> information at the subTLV level - it is simply a container 
> for the subsubTLV information. This would allow for a more 
> efficient encoding without any loss of functionality.

Thanks for pointing this out, we are going to delete the TE Node Cap level and define the Control Plane Cap and Data plane Cap as sub-TLVs of the ISIS Capability TLV.

> 
> Whatever the resolution, all the text relating to this must 
> be reworded to be consistent and accurate.

OK will be done

> 
> I would also prefer that the wording of the following 
> sentence in Section 5.2:
> 
> "An empty TE Node Capability Descriptor MUST be discarded."
> 
> be changed to
> 
> "An empty TE Node Capability Descriptor MUST be ignored."
> 
> The term "discard" suggests that a router which leaks a CAP 
> TLV containing TE node capability information might be 
> required to modify the contents of that TLV - which is 
> something to be avoided.

OK, the text will be updated as per your proposal.

Thanks,

JL

> 
>    Les
> 
> > -----Original Message-----
> > From: Adrian Farrel [mailto:[email-masked]]
> > Sent: Saturday, October 21, 2006 5:22 AM
> > To: [email-masked]; [email-masked]; [email-masked]
> > Cc: Brungard, Deborah A, ALABS; [email-masked]
> > Subject: [Isis-wg] CCAMP Working group last call
> ondraft-ietf-ccamp-te-
> > node-cap-02.txt
> > 
> > Heads up.
> > 
> > The CCAMP working group is holding a working group last 
> call on this 
> > draft.
> > 
> > Your input is invited.
> > 
> > Please send comments direct to the CCAMP mailing list or to the WG
> chairs
> > (Deborah and/or me) if you are not subscribed to CCAMP.
> > 
> > Thanks,
> > Adrian
> > ----- Original Message -----
> > From: "Adrian Farrel" <[email-masked]>
> > To: <[email-masked]>
> > Sent: Tuesday, October 17, 2006 1:34 AM
> > Subject: Working group last call on
> draft-ietf-ccamp-te-node-cap-02.txt
> > 
> > 
> > > This starts a two week working group last on
> > >
> http://www.ietf.org/internet-drafts/draft-ietf-ccamp-te-node-c
ap-02.txt
> > >
> > > Please send your comments to the CCAMP mailing list by 12 midnight
> UK
> > time
> > > on Monday 30th October
> > >
> > > Thanks,
> > > Adrian and Deborah
> > 
> > 
> > 
> > _______________________________________________
> > Isis-wg mailing list
> > [email-masked]
> > https://www1.ietf.org/mailman/listinfo/isis-wg
> 
> _______________________________________________
> mpls mailing list
> [email-masked]
> https://www1.ietf.org/mailman/listinfo/mpls
> 

_______________________________________________
Isis-wg mailing list
[email-masked]
https://www1.ietf.org/mailman/listinfo/isis-wg