Re: [Last-Call] Genart last call review of draft-ietf-lisp-vendor-lcaf-09

Luigi Iannone <ggx@gigix.net> Tue, 12 April 2022 07:14 UTC

Return-Path: <ggx@gigix.net>
X-Original-To: last-call@ietfa.amsl.com
Delivered-To: last-call@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6B8FE3A11CC for <last-call@ietfa.amsl.com>; Tue, 12 Apr 2022 00:14:33 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.909
X-Spam-Level:
X-Spam-Status: No, score=-1.909 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gigix-net.20210112.gappssmtp.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 9LzYGWlT_CR4 for <last-call@ietfa.amsl.com>; Tue, 12 Apr 2022 00:14:29 -0700 (PDT)
Received: from mail-wr1-x42a.google.com (mail-wr1-x42a.google.com [IPv6:2a00:1450:4864:20::42a]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A80C93A1247 for <last-call@ietf.org>; Tue, 12 Apr 2022 00:14:27 -0700 (PDT)
Received: by mail-wr1-x42a.google.com with SMTP id e8so9901471wra.7 for <last-call@ietf.org>; Tue, 12 Apr 2022 00:14:27 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gigix-net.20210112.gappssmtp.com; s=20210112; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=0uiomicc3a+X/c/GWWsVtTe/H6lQa5fwqlQRtYRu5gc=; b=hTlm9AircxWqF9Ur5evd1GrP5OzRMPchNe178rLnTCCV23gUEXZC5Z54u8BO/DBVks TUUebS0n2NBaelqsHh1+E0fdrTW2jIaimvwQ7kyn6yIGoc/y9VODCS67Mmt9L/Lrv/uM pF8fW0KpHrxolwfvUIbcelFVQchElvqxlhEXoMY/k5YRZR3vNHou3HtvIddkFl8OMUji 7GweSM04fW9MOu6NVl7sPMNswF9X5xjm6qiF79/mmmfncKQPc4PZrzW26EJ7g79D/EsF 5hLV2/1oMGk8YwT9RZuC6+6DCB+EMzFszGPgXmNOdL6qMg9hpr4S48K6e2ZZfeoWT3/Z +spQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=0uiomicc3a+X/c/GWWsVtTe/H6lQa5fwqlQRtYRu5gc=; b=b71h7N3Z/mr+cXNK9vqWSWzWzkXjZHiwSN5TiPvlxSH9WdUtW/ZYm+Y9eIW32R+zP6 oHcaQiOcKwnHXNpRCIOoyDf6B+gsTJ9DbH/e69XXUtmkrEEG03qgUUE6u1Ta1gsgUAE7 wR+ri8+FYGbvvOAy8JArrRjOmp4PTxanrf0evZn7TrG8iXzHEZE6YxT+HUv8PJBAC/Dy AqadsEcfHvfAD8OtHrvdFTytUjV3/IwCPk29Imh1HySoS9REVNl8TSO5wJgLxKERDlVX AdI0bwyS3caTMMnyvBvejRumWDQ5gVmwwLALr9SkMZXHLZLA4MTv1BQPGezvWjt30eOB Rqww==
X-Gm-Message-State: AOAM5328WRzwz6XgfnmiUsJs9sMEaDJd8+edet5oh75n/ffRWO1ji8lm rPOgD0aE4rDiqoLdIWELiGOugg==
X-Google-Smtp-Source: ABdhPJyW6wpLu1HvX+EjAKREPVCMi6yUo4kwOQD4HUteVwkF+rTPsH7B8yejia7Y063s8LTUiw13ag==
X-Received: by 2002:a5d:6dd1:0:b0:207:92c4:eaef with SMTP id d17-20020a5d6dd1000000b0020792c4eaefmr16526391wrz.498.1649747665211; Tue, 12 Apr 2022 00:14:25 -0700 (PDT)
Received: from smtpclient.apple ([2a01:e0a:1ec:470:cc47:d397:ac7e:7277]) by smtp.gmail.com with ESMTPSA id v8-20020a1cf708000000b0034d7b5f2da0sm1579162wmh.33.2022.04.12.00.14.23 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 12 Apr 2022 00:14:24 -0700 (PDT)
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3696.80.82.1.1\))
From: Luigi Iannone <ggx@gigix.net>
In-Reply-To: <164970933406.11609.7107781510824246342@ietfa.amsl.com>
Date: Tue, 12 Apr 2022 09:14:23 +0200
Cc: gen-art@ietf.org, draft-ietf-lisp-vendor-lcaf.all@ietf.org, last-call@ietf.org, lisp@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <28E66664-BA03-4A9B-9C32-0AAC06C9D4B6@gigix.net>
References: <164970933406.11609.7107781510824246342@ietfa.amsl.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>
X-Mailer: Apple Mail (2.3696.80.82.1.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/last-call/7LsOtggr0tHiOicKvic3WLcwcrU>
Subject: Re: [Last-Call] Genart last call review of draft-ietf-lisp-vendor-lcaf-09
X-BeenThere: last-call@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: IETF Last Calls <last-call.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/last-call>, <mailto:last-call-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/last-call/>
List-Post: <mailto:last-call@ietf.org>
List-Help: <mailto:last-call-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/last-call>, <mailto:last-call-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 12 Apr 2022 07:14:38 -0000

Hi Christer,

Thanks for the review.

As a shepherd I have a couple of comments inline.

> On 11 Apr 2022, at 22:35, Christer Holmberg via Datatracker <noreply@ietf.org> wrote:
> 
> Reviewer: Christer Holmberg
> 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-lisp-vendor-lcaf-09
> Reviewer: Christer Holmberg
> Review Date: 2022-04-11
> IETF LC End Date: 2022-04-12
> IESG Telechat date: Not scheduled for a telechat
> 
> Summary:
> 
> The document is well written, and easy to read and understand. However, I do
> have a couple of issues.
> 
> Major issues:
> 
> Q1:
> 
> I do wonder why the document is published as Experimental, however, due to the
> following reasons:

It is experimental because is an update to RFC 8060, which is experimental.
So unless we move that one to standard track I would say that is the right type of RFC.


> 
>   a)
> 
>   The document defines usage of the Type value 255.
> 
>   b)
> 
>   Section 3 says:
> 
>      "If a LISP device receives a LISP message containing a Vendor Specific
>       LCAF with an OUI that it does not understand, it MUST drop the
>       message and it SHOULD create a log message."
> 
>   This sounds like an update to LISP.
> 

Excellent point. Actually this document updates RFC 8060, and this should be stated in the document.


>   c)
> 
>   Section 3 defines new header fields.
> 
> Minor issues:
> 
> N/A
> 
> Nits/editorial comments:
> 
> Q2:
> 
> Section 1 says:
> 
>   “The Vendor Specific LCAF allows organizations to create
>   LCAF addresses to be used only internally on particular LISP
>   deployments.”
> 
> Is “allows” the best wording? Where organizations previously disallowed to do
> this?
> 
> Would it be more correct to say “defines how organizations can create…”?

Yes, this wording is more correct.

Ciao

L.




> 
> 
>