Review of draft-ietf-trill-over-ip-08

Ines Robles <maria.ines.robles@ericsson.com> Wed, 28 December 2016 05:23 UTC

Return-Path: <maria.ines.robles@ericsson.com>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 89691129570; Tue, 27 Dec 2016 21:23:21 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Ines Robles <maria.ines.robles@ericsson.com>
To: rtg-dir@ietf.org
Subject: Review of draft-ietf-trill-over-ip-08
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148290260152.14213.11124890517026127285.idtracker@ietfa.amsl.com>
Date: Tue, 27 Dec 2016 21:23:21 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/sR3INF3jKx-CpANWLjvboQX2SDE>
Cc: draft-ietf-trill-over-ip.all@ietf.org, ietf@ietf.org, trill@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 28 Dec 2016 05:23:21 -0000

Reviewer: Ines Robles
Review result: Has Nits

Hi,

QA review for "TRILL (Transparent Interconnection of Lots of Links)
over IP" I-D:


Document: draft-ietf-trill-over-ip-08.txt

Reviewer: Ines Robles

Review Date: December 28, 2016

Intended Status: Proposed Standard 


Summary:

 I believe the draft is technically good. I have some minor comments.


Comments:


	Major Issues:

		I have no “Major” issues with this I-D.

	Minor Issues and Nits:

		Section 1: Introduction

			- I would add a reference to [draft-ietf-6man-rfc2460bis-08] when
IPv6 is mentioned.


		Section 2: Terminology

			- In RBridge definition, I would add a reference to [RFC6325]

			- In VNI definition, I would extend VXLAN - "...In Virtual
eXtensible Local Area Network (VXLAN) [RFC7348]..."


		Section 3: Use Cases for TRILL over IP

			The text of the uses cases is clear. However, I think it would be
nice to add a graph for each use case like:
https://www.ietf.org/proceedings/82/slides/trill-13.pdf (Slide 4 and
5) 

		Section 4.5: TRILL Over IP IS-IS SubNetwork Point of Attachment

			I think it would be nice to add in the text the meaning of "111111"
depicted in the figure.

		Section 8.1: Congestion Considerations

			I would expand ECMP in the 3rd paragraph. 

		Section 9.2.3.2

			I think it would be nice to add a reference to RFC 7172 when
Inner.FGL is mentioned.

		Section 11.1: Port Assignments

			I would add the registry for this request - "Service Name and
Transport Protocol Port Number Registry"
[https://www.iana.org/assignments/service-names-port-numbers/service-names-port-numbers.xhtml]



Thank you very much,

Ines.