Last Call Review of draft-ietf-bess-evpn-overlay-10
review-ietf-bess-evpn-overlay-10-genart-lc-gurbani-2017-12-21-01

Request Review of draft-ietf-bess-evpn-overlay
Requested rev. no specific revision (document currently at 12)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2017-12-22
Requested 2017-12-08
Authors Ali Sajassi, John Drake, Nabil Bitar, Ravi Shekhar, Jim Uttaro, Wim Henderickx
Draft last updated 2017-12-21
Completed reviews Rtgdir Early review of -04 by Keyur Patel (diff)
Rtgdir Early review of -04 by IJsbrand Wijnands (diff)
Rtgdir Telechat review of -10 by Lou Berger (diff)
Opsdir Last Call review of -10 by Fred Baker (diff)
Genart Last Call review of -10 by Vijay Gurbani (diff)
Assignment Reviewer Vijay Gurbani
State Completed
Review review-ietf-bess-evpn-overlay-10-genart-lc-gurbani-2017-12-21
Reviewed rev. 10 (document currently at 12)
Review result Ready with Issues
Review completed: 2017-12-21

Review
review-ietf-bess-evpn-overlay-10-genart-lc-gurbani-2017-12-21

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-bess-evpn-overlay-??
Reviewer: Vijay Gurbani
Review Date: 2017-12-21
IETF LC End Date: 2017-12-22
IESG Telechat date: 2018-01-11

Summary: The draft is ready as a PS.  It has one Minor issue that needs some discussion.

Major issues: None.

Minor issues: One.

Nits/editorial comments:  None.

Minor issues:
In S13 (IANA Considerations), I am not sure whether this draft is allocating the tunnel types or not.  The responsible IANA registry [0] shows that the tunnel types mentioned in this draft are already allocated through a precursor individual draft [1], which has long expired.  However, when I look at the IANA Consideration section in [1], it is empty!  So I am not sure how the IANA registry in [0] received the allocations.  I think the IANA section of this draft needs to be updated accordingly.

[0] https://www.iana.org/assignments/bgp-parameters/bgp-parameters.xhtml#tunnel-types
[1] http://www.iana.org/go/draft-sd-l2vpn-evpn-overlay (the expired individual draft itself is at https://tools.ietf.org/html/draft-sd-l2vpn-evpn-overlay-03)