[Gen-art] review of draft-ietf-trill-directory-assist-mechanisms-10.txt

Francis Dupont <Francis.Dupont@fdupont.fr> Mon, 09 January 2017 14:14 UTC

Return-Path: <Francis.Dupont@fdupont.fr>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C249D1294DB; Mon, 9 Jan 2017 06:14:15 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.101
X-Spam-Level:
X-Spam-Status: No, score=-5.101 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-3.199, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id sJiiPFdj66Oj; Mon, 9 Jan 2017 06:14:14 -0800 (PST)
Received: from givry.fdupont.fr (givry.fdupont.fr [IPv6:2001:41d0:1:6d55:211:5bff:fe98:d51e]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 63942129466; Mon, 9 Jan 2017 06:14:14 -0800 (PST)
Received: from givry.fdupont.fr (localhost [IPv6:::1]) by givry.fdupont.fr (8.14.7/8.14.7) with ESMTP id v09E5bkP060928; Mon, 9 Jan 2017 15:05:37 +0100 (CET) (envelope-from dupont@givry.fdupont.fr)
Message-Id: <201701091405.v09E5bkP060928@givry.fdupont.fr>
From: Francis Dupont <Francis.Dupont@fdupont.fr>
To: gen-art@ietf.org
Date: Mon, 09 Jan 2017 15:05:37 +0100
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/pAFSKQtcwNj5fCjySIB-qYDWQMs>
Cc: draft-ietf-trill-directory-assist-mechanisms.all@ietf.org
Subject: [Gen-art] review of draft-ietf-trill-directory-assist-mechanisms-10.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 09 Jan 2017 14:14:15 -0000

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...