Hi Ben, Thanks for the review. I’ll fix both of these in the next revision.
Thanks, Acee > On Mar 1, 2023, at 2:14 PM, Ben Niven-Jenkins via Datatracker > <nore...@ietf.org> wrote: > > Reviewer: Ben Niven-Jenkins > Review result: Has Nits > > Hello > > I have been selected to do a routing directorate “early” review of > draft-ietf-rtgwg-vrrp-rfc5798bis-02. > > The routing directorate will, on request from the working group chair, perform > an “early” review of a draft before it is submitted for publication to the > IESG. The early review can be performed at any time during the draft’s > lifetime > as a working group document. The purpose of the early review depends on the > stage that the document has reached. > > For more information about the Routing Directorate, please see > http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir > > Document: draft-ietf-rtgwg-vrrp-rfc5798bis-02.txt > Reviewer: Ben Niven-Jenkins > Review Date: 1st March 2023 > Intended Status: Standards Track > > Summary: This document is basically ready for publication, but has nits that > should be considered prior to being submitted to the IESG. > > The document is well written and understandable. > > I have two minor comments that I think should be resolved before it is > submitted to the IESG. > > 1) Section 1.1 bullet 3 states “Appendices describing operation over legacy > technologies (FDDI, Token Ring, and ATM LAN Emulation) were removed.” however > the diagrams in Section 4.1 & 4.2 include “Ethernet, Token Ring, or FDDI”. > Should the reference to Token Ring & FDDI be also removed from those diagrams? > > 2) Section 4.2 states: “In the IPv4 example above, i.e., IPvX is IPv4 > everywhere in the figure, half of the hosts have configured a static default > route through Router-1's IPv4 A, and half are using Router-2's IPv4 B.” (and > has similar text in the next paragraph for IPv6), however the diagram shows > all > hosts with “Default Router IPvX Addresses” of IPvX A, whereas I think the > diagram should be showing two of the four hosts with “Default Router IPvX > Addresses” of IPvX A and the other two with “Default Router IPvX Addresses” of > IPvX B? > > Regards > Ben > > > _______________________________________________ rtgwg mailing list rtgwg@ietf.org https://www.ietf.org/mailman/listinfo/rtgwg
