[alto] Comments for draft-ietf-alto-oam-yang-03

[Jordi Ros Giralt <[e-mail]>]

Hi Jensen and authors,

Below my comments, thanks.


Comments related to the two YANG Models:
-----------------------------------------------------------

* In the Abstract paragraph: "The operator can use the data model to create and update ALTO information resources". Then, in Section 5.4.2 we have "The operator of the ALTO server can use this model to create, update, and remove the ALTO information resource", indicating the support for "removing" information resources. Should we change the sentence in the abstract to include "remove"? It would look like this:

s/The operator can use the data model to create and update ALTO information resources/The operator can use the data model to create, update and remove ALTO information resources/

(The same change would need to be applied in the Introduction where that same sentence appears.)

* In the Abstract paragraph: "(...) manage the access control, configure server-to-server communication and server discovery (...)". Yet in Section 5.3.1.2 we have "The YANG module defined in this document does not contain any configuration for the communication between two ALTO servers". So the two statements seem to be somewhat contradictory, the first one stating that the model can be used to "configure server-to-server communication" while the second one implying that "configuration for the communication between two ALTO servers" is out of scope.

(The same change would need to be applied in the Introduction where that same sentence appears.)

* Paragraph 5.4.2 says: "Each resoruce entry provides configurations defining how to create or update an ALTO information resource." This seems to indicate that the OAM model provides semantics to allow the operator to create and update an IR, but it's not obvious how this is done. Would you be able to provide an example of how the model can be used to create (or update) an IR?

* The sentence "One categories of the type-specific parameters are common for the same type of ALTO information resource" is a bit confusing ("One" is singular, "categories" is plural)

* I think where it says "The other categories of the type-specific parameters are algorithm-specific" it should say "The other category of the type-specific parameters is algorithm-specific". Since earlier the text says there are "two categories" and this refers to the 2nd category (singular).

* In the 'http-listen' feature in Section 7.1, should we add a 'reference' field pointing to RFC 7285?


Implementation questions:
-------------------------------------------------------

* Have the two YANG models described in this document being implemented as part of OpenALTO? Are there plans to do so? Would be a great way to get a sense of how the YANG models perform on "running code".

Comments about grammar and nits:
--------------------------------------------------

s/What is in the scope of this document?/The following is in the scope of this document/

s/What is not in the scope of this document?/The following is not in the scope of this document/

s/Data model for deploy an ALTO server/Data model for deploying an ALTO server/

s/Data model for operate and manage/Data model for operating and managing/

s/configuration for ALTO services/configuration of ALTO services/

s/in this document provide all the/in this document provides all the/

s/how an ALTO client discover/how an ALTO client discovers/

s/Cases for other mechanisms can be augmented in the future modules./Future modules can be augmented to support other mechanisms./

s/If the on-change attribute presents,/If the on-change attribute is present,/

s/Otherwise, if the feed-interval attribute presents,/Otherwise, if the feed-interval attribute is present,/

s/MUST present/MUST be present/

s/to create, update, and remove the ALTO information resource/to create, update, and remove ALTO information resources/

s/Each resoruce/Each resource/

s/MUST also include the a dependency/MUST also include a dependency/

s/One categories of the type-specific parameters are common for the same type of ALTO information resource/One category of the type-specific parameters are common for the same type of ALTO information resource/

s/all the other types of resource entries have augmented algorithm node/all the other types of resource entries have an augmented algorithm node/

s/An example of extending algorithm node/An example of extending the algorithm node/

s/at information resource level/at the information resource level/

s/is supported by the following configuration/is supported using the following configuration/

s/depolyment/deployment/

s/This grouping defines data model/This grouping defines a data model/

s/This grouping defines base data model/This grouping defines the base data model/

s/Grouping for configuration of how to set up server discovery/Grouping for the configuration of how to set up server discovery/

-------------------

Thanks,
Jordi  (Speaking as individual)