Re: [Isis-wg] WG LC draft-ietf-isis-purge-tlv-03.txt

[Tony Li <[e-mail]>]

Hi Shane,

Thanks for your comments.

> 2)  In Section 4, it is mentioned that the Dynamic Hostname TLV can be included in purges.  When an IS is inserting two System ID's into the Purge TLV (first, of the IS that inserted the Purge TLV and, second, of the IS that the purge was received from), presumably the Dynamic Hostname TLV would only include the former (the IS that inserted the Purge TLV), correct?  Or, is it permitted for this IS that's inserting two System ID's to insert two Dynamic Hostname TLV's, (assuming the Dynamic Hostname of the IS that purge was first recv'd from was available)?


The intent was that it was only the local hostname that would be inserted.  I'll add some clarifying text.


> 3)  In Section 5, there appears to be a placeholder in there that says "NEW TEXT:".  Should those two words be stricken?


<blush>  Whoops.  Yes, text processing error on my part.  </blush>


> 4)  In Section 6, why is there an RFC Editor Note to remove this section?  Shouldn't this section stay "as is"?


My understanding is that IANA requests are supposed to be removed.  If I'm out of sync here, I'm happy for it to stay.


> Re: draft-li-reg-purge-01
> 2)  draft-ietf-isis-purge-tlv states that it's acceptable for the Dynamic Hostname TLV to be included in Purges.  Thus, shouldn't draft-li-reg-purge specify, in IANA Considerations, that the Purge Column should be augmented to also include TLV 137, "Dynamic Name"?  Or, perhaps it would be cleaner if in the IANA Considerations of draft-ietf-isis-purge-tlv stated that TLV 137 also have the Purge column say "y", since Dynamic Hostname TLV is mentioned in this draft and isn't talked about at all in draft-li-reg-purge?


?

Section 4, para 2:
   The 'Purge' column should initially contain a 'y' for TLV type 10
   (Authentication) and for TLV type 137 (Dynamic hostname).  All other
   entries in this column should have an 'n'.  Other additions to this
   registry should explicitly specify their value for this column.

Tony