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

[Shane Amante <[e-mail]>]

Hi Tony,

On Sep 16, 2010, at 10:04 MDT, Tony Li wrote:
> 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.

Sounds good.


[--snip--]

>> 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.

By way of precedent I could cite a small handful of (random) IS-IS RFC's with IANA Considerations:
http://tools.ietf.org/html/rfc5305#section-5
http://tools.ietf.org/html/rfc5310#section-6
http://tools.ietf.org/html/rfc5304#section-4
http://tools.ietf.org/html/rfc5308
... thus, I'd recommend it stays in.  However, I'll defer to the co-chairs judgement if you're still unsure/uncomfortable.
 


>> 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.

My turn to blush.  Looks good.  :-)

-shane