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 rev. no specific revision (document currently at 10)
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
Draft 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
Tsvart Last Call review of -10 by Kyle Rose
Opsdir Last Call review of -10 by Tim Chown
Assignment Reviewer Mohit Sethi
State Completed
Review review-ietf-ospf-ospfv2-hbit-10-genart-lc-sethi-2019-10-31
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/Q5qavevOtCUj9lQ12HGmGTd2bKE
Reviewed rev. 10
Review result Ready
Review completed: 2019-10-31

Review
review-ietf-ospf-ospfv2-hbit-10-genart-lc-sethi-2019-10-31

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.