[RTG-DIR] RtgDir review: draft-ietf-trill-tree-selection-01

Daniele Ceccarelli <daniele.ceccarelli@ericsson.com> Wed, 13 January 2016 20:52 UTC

Return-Path: <daniele.ceccarelli@ericsson.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 88CFB1B3250; Wed, 13 Jan 2016 12:52:08 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham
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 SSf57uU1ZrTN; Wed, 13 Jan 2016 12:52:06 -0800 (PST)
Received: from sesbmg22.ericsson.net (sesbmg22.ericsson.net [193.180.251.48]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4FABE1B3261; Wed, 13 Jan 2016 12:52:05 -0800 (PST)
X-AuditID: c1b4fb30-f79a76d000000a93-95-5696b8f3c907
Received: from ESESSHC016.ericsson.se (Unknown_Domain [153.88.183.66]) by sesbmg22.ericsson.net (Symantec Mail Security) with SMTP id 3F.A4.02707.3F8B6965; Wed, 13 Jan 2016 21:52:03 +0100 (CET)
Received: from ESESSMB301.ericsson.se ([169.254.1.95]) by ESESSHC016.ericsson.se ([153.88.183.66]) with mapi id 14.03.0248.002; Wed, 13 Jan 2016 21:52:03 +0100
From: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>
To: "<rtg-ads@ietf.org> (rtg-ads@ietf.org)" <rtg-ads@ietf.org>
Thread-Topic: RtgDir review: draft-ietf-trill-tree-selection-01
Thread-Index: AdFJTcDgZOyKAXfnSTmgeQUUxrqYXA==
Date: Wed, 13 Jan 2016 20:52:02 +0000
Message-ID: <4A1562797D64E44993C5CBF38CF1BE4812B20594@ESESSMB301.ericsson.se>
Accept-Language: it-IT, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.16]
Content-Type: multipart/alternative; boundary="_000_4A1562797D64E44993C5CBF38CF1BE4812B20594ESESSMB301erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprIIsWRmVeSWpSXmKPExsUyM2K7k+7nHdPCDM4eVbC4dvUzs8XJOT+Y LRasecpu8X7ydjYHFo8lS34yeXy5/JktgCmKyyYlNSezLLVI3y6BK+Pfv+VMBV2PGSuamycz NTD23GXsYuTgkBAwkfi306mLkRPIFJO4cG89WxcjF4eQwGFGiVnnpzNCOIsZJTaeusQO0sAm YCXx5JAPiCkiYCcx66IuSAmzwF5GiUPHJzKBDBIGKjm1bDqYLSJgL7H1WC8LhK0nsfbJXyaQ XhYBVYmXR+pBwrwCvhKX3k4AK2EUkJWYsHsRI4jNLCAucevJfCaI2wQkluw5zwxhi0q8fPyP FcJWlNh5tp0Zoj5f4tz+x4wQMwUlTs58wjKBUXgWklGzkJTNQlI2C+giZgFNifW79CFKFCWm dD9kh7A1JFrnzGVHFl/AyL6KUbQ4tTgpN93ISC+1KDO5uDg/Ty8vtWQTIzCiDm75bbCD8eVz x0OMAhyMSjy8G/ZODRNiTSwrrsw9xCjBwawkwhuwYlqYEG9KYmVValF+fFFpTmrxIUZpDhYl cd4kmcYwIYH0xJLU7NTUgtQimCwTB6dUA2NsrvCSylfKc9niHbkXpm5ReWndtOT9DgPp2GIV T62DFloNz79duek00zGGadqMs6Eh5WYXEg13XLEuPqN1s2HRv1rBgNZnPtI2m31znc28b2/d xrhO+5Lg5CliX+Lfp9xecO6tCHc9w/2z9sY5SZO/VOrEbPzSsfdw4zvm4iNMRbu+rdJ99kGJ pTgj0VCLuag4EQA5SUF/pAIAAA==
Archived-At: <http://mailarchive.ietf.org/arch/msg/rtg-dir/BKIsoMGyjEukA9NJRYvqN5J2l84>
Cc: "draft-ietf-trill-tree-selection.all@tools.ietf.org" <draft-ietf-trill-tree-selection.all@tools.ietf.org>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "trill@ietf.org" <trill@ietf.org>
Subject: [RTG-DIR] RtgDir review: draft-ietf-trill-tree-selection-01
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-dir/>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 13 Jan 2016 20:52:08 -0000

Hello,

I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see ​http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir<http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir>

Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft.

Document: draft-ietf-trill-tree-selection-01
Reviewer: Daniele Ceccarelli
Review Date: Jan 07 2015
IETF LC End Date: September 2015
Intended Status: Standard Track
Summary:

  *   This document is basically ready for publication, but has nits that should be considered prior to publication.
Comments:

  *   Overall the draft if well written and easy to understand. I found it quite difficult to go through section 1 but I guess it’s due to my lack of knowledge of the technology. A rearrangement accordingly to the minor issues IMO would improve readability of the document.
Major Issues:

  *   No major issues found
Minor Issues:

  *   Section 3.2 – A list of the 4 TLVs with their name and one sentence summarizing its use would improve readability. Same applies in the following to e.g. section 3.2.1 where the paragraph before the figure is not very clear on the usage of the TLV.
  *   Section 3.5 – Section 3.5 defines a fifth TLV, why not listing it with the other 4 ?
Nits:

  *   Section 1.1: s/ the ingress RBridge use/the ingress RBridge uses
  *   Section 2: s/campus/Campus
  *   Section 3.1 s/is/has ??
  *   Section 3.3 s/rfc7180bis/RFC7180bis
BR
Daniele