Re: [alto] Opsdir early review of draft-ietf-alto-oam-yang-06

Jensen Zhang <jingxuan.n.zhang@gmail.com> Tue, 25 April 2023 13:01 UTC

Return-Path: <jingxuan.n.zhang@gmail.com>
X-Original-To: alto@ietfa.amsl.com
Delivered-To: alto@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 57193C151522; Tue, 25 Apr 2023 06:01:41 -0700 (PDT)
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, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-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 oQQAPTpvKQ4g; Tue, 25 Apr 2023 06:01:37 -0700 (PDT)
Received: from mail-wm1-x331.google.com (mail-wm1-x331.google.com [IPv6:2a00:1450:4864:20::331]) (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 55E28C14CE42; Tue, 25 Apr 2023 06:01:37 -0700 (PDT)
Received: by mail-wm1-x331.google.com with SMTP id 5b1f17b1804b1-3f19c473b9eso81306705e9.0; Tue, 25 Apr 2023 06:01:37 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1682427693; x=1685019693; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=iKz4w4qeZnOZxJ6EvdPaNMntI/8pxTifJh20uHA2uMY=; b=JxK+W4tjJeffards/YvB/bFEs5X6EESuKHoLOgSKjMKXvDqtl+7GwBc52aJp6V+nNb uO/2TH6zAli7LA1nr0WqLyMhVeD7qWsuR09uFCuNdSG5pTXrLiQH7grVO30xT3udM2/C VA//jJaInk6YTAYfyS7dytDoXtK3mx3FX4I1dRbcFXTwX/0eEZVVMKhEb0eYsXL0JWUI 8aUW3scPAhyHDLRQRBGnt3tKsYzu7pLpEw43QT9mEeNGUfSJMk5asImz+D0OuBgAjHxs Y4Oqna/1S8D4WtoH11MoNEBA2natGDIiz3RfV472dyVqWmOVbYEGQOjL1w5g4imBIccD 6QHA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1682427693; x=1685019693; 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=iKz4w4qeZnOZxJ6EvdPaNMntI/8pxTifJh20uHA2uMY=; b=I0Lzd8hNphGTUHuIppXZIpFKhkTidbDeymhX30cIy3ERz5N7WF3a+mXg2E1ZqoFBHt uGQIO1q/czjUUKIdZGb7aAtWd6c+TAkyueowzaGN+l+1vfscPb94EMUm6xg0IqE/QqKg Z+3zY6McEH61Jfg1HosJQrhTssozEh5ZkIvQGkixDTj+aNaw2ZeVcxUgcxxcRYoWYltQ l5V7u6qgnAeVeaC9XsAcSgd5xiDJPiIdoS+IbAPDJIHu0aUevfgIYAepLPctvgmKSfkV sCZ300VEWPEI8iZf62bpaSJKZnaxZcRCDAB86IzslIlcGYKOlL//g+lLtgWO2YKvn3VB eJSQ==
X-Gm-Message-State: AAQBX9dnJD7jj3W8RvYpGerHkeEcJ48XZZ0u/4osk49nzTwH0mq+gL5r oSismv2pY20Nltu2mouGTYGTiq4VeyyEFWyKK0tqLCOg2Xk=
X-Google-Smtp-Source: AKy350aAin1rkGkBY70Me07b7tPJwVQrp4xEI6k8hdMr/0kEwIFiaT/F+BsA0OFd19MRA76++9z+ETcWZ/Zb503gMRQ=
X-Received: by 2002:a7b:cc15:0:b0:3f1:65cb:8156 with SMTP id f21-20020a7bcc15000000b003f165cb8156mr10499069wmh.0.1682427693142; Tue, 25 Apr 2023 06:01:33 -0700 (PDT)
MIME-Version: 1.0
References: <168206401546.30918.7998792221903169633@ietfa.amsl.com>
In-Reply-To: <168206401546.30918.7998792221903169633@ietfa.amsl.com>
From: Jensen Zhang <jingxuan.n.zhang@gmail.com>
Date: Tue, 25 Apr 2023 21:01:20 +0800
Message-ID: <CAAbpuypG54s4JOo3irhYZ9VXTXpDAWxntWVdGORSW0RsTKbgJw@mail.gmail.com>
To: Dan Romascanu <dromasca@gmail.com>
Cc: ops-dir@ietf.org, alto@ietf.org, draft-ietf-alto-oam-yang.all@ietf.org
Content-Type: multipart/alternative; boundary="00000000000002990d05fa28b937"
Archived-At: <https://mailarchive.ietf.org/arch/msg/alto/3sEgCOw6ayEG9pHbPaPGmPITcqk>
Subject: Re: [alto] Opsdir early review of draft-ietf-alto-oam-yang-06
X-BeenThere: alto@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "Application-Layer Traffic Optimization \(alto\) WG mailing list" <alto.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/alto>, <mailto:alto-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/alto/>
List-Post: <mailto:alto@ietf.org>
List-Help: <mailto:alto-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/alto>, <mailto:alto-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 25 Apr 2023 13:01:41 -0000

Hi Dan,

Sorry for the delay. Many thanks for your review. Please see our response
inline below.

On Fri, Apr 21, 2023 at 4:00 PM Dan Romascanu via Datatracker <
noreply@ietf.org> wrote:

> Reviewer: Dan Romascanu
> Review result: Has Nits
>
> Ready with Nits
>
> This document defines YANG data models for Operations, Administration, and
> Maintenance (OAM) & Management of ALTO. The operator can use these data
> models
> to set up an ALTO server, create, update and remove ALTO information
> resources,
> manage the access control, configure server discovery, and collect
> statistical
> data.
>
> I like this document. It is clearly written and very well structured. I
> liked
> the description of requirements, the information model corresponding to the
> requirements, and the extension example modules in the Appendices. These
> are
> all very useful for operators who need to understand and use the YANG
> modules.
>
> Understanding and using this document requires a good knowledge of ALTO.
>
> My review is focused on the design and data modelling issues relevant for
> operations and manageability. I did not perform a YANG review, I assume
> that
> YANG Doctors review is performed separately.
>
> This document is Ready with a couple of editorial comments.
>
> Editorial & Nits:
>
> 1. There are many more acronyms not included in Section 3 or expanded at
> first
> occurrence. Maybe the respective acronyms sections in the ALTO documents
> should
> be mentioned / referred
>

Thanks for pointing out this issue. We have included all the acronyms that
occur in the document in Sec 3. You can check the changes in our early edit
[1].

[1]:
https://ietf-wg-alto.github.io/draft-ietf-alto-oam-yang/draft-ietf-alto-oam-yang.html#name-acronyms-and-abbreviations

But we are not sure if the acronyms that only occur in the YANG modules
should also be included.


>
> 2. In Section 5.3.1.2
>
> > In practice, multiple ALTO servers can be deployed for scalability.
>    That may require communication among different ALTO servers.
>
>    The "ietf-alto" module does not contain any configuration for the
>    communication between peer ALTO servers.  Instead, it provides the
>    configuration for how an ALTO server can be discovered by another
>    ALTO server on demand (Figure 6).
>
> I understand that the communication between ALTO servers is out of scope.
> However, I do not understand how is the scalability requirement met. Is
> there /
> Will there be another YANG module to define this data model? Something else
> than YANG? Maybe this is described in another ALTO document that I did not
> find.
>

The scalability requirement is not explicitly defined in this document. It
looks like a part of R1 but is not mandatory.

And I am not quite sure what is the scalability requirement that you
mentioned here. There can be two kinds of scalability issues:

1. The scalability of a large number of network domains and elements. This
issue requires the deployment of multiple ALTO server instances in
different domains and communications among different ALTO servers in
different domains. WG is still discussing the related topic [2]. The
solution is not mature. So we consider it to be out of the scope of this
document.

[2]: https://mailarchive.ietf.org/arch/msg/alto/Hpay0QShfob_3LR7ERfpIjXvGI0/

2. The scalability of a large number of client connections. i.e., the load
balance issue. This may need some autoscaling or load-balancing
configuration parameters. Is this what you want to add?

Thanks,
Jensen