Re: [lisp] [Last-Call] Tsvart last call review of draft-ietf-lisp-pubsub-10

mohamed.boucadair@orange.com Fri, 27 January 2023 14:53 UTC

Return-Path: <mohamed.boucadair@orange.com>
X-Original-To: lisp@ietfa.amsl.com
Delivered-To: lisp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DBF38C14CF05; Fri, 27 Jan 2023 06:53:07 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.095
X-Spam-Level:
X-Spam-Status: No, score=-7.095 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, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001, 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=orange.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 QDxO6XGpIxzU; Fri, 27 Jan 2023 06:53:03 -0800 (PST)
Received: from relais-inet.orange.com (relais-inet.orange.com [80.12.70.35]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 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 A5CC1C14F726; Fri, 27 Jan 2023 06:53:03 -0800 (PST)
Received: from opfednr00.francetelecom.fr (unknown [xx.xx.xx.64]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by opfednr27.francetelecom.fr (ESMTP service) with ESMTPS id 4P3LCx6brhz4wmx; Fri, 27 Jan 2023 15:53:01 +0100 (CET)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=orange.com; s=ORANGE001; t=1674831181; bh=BV0t7yFT6aTWNeMTSEkmZk8x6FWwRg4vl7kV4iN3Bw4=; h=From:To:Subject:Date:Message-ID:Content-Type:MIME-Version; b=HnNxXzZlay94iMp6ONRXkowuOmIBwiOeA39vtmktIVfraONgHJunGsacVn0iHA+Jc fk7w0cGlXFmaW6nRKo1qkef4DWOjPtnuydWhqtD3wYtMnn8ox4/qzNpP/ndL/B28ZF 1SL08HEVnaFF8QkzVZofqW6XCReg+1H8siMw71VGoIUHtiZ3GAPYiAnkfr7JL4E+A9 22QiNEaFhraMx6jfcL9KbBXRffqoheMn40mSj0VNlSjDS4hIirI2TZ57IzvKZ/DUmJ PYQ5eHWHp66eHFYG7E5Or2uu99mRJTnca1BZPePkk64EDTQ2zYAbIRx5y+Q6vC2tgQ +zxdB3AgWzbFQ==
From: mohamed.boucadair@orange.com
To: Magnus Westerlund <magnus.westerlund@ericsson.com>, "tsv-art@ietf.org" <tsv-art@ietf.org>
CC: "draft-ietf-lisp-pubsub.all@ietf.org" <draft-ietf-lisp-pubsub.all@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "lisp@ietf.org" <lisp@ietf.org>
Thread-Topic: [Last-Call] Tsvart last call review of draft-ietf-lisp-pubsub-10
Thread-Index: AQHZMbC332AORioHLUKRxsKjz+Xoqa6yDnMzgABF7UA=
Content-Class:
Date: Fri, 27 Jan 2023 14:53:01 +0000
Message-ID: <22592_1674831181_63D3E54D_22592_243_1_7e419df653f94c8e951c8b17a2b3fdd3@orange.com>
References: <167456640879.36895.3989101552718202380@ietfa.amsl.com> <9846_1674756296_63D2C0C8_9846_250_1_bcfb15bc63794300b385ce5ad7a2e4c5@orange.com> <PA4PR07MB8414BEA7754C5E54FACE695795CC9@PA4PR07MB8414.eurprd07.prod.outlook.com>
In-Reply-To: <PA4PR07MB8414BEA7754C5E54FACE695795CC9@PA4PR07MB8414.eurprd07.prod.outlook.com>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
msip_labels: MSIP_Label_07222825-62ea-40f3-96b5-5375c07996e2_Enabled=true; MSIP_Label_07222825-62ea-40f3-96b5-5375c07996e2_SetDate=2023-01-27T14:30:40Z; MSIP_Label_07222825-62ea-40f3-96b5-5375c07996e2_Method=Privileged; MSIP_Label_07222825-62ea-40f3-96b5-5375c07996e2_Name=unrestricted_parent.2; MSIP_Label_07222825-62ea-40f3-96b5-5375c07996e2_SiteId=90c7a20a-f34b-40bf-bc48-b9253b6f5d20; MSIP_Label_07222825-62ea-40f3-96b5-5375c07996e2_ActionId=ffda1cd4-9a25-4b95-8594-53501477979e; MSIP_Label_07222825-62ea-40f3-96b5-5375c07996e2_ContentBits=0
x-originating-ip: [10.115.26.52]
Content-Type: multipart/alternative; boundary="_000_7e419df653f94c8e951c8b17a2b3fdd3orangecom_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/lisp/b3OPjCvG72MIpAJ3LugKuQLMU4U>
Subject: Re: [lisp] [Last-Call] Tsvart last call review of draft-ietf-lisp-pubsub-10
X-BeenThere: lisp@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: List for the discussion of the Locator/ID Separation Protocol <lisp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lisp>, <mailto:lisp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lisp/>
List-Post: <mailto:lisp@ietf.org>
List-Help: <mailto:lisp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lisp>, <mailto:lisp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 27 Jan 2023 14:53:08 -0000

Re-,

Thanks Magnus for clarifying.

I suggest to add the following in Section 6:

NEW:
   As a reminder, the initial transmission and retransmission of Map-
   Notify messages by a Map-Server follow the procedure specified in
   Section 5.7 of [RFC9301].  Some state changes may trigger an overload
   that would impact, e.g., the outbound capacity of a Map-Server.  A
   similar problem may be experienced when a large number of state were
   simultaneously updated.  To prevent such phenomena, Map-Servers
   SHOULD be configured with policies to control the maximum number of
   subscriptions and also the pace of Map-Notify messages.  The exact
   details to characterize such policies are deployment and
   implementation specific.  Likewise, this document does not specify
   which notifications take precedence when these policies are enforced.

Do we need to say more without going too much into implementation territory?

Cheers,
Med

De : Magnus Westerlund <magnus.westerlund@ericsson.com>
Envoyé : vendredi 27 janvier 2023 11:39
À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@orange.com>; tsv-art@ietf.org
Cc : draft-ietf-lisp-pubsub.all@ietf.org; last-call@ietf.org; lisp@ietf.org
Objet : Re: [Last-Call] Tsvart last call review of draft-ietf-lisp-pubsub-10

Hi Med,

Overall, the spec leverages the mechanisms in both RFC9301 and RFC9303. I don't know if you checked those when performing your review.

MW: Yes, I looked at those, and as you cite some of it I can explain why I think this isn't sufficient for this specification.

> -----Message d'origine-----
> De : last-call <last-call-bounces@ietf.org<mailto:last-call-bounces@ietf.org>> De la part de Magnus
> Westerlund via Datatracker
> Envoyé : mardi 24 janvier 2023 14:20
> À : tsv-art@ietf.org<mailto:tsv-art@ietf.org>
> Cc : draft-ietf-lisp-pubsub.all@ietf.org<mailto:draft-ietf-lisp-pubsub.all@ietf.org>; last-call@ietf.org<mailto:last-call@ietf.org>;
> lisp@ietf.org<mailto:lisp@ietf.org>
> Objet : [Last-Call] Tsvart last call review of draft-ietf-lisp-
> pubsub-10
>
> Reviewer: Magnus Westerlund
> Review result: Not Ready
>
> This document has been reviewed as part of the transport area
> review team's ongoing effort to review key IETF documents. These
> comments were written primarily for the transport area directors,
> but are copied to the document's authors and WG to allow them to
> address any issues raised and also to the IETF discussion list for
> information.
>
> When done at the time of IETF Last Call, the authors should
> consider this review as part of the last-call comments they
> receive. Please always CC tsv-art@ietf.org<mailto:tsv-art@ietf.org> if you reply to or
> forward this review.
>
> My review comments are:
>
>
> C.      When a Map-Notify is to be sent there are no discussion in
> regards to
> congestion control of the transmission of the Map-Notify.

[Med] CC is already covered in 9301. We are not repeating what is already specified in Section 5.7 of 9301:

   A Map-Server sends an unsolicited Map-Notify message (one that is not
   used as an acknowledgment to a Map-Register message) only in
   conformance with Section 3.1 ("Congestion Control Guidelines") of
   [RFC8085] and Section 3.3 ("Reliability Guidelines") of [RFC8085].  A
   Map-Notify is retransmitted until a Map-Notify-Ack is received by the
   Map-Server with the same nonce used in the Map-Notify message.  An
   implementation SHOULD retransmit up to 3 times at 3-second
   retransmission intervals, after which time the retransmission
   interval is exponentially backed off (base of 2, that is, the next
   backoff timeout interval is doubled) for another 3 retransmission
   attempts.  Map-Notify-Ack messages are only transmitted upon the
   reception of an unsolicited Map-Notify; Map-Notify-Ack messages are
   not retransmitted.
MW: Yes, the issue is that in the context of subscriber service, I think it is relevant to discuss how to ensure that the server does not try to overload its own outgoing paths as N subscribers to a particular mapping will result in N outgoing message when that mapping is updated. That N messages can't in general be sent all immediately as it will result in a large spike, for large enough values of N overloading. That needs some discussion. And as I discussed in my review the retransmission timer being fixed at 3 seconds will need to be taken into account. Because if one pace so that the pacing of the N Map-Notify so that it will take more than 3 seconds then the retansmissions will also start result in additional load when they occur.


"unsolicited Map-Notify" is what draft-ietf-lisp-pubsub is about. Section 5.7 of 9301:

   The fields of the Map-Notify are copied from the corresponding Map-
   Register to acknowledge its correct processing.  In the Map-Notify,
   the 'Authentication Data' field is recomputed using the corresponding
   per-message key and according to the procedure defined in the
   previous section.  The Map-Notify message can also be used in an
   unsolicited manner.  This topic is out of scope for this document.
   See [LISP-PUBSUB] for details.

 As the
> Map-Notify appear to be unicast IP/UDP packets being sent one per
> subscriber at the time an updated mapping is registered with the
> map-server all these messages will be generated. There need to be
> some rate limiting for this transmission to prevent congestion
> near the map-server. If sufficient number of subs are in place
> also the retransmission will have to be rate limited as not all
> Map-Notify messages will have been sent when its time to start to
> perform retransmissions.

Yes, I understood that this is what is referenced.
My point is that the pubsub mechanism creates new issues as it both build up states and when the Map-Notify messages are triggered the whole state needs to be considered in doing reasonable things.
Cheers
Magnus

_________________________________________________________________________________________________________________________

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.