[Gen-art] Genart last call review of draft-ietf-trill-multilevel-single-nickname-09
Dan Romascanu via Datatracker <noreply@ietf.org> Wed, 20 May 2020 08:36 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 998953A11FC; Wed, 20 May 2020 01:36:52 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Dan Romascanu via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: draft-ietf-trill-multilevel-single-nickname.all@ietf.org, last-call@ietf.org, dromasca@gmail.com
X-Test-IDTracker: no
X-IETF-IDTracker: 6.130.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <158996381247.12248.849811068182849409@ietfa.amsl.com>
Reply-To: Dan Romascanu <dromasca@gmail.com>
Date: Wed, 20 May 2020 01:36:52 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/n5fBf9ryPlxrBdH02gJIe-LFFnU>
Subject: [Gen-art] Genart last call review of draft-ietf-trill-multilevel-single-nickname-09
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 20 May 2020 08:36:53 -0000
Reviewer: Dan Romascanu Review result: Ready with Issues 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-multilevel-single-nickname-09 Reviewer: Dan Romascanu Review Date: 2020-05-20 IETF LC End Date: 2020-06-11 IESG Telechat date: Not scheduled for a telechat Summary: This is not easy reading for non-TRILL experts, but I can guess that this document is useful and clear enough for the people working in the field. The document is READY but there are a few issues that seem to be in need of being clarified and addressed if necessary. Major issues: 1. Something is not clear to me about the Intended Status of this document. It is supposed to solve a problem introduced or left unclear by RFC 8243, but that document is Informational. Why is then the Intended Status Standards Track? Minor issues: 1. All the examples in the text are static. Changes however happen in the configuration of the network. What happens when an Rbridge is added at the border of an L1 area? 2. Section 6 'RB1 SHOULD use a single area nickname for all these areas.' - why is this only a SHOULD? It seems to me that in any exception case the scheme breaks 3. I am used with documents in the routing area to include Operational and Manageability considerations. These are missing here, with the exception of backwards compatibility which is addressed. Are any configuration operations necessary for example? If these are addressed in other documents a reference would be useful. Nits/editorial comments: 1. Need to correct grammar/syntax problems 2. Inconsistent writing Level 1 / Level-1 ; Level 2 / Level-2 3. Section 1: s/nicknames in L2 MUST be unique/nicknames in L2 areas MUST be unique/ 4. It would be useful to add Level to the terminology 5. Section 3.1: s/D's location is learned by the relevant TRILL switches already/D's location has been learned by the relevant TRILL switches already/ 6. Section 3.2: s/ESADI protocol/ESADI protocol [RFC7357]/
- Re: [Gen-art] Genart last call review of draft-ie… Dan Romascanu
- Re: [Gen-art] Genart last call review of draft-ie… Zhangmingui (Martin)
- [Gen-art] Genart last call review of draft-ietf-t… Dan Romascanu via Datatracker
- Re: [Gen-art] Genart last call review of draft-ie… Zhangmingui (Martin)