Last Call Review of draft-ietf-pim-light-06
review-ietf-pim-light-06-rtgdir-lc-rogge-2024-08-27-00
Request | Review of | draft-ietf-pim-light |
---|---|---|
Requested revision | No specific revision (document currently at 09) | |
Type | Last Call Review | |
Team | Routing Area Directorate (rtgdir) | |
Deadline | 2024-08-12 | |
Requested | 2024-07-30 | |
Requested by | Gunter Van de Velde | |
Authors | Hooman Bidgoli , Stig Venaas , Mankamana Prasad Mishra , Zhaohui (Jeffrey) Zhang , Mike McBride | |
I-D last updated | 2024-08-27 | |
Completed reviews |
Rtgdir Last Call review of -06
by Henning Rogge
(diff)
Genart Last Call review of -08 by Mallory Knodel (diff) Secdir Last Call review of -08 by Chris M. Lonvick (diff) Opsdir Last Call review of -08 by Susan Hares (diff) Tsvart Last Call review of -08 by Michael Tüxen (diff) |
|
Comments |
This document is being prepared for IETF Last Call. No external reviews have been requested to help AD review the document |
|
Assignment | Reviewer | Henning Rogge |
State | Completed | |
Request | Last Call review on draft-ietf-pim-light by Routing Area Directorate Assigned | |
Posted at | https://mailarchive.ietf.org/arch/msg/rtg-dir/qYwl-5t4XPJy2cqwW3fENVy_BI4 | |
Reviewed revision | 06 (document currently at 09) | |
Result | Has nits | |
Completed | 2024-08-27 |
review-ietf-pim-light-06-rtgdir-lc-rogge-2024-08-27-00
Hello, I was asked to do a RTG review on the pimp-light-draft, this review is based on revision 06 of the draft. General comment: Multicast in general can be quite complex to get right, so having simplified protocol options (like this draft) for some situations is a good idea. Content of this draft reads good, its just the graphics (and their explanation) could use a bit more polish. Section 3.2.2: I think the graphics in 3.2.2 is a little bit too compact with too many things going on. I assume that the domains are meant to be vertical "slices" and the PIM Domains are the groups A-B-E and the group D-E-F, the BIER domain would contain B-C-D-E-F? I think this part needs a bit more text that just states which group is meant by each "annotation". Its difficult to see which "line" in this graphics is a connection between nodes and which is meant to mark the domain areas. Maybe it would help to explicitly state the domain membership when the domain are first mentioned in this section? Section 3.4: I think the description of the graphics here in the text is better than in 3.2.2, but this graphics also feels a bit "cramped". Maybe UBER and DBER could be just described in the text instead of putting it into the graphics? Something like "In another example, if PLI is configured automatically, as an example in BIER case, when the downstream BIER Edge Router (DBER) node D is no longer reachable, the upstream BIER Edge Router (UBER) node B..." Henning Rogge