Thanks! On Mon, Oct 10, 2022 at 10:29 AM Ketan Talaulikar <[email protected]> wrote:
> Hi Murray, > > This update includes the changes we discussed below: > https://datatracker.ietf.org/doc/html/draft-ietf-lsr-ospf-reverse-metric-13 > > Thanks, > Ketan > > > On Mon, Oct 10, 2022 at 10:45 PM Ketan Talaulikar <[email protected]> > wrote: > >> Hi Murray, >> >> Thanks for your review and please find below responses inline. >> >> These changes will be reflected in the upcoming update for the document. >> >> >> On Mon, Oct 10, 2022 at 9:37 PM Murray Kucherawy via Datatracker < >> [email protected]> wrote: >> >>> Murray Kucherawy has entered the following ballot position for >>> draft-ietf-lsr-ospf-reverse-metric-12: No Objection >>> >>> When responding, please keep the subject line intact and reply to all >>> email addresses included in the To and CC lines. (Feel free to cut this >>> introductory paragraph, however.) >>> >>> >>> Please refer to >>> https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ >>> for more information about how to handle DISCUSS and COMMENT positions. >>> >>> >>> The document, along with other ballot positions, can be found here: >>> https://datatracker.ietf.org/doc/draft-ietf-lsr-ospf-reverse-metric/ >>> >>> >>> >>> ---------------------------------------------------------------------- >>> COMMENT: >>> ---------------------------------------------------------------------- >>> >>> Apologies for being late to the party. Just a few things to add beyond >>> the >>> feedback my colleagues have already provided: >>> >>> The first sentence in Section 2.2 uses the phrase "toward the core" three >>> times. Seems like it could do with some common factoring. >>> >> >> KT> Updated. >> >> >>> >>> There's a SHOULD at the bottom of Section 6. Why's it only a SHOULD? >>> When >>> might an implementer legitimately decide to do something else? >>> >> >> KT> Thanks for catching that. Based on discussion with Alvaro, I changed >> the SHOULD to a qualified MUST earlier in section 6 related to the reverse >> metric but missed making a similar change for the TE reverse metric. >> >> >>> >>> In Section 9, I suggest making it explicit that you're talking about the >>> "Link >>> Local Signalling TLV Identifiers" registry in the "Open Shortest Path >>> First >>> (OSPF) Link Local Signalling (LLS) - Type/Length/Value Identifiers (TLV)" >>> registry group. >>> >> >> KT> Fixed. >> >> Thanks, >> Ketan >> >
_______________________________________________ Lsr mailing list [email protected] https://www.ietf.org/mailman/listinfo/lsr
