Re: [CCAMP] FW: I-D Action: draft-ietf-ccamp-l1csm-yang-02.txt

["t.petch" <[e-mail]>]

----- Original Message -----
From: "Leeyoung" <[e-mail]>
To: <[e-mail]>
Cc: "Martin, David (Nokia - CA/Ottawa)" <[e-mail]>
Sent: Wednesday, June 20, 2018 4:49 PM

> Hi,
>
> The revision incorporated comments received from the last CCAMP
meeting and the MEF Q2 meeting in April.
>
> I believe the draft is stable.

I hope not:-)

I find it hard to read; the indentation is, for me excessive, causing
lines to spill off the page.  You are indenting each lower level in the
YANG model by 10 spaces whereas I would prefer you followed in the
footsteps of
draft-ietf-ccamp-mw-yang
which uses two.

The tree diagram notation is now an RFC and I would expect you to
reference that and not include your own notation, again as in
draft-ietf-ccamp-mw-yang

5 Security Considerations needs to call out the sensitive nodes by name
or say that there are none, again as in
draft-ietf-ccamp-mw-yang-.

The YANG modules need Copyright as in
draft-ietf-ccamp-mw-yang

The Introduction needs to say whether or not the YANG module is NMDA
compatible, as in
draft-ietf-ccamp-mw-yang

The import in the YANG modules should have reference statements, as in
draft-ietf-ccamp-mw-yang

You need references in the I-D for YANG and Netconf, as in
draft-ietf-ccamp-mw-yang

I would expect to see examples, as in
draft-ietf-ccamp-mw-yang

I think that some abbreviations should be expanded in the definitions
 SLS
UNI
PE
CE
p,c,o

I wonder if the future would be simpler if the protocol types were
grouped, eg into Ethernet, FiberChannel, STM etc.  It depends how they
will be used, but I would expect there to be conditional statements
based on these groups.Your view of what makes a sensible group will be
better than mine but I think that the current flat structure is unlikely
to be easy to use.  A comparable change was introduced after the YANG
doctor review of
draft-ietf-ccamp-mw-yang

Tom Petch

                                Please feel free to provide your
feedback.
> Thanks & best regards,
>
> Young (on behalf of other co-authors)
>
> -----Original Message-----
> From: CCAMP [mailto:[e-mail]] On Behalf Of
[e-mail]
> Sent: Wednesday, June 20, 2018 10:39 AM
> To: [e-mail]
> Cc: [e-mail]
> Subject: [CCAMP] I-D Action: draft-ietf-ccamp-l1csm-yang-02.txt
>
>
> A New Internet-Draft is available from the on-line Internet-Drafts
directories.
> This draft is a work item of the Common Control and Measurement Plane
WG of the IETF.
>
>         Title           : A Yang Data Model for L1 Connectivity
Service Model (L1CSM)
>         Authors         : G. Fioccola
>                           K. Lee
>                           Y. Lee
>                           D. Dhody
>                           D. Ceccarelli
> Filename        : draft-ietf-ccamp-l1csm-yang-02.txt
> Pages           : 24
> Date            : 2018-06-20
>
> Abstract:
>    This document provides a YANG data model for Layer 1 Connectivity
>    Service Model (L1CSM).
>
>
>
>
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-ietf-ccamp-l1csm-yang/
>
> There are also htmlized versions available at:
> https://tools.ietf.org/html/draft-ietf-ccamp-l1csm-yang-02
> https://datatracker.ietf.org/doc/html/draft-ietf-ccamp-l1csm-yang-02
>
> A diff from the previous version is available at:
> https://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-l1csm-yang-02
>
>
> Please note that it may take a couple of minutes from the time of
submission until the htmlized version and diff are available at
tools.ietf.org.
>
> Internet-Drafts are also available by anonymous FTP at:
> ftp://ftp.ietf.org/internet-drafts/
>
> _______________________________________________
> CCAMP mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/ccamp
>
> _______________________________________________
> CCAMP mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/ccamp