Skip to main content

Last Call Review of draft-ietf-trill-directory-assist-mechanisms-10
review-ietf-trill-directory-assist-mechanisms-10-genart-lc-dupont-2017-01-09-00

Request Review of draft-ietf-trill-directory-assist-mechanisms
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2017-01-02
Requested 2016-12-19
Authors Donald E. Eastlake 3rd , Linda Dunbar , Radia Perlman , Yizhou Li
I-D last updated 2017-01-09
Completed reviews Rtgdir Early review of -03 by Matthew Bocci (diff)
Rtgdir Early review of -03 by Joel M. Halpern (diff)
Secdir Last Call review of -11 by Daniel Fox Franke (diff)
Genart Last Call review of -10 by Francis Dupont (diff)
Opsdir Last Call review of -10 by Tianran Zhou (diff)
Assignment Reviewer Francis Dupont
State Completed
Request Last Call review on draft-ietf-trill-directory-assist-mechanisms by General Area Review Team (Gen-ART) Assigned
Reviewed revision 10 (document currently at 12)
Result Ready
Completed 2017-01-09
review-ietf-trill-directory-assist-mechanisms-10-genart-lc-dupont-2017-01-09-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-trill-directory-assist-mechanisms-10.txt
Reviewer: Francis Dupont
Review Date: 20170105
IETF LC End Date: 2017019
IESG Telechat date: unknown

Summary: Ready

Major issues: None

Minor issues: None

Nits/editorial comments: 
 - 1.1 page 5: nomrally -> normally

 - 1.2 page 6: MscSA -> MacSA

 - 2.3.2 page 11: cancelled -> canceled

 - 3.1 page 19: acknowledgement -> acknowledgment

 - 3.2.1 page 21: AFN is from an unknown space: please add
  a reference to where AFN is defined or to its registry (or both).

 - 3.2.2.2 page 26 MacDA: in "this MAC address must be unicast"
  as it is a requirement consider to put a MUST or to change to "has to"

 - 3.3 page 27: in "A Pull Directory server may have a limit" use
  a MAY?

 - 3.3 page 28: you use the F, P, N, etc bits when their meanings are
  in 3.3.1 so:
  * please add a forward reference to 3.3.1
  * add their names (flood, positive, negative) at first use

 - 3.3.1 page 30: in "Message must have either" must -> MUST/has to

 - 3.3.2 page 31: appropiately -> appropriately

 - 3.5.1 page 33: knowns -> knows

 - 3.5.3 page 34: Chanel -> Channel

 - 3.6 page 35 (twice): [Aa]cknowledgement -> [Aa]cknowledgment

 - 3.6.3 page 37: QTYPE 3 and 4 are unassigned

 - 3.7 page 37: chagnes -> changes

 - 7.1 page 45: only use of "IANA will assign" vs
  "IANA is requested to assign". BTW look for what is the best and
  use only  this one.

 - 7.1 page 45: ESDADI -> ESADI and ESDAI -> ESADI

 - 7.1 page 45: in the figure break the line between "for" and "expansion".
  BTW it doesn't matter if a varaible field spreads over more than one
 line...

 - Authors' Addresses page 52: (perhaps a troff macro issue):
  please insert a comma befor the country name.

 - Authors' Addresses page 52: China -> PR China (or any variant
  including CN (ISO IS 3166 2 letter code))

Regards

Francis.Dupont@fdupont.fr

PS: I know some of you are from a country (from a list of 2) not using
international units but millisecond abbrev is ms, not millisec...