Re: [OSPF] Poll for WG adoption of draft-hegde-ospf-node-admin-tag

Dhruv,

Thanks for detailed review and comments.
Pls see in-line for the response.

Rgds
Shraddha

-----Original Message-----
From: OSPF [mailto:ospf-bounces@ietf.org] On Behalf Of Dhruv Dhody
Sent: Saturday, August 30, 2014 6:22 PM
To: Acee Lindem (acee)
Cc: ospf@ietf.org
Subject: Re: [OSPF] Poll for WG adoption of draft-hegde-ospf-node-admin-tag

Hi,

I have read the document and I support it for WG adoption.

I have following comments, that can be handled later

(1) Section 4.1
OLD:

   The format of the TLVs within the body of an RI LSA is the same as
   the format used by the Traffic Engineering Extensions to OSPF
   [RFC3630].

   The LSA payload consists of one or more nested Type/Length/Value
   (TLV) triplets.  The format of each TLV is:

NEW:

   As per [RFC4970], the format of the TLVs within the body of an RI LSA is the same as
   the format used by the Traffic Engineering Extensions to OSPF
   [RFC3630].

   The RI LSA payload consists of one or more nested Type/Length/Value
   (TLV) triplets.  The format of the per-node administrative tag TLV is:

END

<Shraddha> Accept. Will be updated in the next revision.

Also, it should be stated
- if are more than one instance of this TLV in RI LSA are allowed.

<Shraddha>More than one instance of the TLV can be added in same RI-LSA or in a multiple instance as defined
                       In  draft-acee-ospf-rfc4970bis-00.txt

- Minimum one tag must be present in the TLV
<Shraddha> Accept.

- What happens if the implementation does not know the Interpretation of the tag value
<Shraddha> This is mentioned in section 4.2, However will add explicit mention regarding the scenario you mentioned.

(2) It should be explicitly stated that - No IANA registry is required to store the meaning or interpretation of.the tag values.

<Shraddha> It's mentioned in the section 4.2 that no well known  tag values will be defined by this document.

(3) Backward compatibility - few lines may be added to state that as per [RFC4970], unknown TLV would be silently ignored.
<Shraddha> Accept

Nits
- Avoid using reference in abstract
- Expand LFA on first use
- Administrative Tag TLV or 'per-node Administrative Tag' : consistent naming through the document would be nice

<Shraddha> Accept.

Regards,
Dhruv

On Tue, Aug 26, 2014 at 2:48 AM, Acee Lindem (acee) <acee@cisco.com> wrote:
> There are situations where node level policy is required and an OSPF 
> advertised admin tag simplifies this. For example, advertisement of 
> remote-LFA eligibility.
>
> Please indicate your support or objections to adopting this draft as 
> an OSPF WG document.
>
> Thanks,
> Acee
>
> _______________________________________________
> OSPF mailing list
> OSPF@ietf.org
> https://www.ietf.org/mailman/listinfo/ospf
>

_______________________________________________
OSPF mailing list
OSPF@ietf.org
https://www.ietf.org/mailman/listinfo/ospf