Skip to main content

Last Call Review of draft-ietf-pim-drlb-13
review-ietf-pim-drlb-13-opsdir-lc-clarke-2019-10-30-00

Request Review of draft-ietf-pim-drlb
Requested revision No specific revision (document currently at 15)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2019-11-07
Requested 2019-10-24
Authors Yiqun Cai , Heidi Ou , Sri Vallepalli , Mankamana Prasad Mishra , Stig Venaas , Andy Green
I-D last updated 2019-10-30
Completed reviews Rtgdir Last Call review of -13 by Ben Niven-Jenkins (diff)
Genart Last Call review of -13 by Pete Resnick (diff)
Secdir Last Call review of -13 by Carl Wallace (diff)
Tsvart Last Call review of -13 by Michael Scharf (diff)
Opsdir Last Call review of -13 by Joe Clarke (diff)
Assignment Reviewer Joe Clarke
State Completed
Request Last Call review on draft-ietf-pim-drlb by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/MG70V3KiwkgFvQWBUHOJKbPRVHk
Reviewed revision 13 (document currently at 15)
Result Ready
Completed 2019-10-30
review-ietf-pim-drlb-13-opsdir-lc-clarke-2019-10-30-00
I was asked to review this document on behalf of the ops directorate.  This
document describes a new protocol to do PIM DR load balancing.  In general, I
think this document is ready.  I appreciate both the backwards compat and
operator considerations sections.  In fact, as I read through this, I kept
thinking, "I hope they talk about legacy vs. new routers on the same shared
LAN".  One thing that might be good to add is a callout to vendors/implementors
that they are explicit in which GDR for which group/source.  Thinking with a
troubleshooting mind, this changes the paradigm in forwarding and knowing how
that behaves will be critical.

I also found three really small nits:

Section 5.2.1:

s/ordinal number of router X/ordinal number of Router X/

===

Section 5.4:

s/permissable/permissible/

===

Section 5.8:

s/take part in an load-balancing/take part in load-balancing/