Re: [GROW] RtgDir review: draft-ietf-grow-bgp-session-culling-04.txt

Matt Griswold <grizz@20c.com> Thu, 21 September 2017 00:14 UTC

Return-Path: <grizz@20c.com>
X-Original-To: grow@ietfa.amsl.com
Delivered-To: grow@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A351112ECEC for <grow@ietfa.amsl.com>; Wed, 20 Sep 2017 17:14:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.6
X-Spam-Level:
X-Spam-Status: No, score=-2.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_LOW=-0.7] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=20c-com.20150623.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 dg1u6OyE7KTU for <grow@ietfa.amsl.com>; Wed, 20 Sep 2017 17:14:14 -0700 (PDT)
Received: from mail-it0-x234.google.com (mail-it0-x234.google.com [IPv6:2607:f8b0:4001:c0b::234]) (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 2864413219C for <grow@ietf.org>; Wed, 20 Sep 2017 17:14:12 -0700 (PDT)
Received: by mail-it0-x234.google.com with SMTP id o200so4795259itg.0 for <grow@ietf.org>; Wed, 20 Sep 2017 17:14:12 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=20c-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=R8Fx27EoN4Lo5w3kT6y7I7BcSnxAAXP7/+R/wgxZFbY=; b=uLQD8YOXt+/ro7vk6yZATns9M4r61phRmRXsYTncx0Ij571Mnk+jy2xdnfD5c5d6tw z1Zg+tXjzu6ZMnVhxd0zoEURXtgKwx9IBF3DJM+rByWSjoWhIA+roId33j2SxFYQATyr r1zq3aAfhAUSNphDayo+hoLj44bGO8yzqf3WRcGRLSCR292cBP1nEEGHdY8Wmfsw/W/g GWAaUiX8cEsGeIVGY910n+rKe6SFmXm9LL9GQaJrPrKQmDgtoNrb2Z3cmKtR4N9eEybI ZYWv7WbsdhYOj/XWuulC7gABg2MgubeILzrIWvzJD8Y/iCcSif1UsSHczBTg/aveTw/C Lb/w==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=R8Fx27EoN4Lo5w3kT6y7I7BcSnxAAXP7/+R/wgxZFbY=; b=GEWcRfMNUAPYTSUeDtkQqWZHsMRD+2gPxZ4MYskxuUX8J9MtMpWjam4QAjkUJD2zTy bZzV1DOsZEJiNEyz6+Le1Ru3LgoyvX2LE1EiOIsCWipXuYJ6RTwbXLQxz0tAsSUPGjDI qC7ocd19ntj179G9jC5cRE9SvoE8j5dOOpFjOm6hycX/GpAmn4MbKcL1a9lTrE7b49zY YOahSZuyL/PvWrQyTvC9ts177looRcDW/vpOFgJYXqTlnK/kp3jrN2aues3oATnUMwTA +oA6O3j7XhnbAVndq0HEJXpy/h5IYzNC2+qYuvO/6aBUeNTceJDJ+MmLpvJb39uaHiCG /o8g==
X-Gm-Message-State: AHPjjUjyQ0Ek46pcMr/sqleDODZgFhOi9ytz7mS3sJOz9CO/MG5MFme/ FJ1vykZuSFOQLI2z0NpqjrWNiA==
X-Google-Smtp-Source: AOwi7QA3yuzeOwaCauaqSfHaGxOHgJHFOzQVHr/JNaNjTWESTim436OKqaL9nGdBbKEl4LTIDgb8mA==
X-Received: by 10.36.181.79 with SMTP id j15mr6081043iti.78.1505952851548; Wed, 20 Sep 2017 17:14:11 -0700 (PDT)
Received: from x0r (97-83-204-249.dhcp.eucl.wi.charter.com. [97.83.204.249]) by smtp.gmail.com with ESMTPSA id d3sm87445iog.0.2017.09.20.17.14.09 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 20 Sep 2017 17:14:10 -0700 (PDT)
Date: Thu, 21 Sep 2017 00:14:09 +0000
From: Matt Griswold <grizz@20c.com>
To: Stig Venaas <stig@venaas.com>
Cc: "<rtg-ads@ietf.org>" <rtg-ads@ietf.org>, rtg-dir@ietf.org, grow@ietf.org, draft-ietf-grow-bgp-session-culling.all@ietf.org
Message-ID: <20170921001409.636adbf3@x0r>
In-Reply-To: <CAHANBtLChmeGbSq_SpwZFK7phjLLdzQfkbVChO9apCh4_pSBVw@mail.gmail.com>
References: <CAHANBtLChmeGbSq_SpwZFK7phjLLdzQfkbVChO9apCh4_pSBVw@mail.gmail.com>
X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.28; x86_64-pc-linux-gnu)
MIME-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/grow/JC2zkBDAat-jMQ9-hgUxPPsy3Qo>
Subject: Re: [GROW] RtgDir review: draft-ietf-grow-bgp-session-culling-04.txt
X-BeenThere: grow@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Grow Working Group Mailing List <grow.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/grow>, <mailto:grow-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/grow/>
List-Post: <mailto:grow@ietf.org>
List-Help: <mailto:grow-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/grow>, <mailto:grow-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 21 Sep 2017 00:14:19 -0000

* Stig Venaas <stig@venaas.com> [170918 17:04 -0700]:
> Nits:
> Operator (with capital O) has a specific meaning here. Also operator
> (with lower case o) is used. It seems to be consistently used as
> meaning operator in general (including Operators and Caretakers). This
> is fine, but maybe some readers might be confused by the two meanings.
> Perhaps it is worth pointing this out in the document.
> 
> Are you intentionally not using periods at the end of bullet points?
> It is missing for all bullet points in 3.2.1 and 3.2.2 except in one
> place.
> 
> This URL in the appendix should perhaps start with "https://". It
> currently just says
> "github.com/bgp/bgp-session-culling-config-examples".

Fixed, thanks for reviewing!