[lisp] Genart last call review of draft-ietf-lisp-rfc6833bis-13
Pete Resnick <resnick@episteme.net> Wed, 05 September 2018 17:17 UTC
Return-Path: <resnick@episteme.net>
X-Original-To: lisp@ietf.org
Delivered-To: lisp@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 32C1A130E3A; Wed, 5 Sep 2018 10:17:32 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Pete Resnick <resnick@episteme.net>
To: gen-art@ietf.org
Cc: draft-ietf-lisp-rfc6833bis.all@ietf.org, ietf@ietf.org, lisp@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.83.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <153616785215.19847.9068125271782801845@ietfa.amsl.com>
Date: Wed, 05 Sep 2018 10:17:32 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/lisp/VwgIZHqzO0Nu0hQGQEHDqncmJWU>
Subject: [lisp] Genart last call review of draft-ietf-lisp-rfc6833bis-13
X-BeenThere: lisp@ietf.org
X-Mailman-Version: 2.1.29
List-Id: List for the discussion of the Locator/ID Separation Protocol <lisp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lisp>, <mailto:lisp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lisp/>
List-Post: <mailto:lisp@ietf.org>
List-Help: <mailto:lisp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lisp>, <mailto:lisp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 05 Sep 2018 17:17:33 -0000
Reviewer: Pete Resnick Review result: Ready with Nits 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-lisp-rfc6833bis-13 Reviewer: Pete Resnick Review Date: 2018-09-05 IETF LC End Date: 2018-08-31 IESG Telechat date: 2018-09-13 Summary: Ready with Nits By no means my area of expertise, but particularly comparing this document to 6833, it's clear what changed and the new material looks reasonable. One overall nitty thing below. Major issues: None. Minor issues: None. Nits/editorial comments: Somebody went a bit "2119-mad" in this document. In particular, *most* of the MAYs are just goofy and wrong, and many of the SHOULDs shouldn't be there. When you're putting in a 2119 keyword, they should point out a place where an implementer needs to look to make sure they get their implementation correct. A lot of these aren't helpful in that regard. A few examples: In 8.2: In addition to the set of EID-Prefixes defined for each ETR that MAY register, That's not a protocol option being described. (such as those indicating whether the message is authoritative and how returned Locators SHOULD be treated) That's not a piece of implementation advice. In 8.3: This MAY occur if a Map Request is received for a configured aggregate EID-Prefix for which no more- specific EID-Prefix exists; If "MAY" can be replaced with "might or might not", you probably want "may" or "can". Unless also acting as a Map-Resolver, a Map-Server SHOULD never receive Map-Replies; That's a statement of fact, not an implementation instruction. Please go through and get rid of the bogus ones. If it's not an indication of an implementation option (or lack thereof), it shouldn't be 2119ed.
- [lisp] Genart last call review of draft-ietf-lisp… Pete Resnick
- Re: [lisp] Genart last call review of draft-ietf-… Dino Farinacci
- Re: [lisp] Genart last call review of draft-ietf-… Dino Farinacci