Re: [GROW] Genart last call review of draft-ietf-grow-bgp-gshut-11

Job Snijders <job@ntt.net> Tue, 10 October 2017 12:00 UTC

Return-Path: <job@instituut.net>
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 172C5134D7C for <grow@ietfa.amsl.com>; Tue, 10 Oct 2017 05:00:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.418
X-Spam-Level:
X-Spam-Status: No, score=-1.418 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HEADER_FROM_DIFFERENT_DOMAINS=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SORBS_SPAM=0.5, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
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 FJMT72KFMewP for <grow@ietfa.amsl.com>; Tue, 10 Oct 2017 05:00:06 -0700 (PDT)
Received: from mail-wm0-f52.google.com (mail-wm0-f52.google.com [74.125.82.52]) (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 3B238134D78 for <grow@ietf.org>; Tue, 10 Oct 2017 05:00:01 -0700 (PDT)
Received: by mail-wm0-f52.google.com with SMTP id q132so4563347wmd.2 for <grow@ietf.org>; Tue, 10 Oct 2017 05:00:01 -0700 (PDT)
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:references :mime-version:content-disposition:in-reply-to:user-agent; bh=oDNhtBx4ZaBVUm9pNvtjZQnjWzUK74NSUz6H8LGbQrA=; b=qGILBD/8ERzX3Ix46MCmrNonlWm4Hzoeg+xOiYZ95Q7kBr01rlV5Y9xkQxbzFprz2B 2gqnMZkWPCE/RQrpOeJELt0lqcIOIpnAR5IRzx8HIF2ZyQuU45ts8u/wbglo3yjz41eV 4vZnKdUfFky4uRIvplOQ7VMbj2F4dPuUTqX3xQ0ohGRCUVrqWbR6lrJsiOgwMUU13uQD ExohuYo3AVlnFLCnoPja/AoIU/uQcI6uxEMotvfS/Pbkbld9xw0k6+sxreOHIXnNB1VK m+52nxklINxXcTuLtOh+LjfRjqvCns5wLdATMWEf8GHeQemUxyz9/4pWUbkHHx0jtP6y cbHQ==
X-Gm-Message-State: AMCzsaVml4jDzyKscOT9GhRNeqq2n3aI2pyQRrSlyFCmKjlW4uf+XtQB 3iBmDkJ5yLmBxOhyXLJJGRTYtA==
X-Google-Smtp-Source: AOwi7QCc+GlDkdaOfITJxLumZbjrzd0VbbRjYtMy3CcV855S3Gl1XsD+IbyNeclKf0ZSPaIdyNCGuQ==
X-Received: by 10.80.151.211 with SMTP id f19mr15736815edb.141.1507636799448; Tue, 10 Oct 2017 04:59:59 -0700 (PDT)
Received: from localhost ([188.206.67.206]) by smtp.gmail.com with ESMTPSA id v17sm5768677eda.70.2017.10.10.04.59.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 10 Oct 2017 04:59:58 -0700 (PDT)
Date: Tue, 10 Oct 2017 13:59:57 +0200
From: Job Snijders <job@ntt.net>
To: bruno.decraene@orange.com
Cc: Matthew Miller <linuxwolf+ietf@outer-planes.net>, "gen-art@ietf.org" <gen-art@ietf.org>, "grow@ietf.org" <grow@ietf.org>, "draft-ietf-grow-bgp-gshut.all@ietf.org" <draft-ietf-grow-bgp-gshut.all@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
Message-ID: <20171010115957.GC75071@Vurt.local>
References: <150759929173.18515.8831141207586025582@ietfa.amsl.com> <2413_1507629407_59DC995F_2413_449_1_53C29892C857584299CBF5D05346208A478A8FE2@OPEXCLILM21.corporate.adroot.infra.ftgroup> <20171010100621.GA75071@Vurt.local> <26657_1507635694_59DCB1EE_26657_410_1_53C29892C857584299CBF5D05346208A478A9426@OPEXCLILM21.corporate.adroot.infra.ftgroup>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <26657_1507635694_59DCB1EE_26657_410_1_53C29892C857584299CBF5D05346208A478A9426@OPEXCLILM21.corporate.adroot.infra.ftgroup>
X-Clacks-Overhead: GNU Terry Pratchett
User-Agent: Mutt/1.9.1 (2017-09-22)
Archived-At: <https://mailarchive.ietf.org/arch/msg/grow/sZXJ0lV7x3nF7jpOkGiOsBzjoHk>
Subject: Re: [GROW] Genart last call review of draft-ietf-grow-bgp-gshut-11
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: Tue, 10 Oct 2017 12:00:08 -0000

On Tue, Oct 10, 2017 at 11:41:32AM +0000, bruno.decraene@orange.com wrote:
>  > Any attribute (origin, as_path, aggregator) anywhere can be overloaded
>  > to mean something only significant to the local network. I think the
>  > document is simpler without this and see no point in mentioning this. I
>  > propose:
>  > 
>  > OLD:
>  >     The LOCAL_PREF value must be lower than the one of the alternate
>  >     path. 0 being the lowest value, it can be used in all cases, except
>  >     if it already has a special meaning within the AS.
>  > NEW:
>  >     The LOCAL_PREF value SHOULD be lower than any of the alternative
>  >     paths. It is RECOMMEND to use 0, the lowest LOCAL_PREF value.
> 
> What is really needed is "The LOCAL_PREF value SHOULD be lower than
> the one of the alternative path." Looks reasonable to extend it to
> your proposition " The LOCAL_PREF value SHOULD be lower than any of
> the alternative paths." So I'm changing for this.
> 
> Now the value is truly local to an AS, and I'm not sure to see the
> technical reason to RECOMMEND (SHOULD) a specific value. MAY seems
> more appropriate to me. Hence I'm proposing to keep "Zero being the
> lowest value, it MAY be used whichever LOCAL_PREF values are used by
> the AS."

So the total of the new text is as following?

    "The LOCAL_PREF value SHOULD be lower than any of the alternative
    paths.  Zero being the lowest value, it MAY be used whichever
    LOCAL_PREF values are used by the AS."

I am not sure about the second sentence, it seems hard to read.

I see value in just recommending a value for people moving between ASNs
(debugging other organisation's networks via BGP looking glasses) to
recognise as a highly undesirable path. Reading RFC 2119 the
'RECOMMENDED' seems appropiate, "use 0 unless you have a reason not to".
This is a GROW document and I believe clear-cut guidance will benefit
all.

> I'm open to remove "If LOCAL_PREF zero already has a special meaning
> within the AS, and there is a need to distinguish both usages, another
> low value MAY be used." If you believe that this sentence add
> complexity. I'd agree that it is somewhat redundant, although it does
> provides a specific point to consider.

thanks.

Kind regards,

Job