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
