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/