Hi John Thank you for your careful review and valuable comments. We have addressed them in version 10 of the draft.
Pls also see inline. Thanks Parag From: BESS <bess-boun...@ietf.org> on behalf of John Scudder via Datatracker <nore...@ietf.org> Date: Tuesday, April 25, 2023 at 8:39 PM To: The IESG <i...@ietf.org> Cc: draft-ietf-bess-evpn-lsp-p...@ietf.org <draft-ietf-bess-evpn-lsp-p...@ietf.org>, bess-cha...@ietf.org <bess-cha...@ietf.org>, bess@ietf.org <bess@ietf.org>, Matthew Bocci <matthew.bo...@nokia.com>, matthew.bo...@nokia.com <matthew.bo...@nokia.com> Subject: [bess] John Scudder's Discuss on draft-ietf-bess-evpn-lsp-ping-09: (with DISCUSS and COMMENT) John Scudder has entered the following ballot position for draft-ietf-bess-evpn-lsp-ping-09: Discuss 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-bess-evpn-lsp-ping/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- ### Section 4.4, IP Prefix sub-TLV This might end up being implicit in Lars's DISCUSS points and/or Éric's question, but how does one even know whether the IP Prefix is 4 or 16 octets? Looking at RFC 9136, it's careful to tell us how to infer the address family of the RT-5: "The Length field of the BGP EVPN NLRI for an EVPN IP Prefix route MUST be either 34 (if IPv4 addresses are carried) or 58 (if IPv6 addresses are carried)." If a similar length-based technique is to be used here, I think this spec has to spell it out. Paragj>Added clarification text as follows: The EVPN IP Prefix Sub-TLV has the format as shown in Figure 4. The total length of this sub-TLV can be either 32 if IPv4 addresses are carried or 56 if IPv6 addresses are carried. The IP prefix and gateway IP address must be from the same IP address family. ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- I support Jim, Lars, and Éric's DISCUSSes. I agree with Jim and Éric's comments to the effect that the copy editing of the document is what should be expected for a "finished" document. There are various free spelling and grammar checkers which wouldn't fix all the problems, but would catch many of them. paragj> done. We have also addressed Jim, Lars and Eric’s comments in the new version. Thanks, Parag _______________________________________________ BESS mailing list BESS@ietf.org https://www.ietf.org/mailman/listinfo/bess
_______________________________________________ BESS mailing list BESS@ietf.org https://www.ietf.org/mailman/listinfo/bess
