Re: [Idr] Comment on terminology in Section 4 of draft-ietf-idr-segment-routing-te-policy-24

Ketan Talaulikar <ketant.ietf@gmail.com> Mon, 23 October 2023 10:43 UTC

Return-Path: <ketant.ietf@gmail.com>
X-Original-To: idr@ietfa.amsl.com
Delivered-To: idr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D03A5C14EB1E; Mon, 23 Oct 2023 03:43:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.104
X-Spam-Level:
X-Spam-Status: No, score=-2.104 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id cCavJUCW7mYV; Mon, 23 Oct 2023 03:42:58 -0700 (PDT)
Received: from mail-ed1-x531.google.com (mail-ed1-x531.google.com [IPv6:2a00:1450:4864:20::531]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D949BC14CE51; Mon, 23 Oct 2023 03:42:57 -0700 (PDT)
Received: by mail-ed1-x531.google.com with SMTP id 4fb4d7f45d1cf-53e855d7dacso4817223a12.0; Mon, 23 Oct 2023 03:42:57 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1698057775; x=1698662575; darn=ietf.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=gaF6pFXy07M3Yknu2QjhtGkd9S9YBZD/0Zdw0Y5BBnY=; b=O1gITS0YerrVJv4exiwFfU1u3u3eB9EVcB1Dp3N3bbbXi9Gl9iyoCHqsXbOPIZFkTE hPct48/JRCuQtVRINkwkFkDS/DVMBtsUpa0mTWnPAI4bk0Yp6BQnchQGnhD9ZeRcLLkL 3vmV6HQkbbrLQlAeFT5k6UyLX2lRfcq1RXUJxtkPsQQG/F7xNXShZ7kQRKkEo+GidSKb 6UmI7mN4GsG+2IyYiHuj+hCCDXGzW7IFIQyAxFqkyd30rmBBfL/W5DDumW7I3nsJ+9n0 0kY5ciHAuZzmmuR8pjV4CFJdscJbavTgbYN6n0dMbUt3Bjp53SgCXjZ39vhDxE6phzXR SDwg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1698057775; x=1698662575; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=gaF6pFXy07M3Yknu2QjhtGkd9S9YBZD/0Zdw0Y5BBnY=; b=Sfc7kzczbLRyMOQcYRQsy9jDfLvXlIgJ7Eu/exu1/RL5S3sqevJqBnLXW+N+UdHMc6 /tk04VTd2GpKMyNPS5AY50YQt6mSiljl1ueutp9twiHaEDiG+t32tFJPt2XbWNS2XGP1 I+11dzncCM5RwACq+EMnsX1lo2kXtbaKwPN6lYdODcV89utkeIdZ9pM3i/9oo/VIufeD esdoy4a1MDWZkcGJyT0xxYLFaA/WwTsPM4FipY8WAwh8Gukhiw0DSym0MGU5sd4spRJo /DH88LWMWfWfCuIzg+8FSNuU+J6ihrdy20H/Oyz++yiCm7vIVGdKsN+CJIQEscgTMXbA dcDw==
X-Gm-Message-State: AOJu0YxuvoN75Y9Hi40HQ4hMxwVUQgHbESdEkpDj7SKZ/sv2kysvyXlQ ON1MpTwswYAqQuXBNC/Oc9CAFTkWaAbqxJkDBQPY4GnHH6Q=
X-Google-Smtp-Source: AGHT+IF9iQLMSHABUkg/fmllkTlrFKpCWThH+OeSCfbL8Zn8meqOxXHLwqRv3f1zRn9r3N+ymbg/mWRMMjXlsmKHlVY=
X-Received: by 2002:a17:906:dc92:b0:9c7:59d1:b2ce with SMTP id cs18-20020a170906dc9200b009c759d1b2cemr6132495ejc.5.1698057775264; Mon, 23 Oct 2023 03:42:55 -0700 (PDT)
MIME-Version: 1.0
References: <B9D1C4AC-780C-4E2A-B044-F3032704EF98@juniper.net>
In-Reply-To: <B9D1C4AC-780C-4E2A-B044-F3032704EF98@juniper.net>
From: Ketan Talaulikar <ketant.ietf@gmail.com>
Date: Mon, 23 Oct 2023 16:12:43 +0530
Message-ID: <CAH6gdPyiG-YAORoNbxzi6r8NyfQRBDRG=vt8gc0T4CfRTEfHeA@mail.gmail.com>
To: John Scudder <jgs@juniper.net>
Cc: "draft-ietf-idr-segment-routing-te-policy@ietf.org" <draft-ietf-idr-segment-routing-te-policy@ietf.org>, "idr@ietf.org" <idr@ietf.org>
Content-Type: multipart/alternative; boundary="00000000000080b87b06085fe2a0"
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/PXsWPuWMeeQoPx_pQleoD10rcwc>
Subject: Re: [Idr] Comment on terminology in Section 4 of draft-ietf-idr-segment-routing-te-policy-24
X-BeenThere: idr@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Inter-Domain Routing <idr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/idr>, <mailto:idr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/idr/>
List-Post: <mailto:idr@ietf.org>
List-Help: <mailto:idr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/idr>, <mailto:idr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 23 Oct 2023 10:43:01 -0000

Hi John,

Thanks again for your comments and helpful suggestions. We've just posted
an update with changes to address your comments:
https://datatracker.ietf.org/doc/html/draft-ietf-idr-segment-routing-te-policy-26

There is now an overview of the entire process on the receiving router as
it relates to RFC4271 at the beginning of Sec 4.2 "Reception of an SR
Policy NLRI". As you've indicated, we do need to explain the term "usable"
since it varies from the base BGP spec - that is now included. The term
"acceptable" has been done away with since that is nothing by "validation".
The term "propagate" is fairly well-known and even referenced (though not
formally) in RFC4271 - but a reference to the specific section of RFC4271
is now added as suggested by you.

Could you please take a look and let me know what you think?

Thanks,
Ketan (on behalf of co-authors)


On Tue, Sep 26, 2023 at 9:39 PM John Scudder <jgs@juniper.net> wrote:

> Hi Authors,
>
> I was looking for something in draft-ietf-idr-segment-routing-te-policy-24
> and got hung up on your use of terminology in Section 4 and its subsections.
>
> You use several terms that are colloquially pretty clear — “acceptable,"
> “usable,” and “propagation” are the ones I picked out. My concern is that
> these aren’t well-defined terms of art in the context of BGP. One fix would
> be to rewrite the section with exquisitely correct reference to the model
> used by RFC 4271, talking about Adj-RIBs-In, Adj-RIBs-Out, movement of
> routes between them, and so on. That would be a lot of work, though, and
> would make the document less readable for the casual punter. So I don’t
> want to advocate for that.
>
> It seems to me that a reasonable middle ground would be to introduce
> definitions of these terms, along the lines of,
>
> 4.2.1
>
> The term “acceptable” as used in this section refers to whether a route is
> syntactically and semantically well-formed and is eligible to be considered
> for route selection during the decision process (Section 9.1 of [RFC4271]).
>
> 4.2.2
>
> The term “usable” as used in this section refers to whether a route that
> was selected by the decision process for installation (Section 9.1. of
> [RFC4271]) is eligible to be instantiated in the forwarding plane of the
> local router, but does not limit its eligibility for propagation (see
> Section 4.2.4).
>
> 4.2.4
>
> The term “propagate” as used in this section refers to what Section 9.1.3
> of [RFC4271] calls “route dissemination”.
>
> (I’ll grant you that RFC 4271 does itself talk about propagating things,
> in a few places, so it’s debatable as to whether this definition is
> strictly needed. But read on.)
>
> I think/hope some updates like these might make the sentence that
> currently opens Section 4.2.4 a little less mind-bendingly weird when read
> as plain English: "SR Policy NLRIs that have been determined acceptable and
> valid can be evaluated for propagation, even the ones that are not usable.”
> The (ab)use of the term “usable” is the key here, the other definitions are
> of less importance but it seems worth being thorough. The quoted sentence
> is, in a way, the exception that proves the rule — by which I mean, if the
> model had been straightforward to begin with, it wouldn’t have been
> necessary to introduce a sentence to say “yes yes, you can reflect a route
> even though we told you it’s not ‘usable’."
>
> I doubt the text I’ve proposed is ideal, it’s just a first draft.
>
> I note that a basic problem here is that in our document set, we’ve
> diverged fairly far from the “select, install in Loc-RIB, forward” paradigm
> used by RFC 4271, yet we haven’t introduced any new formal model or terms.
> First, the model was twisted around by l3vpns, but at least those kept the
> concept of different RIBs and FIBs so it still kind of worked. Now you’re
> taking building blocks introduced for l3vpn and pushing them further, but
> without carrying along the RIB and FIB constructs. I’m reminded of cartoon
> tropes where a character stands on thin air and is fine until they look
> down… and then they fall. I kind of have the feeling that we have walked
> out past the “floor” of our formal model, such as it was, and are standing
> on thin air, and some of our specs only work as long as we don’t look down.
> Metaphorically speaking of course.
>
> Hopefully, it will be clear that I’m not seeking terminological perfection
> here, and I don’t think it’s the job of this spec or group of authors to
> fix the formal foundations of our protocol. I'm just hoping to keep the
> document clear and precise enough that people who aren’t part of the
> in-crowd can reasonably hope to use it.
>
> Thanks,
>
> —John