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

Dan Romascanu <dromasca@gmail.com> Wed, 26 April 2023 07:25 UTC

Return-Path: <dromasca@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 242AEC15153D; Wed, 26 Apr 2023 00:25:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.094
X-Spam-Level:
X-Spam-Status: No, score=-2.094 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, 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 VdT0IK8gCe2e; Wed, 26 Apr 2023 00:25:38 -0700 (PDT)
Received: from mail-lj1-x235.google.com (mail-lj1-x235.google.com [IPv6:2a00:1450:4864:20::235]) (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 77B73C151541; Wed, 26 Apr 2023 00:25:38 -0700 (PDT)
Received: by mail-lj1-x235.google.com with SMTP id 38308e7fff4ca-2a8a4cbe473so4439561fa.1; Wed, 26 Apr 2023 00:25:38 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1682493936; x=1685085936; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=6yaS5DkF9jcpR7TEPYghlV7wPhHXep1Xn0s/j7vzDO0=; b=oGZZnNJSr4dZUdziOCqwbWMxMot9LFzwYNLeqp8QR27q8kReH+5pjDeoIUNnMvk7cR 0fKAy8OjfACBbj1FtZLLij1brNF2Dep4saC43CotGFToDC/X0LRfNs7WspK3B60duAe8 qP18F5colatDXQYLdjWRyw4EPC/C9xzU1JhuaRHcPOcusplJuRj838j9fno6L0thDQQ/ kKtdXFYOxNUTqsg2WnanV1LD7jAaOrwqtgYO2hLXAc7zHWMzA0pRIh1ZACM9LW2NMyKy KLZLaWcUxmty1Ds5KYivEqd1veiSifGxtPBQ104BuaDgnDaPS9laZy+FiAJuO19VW11o YxCQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1682493936; x=1685085936; 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=6yaS5DkF9jcpR7TEPYghlV7wPhHXep1Xn0s/j7vzDO0=; b=MFWeLh69IAP6p0HjuQVkAOFhZsN35MRy0H7ALcj+9XT4DefIJ3rsdRrFTkHnQDquON onj6GNdrpuUCqJ0ek16etCe9NRXb+E14sqih0GnzZw/6IylBeia/R+7fYT2Axzg6mnH2 Kv2wDQ73s6jghvOPVJavaD7vnMn/fvyfAhZ8hOgTtn70uHKyR+0v76W3Qa/O4gL+Xogn d3RqYVyD75CgXqpGJ9wMRk2pkmhzaI4J8WQmTp0hgL+vPm0nwBvtiwZbnIARw6MooncR zHLypLbaoKOv5SxFulXt9uv/8UzFp/FFrWHt1CnCviWdWuNHlfEv2SHE3eTdvdHfH44Y rSxw==
X-Gm-Message-State: AC+VfDxiaVgKS5sGkSqYd/xjRqd7jIcKMSfqsEsPmuwbw8qzEkyLKOnh /1VxrQA1aLxe7OuPveGxt1/PyBXlVR0+JVAOPfQ=
X-Google-Smtp-Source: ACHHUZ64NfxznMrPiLs0HE7gFW5NC9DMfnWgNCyPh06x1pDGR9mG/PKEubyJF55v/VwunuOtVQEnAOB1qGwpl6aPlWw=
X-Received: by 2002:a2e:bea6:0:b0:2a5:f4dc:8b4a with SMTP id a38-20020a2ebea6000000b002a5f4dc8b4amr2041505ljr.5.1682493935539; Wed, 26 Apr 2023 00:25:35 -0700 (PDT)
MIME-Version: 1.0
References: <168206401546.30918.7998792221903169633@ietfa.amsl.com> <CAAbpuypG54s4JOo3irhYZ9VXTXpDAWxntWVdGORSW0RsTKbgJw@mail.gmail.com>
In-Reply-To: <CAAbpuypG54s4JOo3irhYZ9VXTXpDAWxntWVdGORSW0RsTKbgJw@mail.gmail.com>
From: Dan Romascanu <dromasca@gmail.com>
Date: Wed, 26 Apr 2023 10:25:24 +0300
Message-ID: <CAFgnS4W2jaSgk_U3rmSNSugqEwqTx2CZk1NESB2AfObTMSztsg@mail.gmail.com>
To: Jensen Zhang <jingxuan.n.zhang@gmail.com>
Cc: ops-dir@ietf.org, alto@ietf.org, draft-ietf-alto-oam-yang.all@ietf.org
Content-Type: multipart/alternative; boundary="0000000000005d5aa305fa3825fd"
Archived-At: <https://mailarchive.ietf.org/arch/msg/alto/hh1oQ92rX-SAyhCztTh6PmcbBEQ>
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: Wed, 26 Apr 2023 07:25:42 -0000

Hi Jensen,

Thank you for your email and for addressing my comments.

See in-line.

Regards,

Dan




On Tue, Apr 25, 2023 at 4:01 PM Jensen Zhang <jingxuan.n.zhang@gmail.com>
wrote:

> 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.
>

I believe that the answer is yes. There is no other separate abbreviations
section for the YANG modules.



>
>
>>
>> 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?
>

I was referring to the scalability issue mentioned in the document in the
sentence 'In practice, multiple ALTO servers can be deployed for
scalability.'. This seems to be related to issue #1 in your answer. If the
solution is considered not mature at this stage, maybe you should mention
just that in the document, to explain why it is out of scope.


>
> Thanks,
> Jensen
>
>
>