Skip to main content

Last Call Review of draft-ietf-ospf-ospfv2-hbit-10
review-ietf-ospf-ospfv2-hbit-10-genart-lc-sethi-2019-10-31-00

Request Review of draft-ietf-ospf-ospfv2-hbit
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2019-11-07
Requested 2019-10-24
Authors Keyur Patel , Padma Pillay-Esnault , Manish Bhardwaj , Serpil Bayraktar
I-D last updated 2019-10-31
Completed reviews Rtgdir Last Call review of -09 by He Jia (diff)
Genart Last Call review of -10 by Mohit Sethi (diff)
Tsvart Last Call review of -10 by Kyle Rose (diff)
Opsdir Last Call review of -10 by Tim Chown (diff)
Assignment Reviewer Mohit Sethi
State Completed
Request Last Call review on draft-ietf-ospf-ospfv2-hbit by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/Q5qavevOtCUj9lQ12HGmGTd2bKE
Reviewed revision 10 (document currently at 12)
Result Ready
Completed 2019-10-31
review-ietf-ospf-ospfv2-hbit-10-genart-lc-sethi-2019-10-31-00
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-ospf-ospfv2-hbit-10
Reviewer: Mohit Sethi
Review Date: 2019-10-31
IETF LC End Date: 2019-11-07
IESG Telechat date: Not scheduled for a telechat

Summary:
This document uses a bit in the link state advertisement (LSA) sent from
routers to indicate that they are hosts which will not forward transit traffic.
The document is READY for publication.

Major issues:

Minor issues:
I think the document would benefit from some more discussion on what happens if
a router that is repelling traffic is on the only path to some destinations?
How is this handled? Is it fair to say that H-bit is only a best effort way of
repelling traffic and does not guarantee that the transit traffic is actually
interrupted?

Any reason that this is only done for OSPFv2 and not v3? Are there ways of
achieving this functionality (of repelling transit traffic) already in v3?

Nits/editorial comments:
- Please expand acronyms like NSSA and LSAs on first usage.
- Abstract has stray " symbol.
-  The list in the acknowledgements section could benefit from an Oxford comma:
Abhay Roy, David Ward, Burjiz Pithawala, and Michael Barnes for their comments.