[GROW] Re: Ketan Talaulikar's Discuss on draft-ietf-grow-bmp-bgp-rib-stats-16: (with DISCUSS)

Jeffrey Haas <jhaas@pfrc.org> Wed, 03 December 2025 17:20 UTC

Return-Path: <jhaas@pfrc.org>
X-Original-To: grow@mail2.ietf.org
Delivered-To: grow@mail2.ietf.org
Received: from localhost (localhost [127.0.0.1]) by mail2.ietf.org (Postfix) with ESMTP id BF3A494B3376; Wed, 3 Dec 2025 09:20:58 -0800 (PST)
X-Virus-Scanned: amavisd-new at ietf.org
X-Spam-Flag: NO
X-Spam-Score: -1.898
X-Spam-Level:
X-Spam-Status: No, score=-1.898 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail2.ietf.org ([166.84.6.31]) by localhost (mail2.ietf.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id VsJl-TnNhnWH; Wed, 3 Dec 2025 09:20:56 -0800 (PST)
Received: from slice.pfrc.org (slice.pfrc.org [67.207.130.108]) by mail2.ietf.org (Postfix) with ESMTP id 43A2A94B3369; Wed, 3 Dec 2025 09:20:56 -0800 (PST)
Received: from smtpclient.apple (99-188-202-8.lightspeed.livnmi.sbcglobal.net [99.188.202.8]) by slice.pfrc.org (Postfix) with ESMTPSA id D3A051E24F; Wed, 3 Dec 2025 12:20:55 -0500 (EST)
Content-Type: multipart/alternative; boundary="Apple-Mail=_A2CF42B8-A690-4A0C-91E2-45CE5D8F6767"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3696.120.41.1.10\))
From: Jeffrey Haas <jhaas@pfrc.org>
In-Reply-To: <PAUP264MB6756F2B9E139429A5D8DD29388D9A@PAUP264MB6756.FRAP264.PROD.OUTLOOK.COM>
Date: Wed, 03 Dec 2025 12:20:55 -0500
Message-Id: <ECB8BC19-6605-470E-A050-1823B9F9DB75@pfrc.org>
References: <176407367468.2442453.4055788151534416089@dt-datatracker-5bd94c585b-wk4l4> <f6fa9cd2-ae47-44c0-a9e8-ee29da35c8a3@ntt.net> <29467fea-057d-4b04-bd78-1d69e244a63b@ntt.net> <PAUP264MB6756F2B9E139429A5D8DD29388D9A@PAUP264MB6756.FRAP264.PROD.OUTLOOK.COM>
To: Mohamed Boucadair <mohamed.boucadair@orange.com>
X-Mailer: Apple Mail (2.3696.120.41.1.10)
Message-ID-Hash: C353A2TZNXHBCLKEOXQLFGI7AKQDWO3J
X-Message-ID-Hash: C353A2TZNXHBCLKEOXQLFGI7AKQDWO3J
X-MailFrom: jhaas@pfrc.org
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-grow.ietf.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
CC: The IESG <iesg@ietf.org>, "draft-ietf-grow-bmp-bgp-rib-stats@ietf.org" <draft-ietf-grow-bmp-bgp-rib-stats@ietf.org>, "grow-chairs@ietf.org" <grow-chairs@ietf.org>, "grow@ietf.org" <grow@ietf.org>, "draft-ietf-grow-bmp-path-marking-tlv@ietf.org" <draft-ietf-grow-bmp-path-marking-tlv@ietf.org>
X-Mailman-Version: 3.3.9rc6
Precedence: list
Subject: [GROW] Re: Ketan Talaulikar's Discuss on draft-ietf-grow-bmp-bgp-rib-stats-16: (with DISCUSS)
List-Id: Grow Working Group Mailing List <grow.ietf.org>
Archived-At: <https://mailarchive.ietf.org/arch/msg/grow/HYdGVdLNZlZ0U7Z-H8R4luFxNNU>
List-Archive: <https://mailarchive.ietf.org/arch/browse/grow>
List-Help: <mailto:grow-request@ietf.org?subject=help>
List-Owner: <mailto:grow-owner@ietf.org>
List-Post: <mailto:grow@ietf.org>
List-Subscribe: <mailto:grow-join@ietf.org>
List-Unsubscribe: <mailto:grow-leave@ietf.org>

Note for the archives: With -17 and stats 24/25 covering primary/backup behavior being removed, and additional clarification on the views for each counter, my comments are resolved.

-- Jeff


> On Dec 3, 2025, at 9:44 AM, mohamed.boucadair@orange.com wrote:
> 
> Hi all, 
> 
> Thank you all for the constructive discussion.
> 
> The authors will release SOON a new version that offloads these two types to the marking TLV spec, better clarify the applicability of the other types, and some other minor edits.
> 
> The other points are beyond the scope of this spec. Specifically, no changes will be made in this draft to per AFI/SAFI and how we demux stats.
> 
> Cheers,
> Med
> 
>> -----Message d'origine-----
>> De : Paolo Lucente <paolo@ntt.net <mailto:paolo@ntt.net>>
>> Envoyé : samedi 29 novembre 2025 18:57
>> À : Ketan Talaulikar <ketant.ietf@gmail.com <mailto:ketant.ietf@gmail.com>>; The IESG
>> <iesg@ietf.org <mailto:iesg@ietf.org>>
>> Cc : draft-ietf-grow-bmp-bgp-rib-stats@ietf.org <mailto:draft-ietf-grow-bmp-bgp-rib-stats@ietf.org>; grow-
>> chairs@ietf.org <mailto:chairs@ietf.org>; grow@ietf.org <mailto:grow@ietf.org>; draft-ietf-grow-bmp-path-marking-
>> tlv@ietf.org <mailto:tlv@ietf.org>
>> Objet : [GROW] Re: Ketan Talaulikar's Discuss on draft-ietf-grow-
>> bmp-bgp-rib-stats-16: (with DISCUSS)
>> 
>> 
>> 
>> Hi Ketan, Med, Authors,
>> 
>> Following up on the two open discussion points:
>> 
>> discuss 1) The only two defined stats that touch the concept of
>> "primary" and "backup" are types 24 and 25; in draft-ietf-grow-
>> bmp-path-marking-tlv path statuses are being defined -- and there
>> is more to it than just primary and backup. Evolving from my
>> previous email, i propose that these two stat types are removed
>> from draft-ietf-grow-bmp-bgp-rib-stats mainly for consistency to
>> draft-ietf-grow-bmp-path-marking-tlv and to avoid dependencies
>> among the two documents; instead we can define stats for all
>> defined path status in the path marking document; this, i guess,
>> would also close this discussion point;
>> 
>> discuss 2) On the specific guidance point for future documents,
>> please see
>> https://fra01.safelinks.protection.outlook.com/?url=https%3A%2F%2F <https://fra01.safelinks.protection.outlook.com/?url=https%3A%2F%2F>
>> mailarchive.ietf.org <http://mailarchive.ietf.org/>%2Farch%2Fmsg%2Fgrow%2F6pqYmYyy2V7eVuNHkERiLd5
>> qnrM%2F&data=05%7C02%7Cmohamed.boucadair%40orange.com <http://40orange.com/>%7Cf83b5eb119
>> cd4e3faf7208de2f70d6d5%7C90c7a20af34b40bfbc48b9253b6f5d20%7C0%7C0%
>> 7C639000358873193586%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRyd
>> WUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%
>> 3D%3D%7C0%7C%7C%7C&sdata=2Wcuk2TYx0ybNZLboIuVjHoMs7uLKyYkF%2Bp4Oa7
>> 5BIM%3D&reserved=0
>> . Away from the greasy technical details, in short, the BMPv4
>> document would be a more suitable place than this document where
>> to provide guidance and straighten a few aspects out.
>> 
>> Paolo
>> 
>> 
>> On 25/11/25 21:52, Paolo Lucente wrote:
>>> 
>>> Hi Ketan,
>>> 
>>> On the two discussion points:
>>> 
>>> discuss 1) Complementing answers from Jeff: while it's not the
>> role of
>>> this document or draft-ietf-grow-bmp-path-marking-tlv to make
>> any
>>> definition (ie. route vs path, primary vs backup etc.), we have
>> two
>>> documents that speak about things with a certain degree of
>> affinity:
>>> maybe we can avoid both to use similar terminology
>> independently; we
>>> could explain the terminology in one document
>>> (draft-ietf-grow-bmp-path-marking-tlv would be the place to do
>> that,
>>> IMO) and place a reference in the other and let it re-use the
>> terminology.
>>> 
>>> The immediate con that comes to mind is that we introduce a
>> dependency
>>> among a document already in IESG court over one that has still a
>> bit
>>> of mileage to do in the WG (although i think we are almost done
>> with it).
>>> 
>>> A further idea could be to lock the two documents up by adding a
>> "path
>>> status" field in relevant stats types defined in
>>> draft-ietf-grow-bmp-bgp-rib-stats referencing the path code
>> points
>>> defined in draft-ietf-grow-bmp-path-marking-tlv; the main con i
>> see is
>>> that - guess we would agree on a static format for stats (see
>> next
>>> point) - it would break auto-parsing of stats in a BMP
>> collector.
>>> 
>>> discuss 2) There is a couple of points to unpack:
>>> 
>>> BMP messages include a per-peer header where there are peer
>> flags.
>>> Turning and twisting some of these, one can say whether content
>> of the
>>> BMP message belongs to Adj-Rib-In pre/post policy, Adj-Rib-Out
>>> pre/post policy, Loc-Rib. Of course one can't mix-and-match
>> stats for
>>> different vantage points as part of the same Stats message; one
>> Stats
>>> message per covered vantage point is needed -- sub-optimal but
>> this is
>>> how BMP operates today and, especially for periodic messages,
>> maybe good enough.
>>> 
>>> On Global vs per-AFI/SAFI messages: where possible i like to
>> favor a
>>> static format, for example every message would be per-AFI/SAFI
>> where
>>> if AFI/SAFI are both set to zero it means it's Global. The pro
>> is that
>>> we would make stats auto-parseable by a collector; the con is
>> that we
>>> would potentially waste 3 bytes per stat TLV -- something we
>> could
>>> further sophisticate, saving auto-parsing, by introducing an
>> innocent
>>> bit saying whether AFI/SAFI will follow or not before the gauge
>> /
>>> value. This would avoid your duplication point, Ketan, and you
>> are
>>> right that currently there is no guidance in this sense -- hence
>> myself throwing some ideas.
>>> 
>>> Paolo
>>> 
>>> 
>>> On 25/11/25 09:27, Ketan Talaulikar via Datatracker wrote:
>>>> Ketan Talaulikar has entered the following ballot position for
>>>> draft-ietf-grow-bmp-bgp-rib-stats-16: Discuss
>>>> 
>>>> When responding, please keep the subject line intact and reply
>> to all
>>>> email addresses included in the To and CC lines. (Feel free to
>> cut
>>>> this introductory paragraph, however.)
>>>> 
>>>> 
>>>> Please refer to
>>>> 
>> https://fra01.safelinks.protection.outlook.com/?url=https%3A%2F%2F <https://fra01.safelinks.protection.outlook.com/?url=https%3A%2F%2F>
>> www
>>>> .ietf.org <http://ietf.org/>%2Fabout%2Fgroups%2Fiesg%2Fstatements%2Fhandling-
>> ballot-posi
>>>> 
>> tions%2F&data=05%7C02%7Cmohamed.boucadair%40orange.com <http://40orange.com/>%7Cf83b5eb11
>> 9cd
>>>> 
>> 4e3faf7208de2f70d6d5%7C90c7a20af34b40bfbc48b9253b6f5d20%7C0%7C0%7C
>> 639
>>>> 
>> 000358873221044%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIl
>> YiO
>>>> 
>> iIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C
>> 0%7
>>>> 
>> C%7C%7C&sdata=%2FvsOouJwIl8lBLY1cbMsb%2F%2FwBSzDuG4PUjFPg%2FdnbxM%
>> 3D&
>>>> reserved=0 for more information about how to handle DISCUSS and
>>>> COMMENT positions.
>>>> 
>>>> 
>>>> The document, along with other ballot positions, can be found
>> here:
>>>> 
>> https://fra01.safelinks.protection.outlook.com/?url=https%3A%2F%2F <https://fra01.safelinks.protection.outlook.com/?url=https%3A%2F%2F>
>> dat
>>>> atracker.ietf.org <http://atracker.ietf.org/>%2Fdoc%2Fdraft-ietf-grow-bmp-bgp-rib-
>> stats%2F&data=0
>>>> 
>> 5%7C02%7Cmohamed.boucadair%40orange.com <http://40orange.com/>%7Cf83b5eb119cd4e3faf7208de
>> 2f7
>>>> 
>> 0d6d5%7C90c7a20af34b40bfbc48b9253b6f5d20%7C0%7C0%7C639000358873237
>> 184
>>>> 
>> %7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwM
>> CIs
>>>> 
>> IlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdat
>> a=F
>>>> VkXH0mnULoMHa2xlZeM0dbVvTg%2Fqid%2BQUK5SI5XQIo%3D&reserved=0
>>>> 
>>>> 
>>>> 
>>>> ---------------------------------------------------------------
>> ------
>>>> -
>>>> DISCUSS:
>>>> ---------------------------------------------------------------
>> ------
>>>> -
>>>> 
>>>> Thanks to the authors and the WG for this document.
>>>> 
>>>> Note: this ballot has been updated for v16 of the document. The
>>>> previous number of points is retained. Points that have been
>>>> addressed are deleted.
>>>> 
>>>> Please find below certain points that I would like to discuss.
>>>> 
>>>> <discuss-1> Semantics of routes, paths, primary, and backup.
>>>> 
>>>> Section 2 of this document says:
>>>> Primary route: A route to a prefix that is considered the best
>> route
>>>> by the BGP decision process [RFC4271] and actively used for
>>>> forwarding traffic to that prefix. Backup route: A backup route
>> is
>>>> eligible for route selection, but it is not selected as the
>> primary
>>>> route and is also installed in the Loc-RIB. It is not used
>> until all
>>>> primary routes become unreachable. Backup routes are used for
>> fast
>>>> convergence in the event of failures.
>>>> 
>>>> Consider an BGP route for destination prefix x/y is a
>> multipath:
>>>> x/y via BGP NH1 (path1) (best)
>>>>      via BGP NH2 (path2) (multipath - say ECMP)
>>>>      via BGP NH3 (path3) (backup)
>>>>      via BGP NH4 (path4) (valid but not best/multipath/backup)
>>>>      via BGP NH5 (path5) (invalid - for whatsover reason)
>>>> 
>>>> This is a single route. The
>> best/multipath/backup/valid/invalid/etc
>>>> are qualifiers of its paths. Except for two stats that refer to
>> paths
>>>> (stale and suppressed), everything is referring to routes. I
>> would
>>>> like to discuss the semantics of route vs path. It seems to me
>> like
>>>> some of the stats are for paths and not routes.
>>>> 
>>>> In general, I think the use of the terms primary/backup which
>> are
>>>> related to forwarding plane aspects can be confusing. Instead,
>>>> perhaps using terms that are more suitable for BGP Loc-RIB
>> would be
>>>> better? I've suggested some of them above for consideration.
>> Also
>>>> refer to draft-ietf-grow-bmp-path-marking-tlv - the terms of
>> stats
>>>> should be aligned across the BMP documents?
>>>> 
>>>> Furthermore, there is a wrong assumption that backup paths are
>> only
>>>> activated when all primary paths are down. This is very much
>>>> implementation dependent.
>>>> Some implementations have a 1:1 provisioning of primary/backup
>> -
>>>> where the backup would get used when its specific primary goes
>> down -
>>>> this draws on the FRR notion in the forwarding planes. Refer to
>> the
>>>> definition in draft-ietf-grow-bmp-path-marking-tlv
>>>> 
>>>> These clarifications have implications on several of the stats
>> as
>>>> they are defined currently.
>>>> 
>>>> <discuss-2> Section 3 has the following text and Section 4
>> introduces
>>>> a table that brings up an interesting aspect.
>>>> 
>>>> "This section defines different statistics type for Adj-RIB-In
>> and
>>>> Adj-RIB-Out monitoring type. Some of these statistics are also
>>>> applicable to Loc-RIB; refer to Section 4 for more details."
>>>> 
>>>> For types 24 through 28, they are applicable for both Adj-RIB-
>> In and
>>>> Loc-RIB.
>>>> How does one know what is being reported? Can this be
>> clarified?
>>>> Seems like this is the first document introducing such
>> overloaded
>>>> types but I don't find the reason why this is being done. There
>> is
>>>> also a sort of duplication for same stat being both global as
>> well as
>>>> per afi/safi - is there any guidance on whether only one of
>> them
>>>> needs to be supported (this way avoiding the race conditions
>> and
>>>> discrepancies in their totaling)?
>>>> 
>>>> It is important to clarify these aspects if this is going to
>> set a
>>>> precedent/guidance for other similar stats in BMP in future
>> documents?
>>>> 
>>>> 
>>>> 
>>>> 
>>>> 
>>>> _______________________________________________
>>>> GROW mailing list -- grow@ietf.org
>>>> To unsubscribe send an email to grow-leave@ietf.org
>> 
>> _______________________________________________
>> GROW mailing list -- grow@ietf.org
>> To unsubscribe send an email to grow-leave@ietf.org
> ____________________________________________________________________________________________________________
> Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
> pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
> a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
> Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.
> 
> This message and its attachments may contain confidential or privileged information that may be protected by law;
> they should not be distributed, used or copied without authorisation.
> If you have received this email in error, please notify the sender and delete this message and its attachments.
> As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
> Thank you.
> 
> _______________________________________________
> GROW mailing list -- grow@ietf.org <mailto:grow@ietf.org>
> To unsubscribe send an email to grow-leave@ietf.org <mailto:grow-leave@ietf.org>