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. > > >
- [Last-Call] Genart last call review of draft-ietf… Christer Holmberg via Datatracker
- Re: [Last-Call] Genart last call review of draft-… Luigi Iannone
- Re: [Last-Call] Genart last call review of draft-… Alberto Rodriguez-Natal (natal)
- Re: [Last-Call] Genart last call review of draft-… Christer Holmberg
- Re: [Last-Call] Genart last call review of draft-… Alvaro Retana
- Re: [Last-Call] [Gen-art] Genart last call review… Lars Eggert