Last Call Review of draft-ietf-trill-directory-assist-mechanisms-10
review-ietf-trill-directory-assist-mechanisms-10-opsdir-lc-zhou-2017-01-10-00

Request Review of draft-ietf-trill-directory-assist-mechanisms
Requested rev. no specific revision (document currently at 12)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2017-01-09
Requested 2016-12-23
Draft last updated 2017-01-10
Completed reviews Rtgdir Early review of -03 by Matthew Bocci (diff)
Rtgdir Early review of -03 by Joel Halpern (diff)
Secdir Last Call review of -11 by Daniel Franke (diff)
Genart Last Call review of -10 by Francis Dupont (diff)
Opsdir Last Call review of -10 by Tianran Zhou (diff)
Assignment Reviewer Tianran Zhou
State Completed
Review review-ietf-trill-directory-assist-mechanisms-10-opsdir-lc-zhou-2017-01-10
Reviewed rev. 10 (document currently at 12)
Review result Ready
Review completed: 2017-01-10

Review
review-ietf-trill-directory-assist-mechanisms-10-opsdir-lc-zhou-2017-01-10

Hi Authors,

I have reviewed this document as part of the Operational directorate's ongoing effort to review all IETF documents being processed by the IESG.  These comments were written with the intent of improving the operational aspects of the IETF drafts. Comments that are not addressed in last call may be included in AD reviews during the IESG review.  Document editors and WG chairs should treat these comments just like any other last call comments.

Document reviewed: draft-ietf-trill-directory-assist-mechanisms-10

Summary:
- This document describes mechanisms for providing directory service to TRILL edge switches, including Pull, Push and Hybrid model.
- I do not find any major issue from the OPS point of view, I think this doc is ready for publication as RFC.

The clarification question:
In 3.5.1, you have "A Pull Directory can be hosted on an end station", and "This capability would be useful in supporting an end station that performs directory assisted encapsulation [DirAsstEncap] or that is a "smart end node" [SmartEN]."

But in Push Model, I did not see the similar option, i.e., the push directory hosted on the end station.
In section 2, you have "one or more Push Directory servers reside at TRILL switches".
Are you indicating that the push directory server can only reside on the trill switch? If so could you please explicitly write down it and give the reason, because it's not obvious to me. 

One editorial:
in 3.5.2 Pull Directory Use by End Stations, s/Use/Used/


Regards,
Tianran