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

Ines Robles <> Wed, 28 December 2016 05:23 UTC

Return-Path: <>
Received: from (localhost [IPv6:::1]) by (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 <>
To: <>
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: <>
Date: Tue, 27 Dec 2016 21:23:21 -0800
Archived-At: <>
X-Mailman-Version: 2.1.17
List-Id: IETF-Discussion <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Wed, 28 Dec 2016 05:23:21 -0000

Reviewer: Ines Robles
Review result: Has Nits


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 


 I believe the draft is technically good. I have some minor 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: (Slide 4 and

		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. 


			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"

Thank you very much,