Dhruv, Thanks for detailed review and comments. Pls see in-line for the response.
Rgds Shraddha -----Original Message----- From: OSPF [mailto:ospf-boun...@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) <a...@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 _______________________________________________ OSPF mailing list OSPF@ietf.org https://www.ietf.org/mailman/listinfo/ospf
