Skip to main content

Early Review of draft-ietf-alto-oam-yang-06
review-ietf-alto-oam-yang-06-opsdir-early-romascanu-2023-04-21-00

Request Review of draft-ietf-alto-oam-yang-06
Requested revision 06 (document currently at 17)
Type Early Review
Team Ops Directorate (opsdir)
Deadline 2023-03-26
Requested 2023-03-12
Requested by Mohamed Boucadair
Authors Jingxuan Zhang , Dhruv Dhody , Kai Gao , Roland Schott , Qiufang Ma
I-D last updated 2023-04-21
Completed reviews Dnsdir Last Call review of -12 by Scott Rose (diff)
Secdir Last Call review of -12 by Rich Salz (diff)
Genart Last Call review of -12 by Dan Romascanu (diff)
Dnsdir Telechat review of -14 by Scott Rose (diff)
Dnsdir Telechat review of -15 by Ted Lemon (diff)
Yangdoctors Early review of -06 by Andy Bierman (diff)
Opsdir Early review of -06 by Dan Romascanu (diff)
Tsvart Early review of -06 by Spencer Dawkins (diff)
Comments
The document is currently in the WGLC. We are interested in the directorate review to assess the design.

Thank you.
Assignment Reviewer Dan Romascanu
State Completed
Request Early review on draft-ietf-alto-oam-yang by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/Wq165rxMFpdz0O61wQ6JDrPLCLg
Reviewed revision 06 (document currently at 17)
Result Has nits
Completed 2023-04-21
review-ietf-alto-oam-yang-06-opsdir-early-romascanu-2023-04-21-00
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

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.