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.
- Review of draft-ietf-trill-over-ip-08 Ines Robles
- Re: Review of draft-ietf-trill-over-ip-08 Donald Eastlake
- Re: Review of draft-ietf-trill-over-ip-08 Ines Robles
- Re: Review of draft-ietf-trill-over-ip-08 Joe Touch
- Re: Review of draft-ietf-trill-over-ip-08 Donald Eastlake
- Re: Review of draft-ietf-trill-over-ip-08 Joe Touch
- Re: Review of draft-ietf-trill-over-ip-08 Joe Touch