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 <ketant.i...@gmail.com> 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 < > nore...@ietf.org> 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 Lsr@ietf.org https://www.ietf.org/mailman/listinfo/lsr