haomianzheng / IETF-ACTN-YANG-Model

IETF Optical YANG models in ACTN Architecture
4 stars 4 forks source link

Few minor editorial updates to OTN Topology YANG model #139

Closed italobusi closed 5 months ago

italobusi commented 1 year ago
  1. the client-svc presence container should be outside the te presence container according to https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/files/10510430/new-ml-topology-models-00.txt

    OLD

    +--rw nt:link* [link-id]
       +--rw tet:te!
          +--rw te-link-attributes
             +--rw otnt:client-svc!
             +--rw interface-switching-capability*
                     [switching-capability encoding]
    
    +--rw nt:termination-point* [tp-id]
       +--rw tet:te!
          +--rw otnt:client-svc!
          +--rw interface-switching-capability*
                  [switching-capability encoding]

    NEW

    +--rw nt:link* [link-id]
       +--rw otnt:client-svc!
       +--rw tet:te!
          +--rw te-link-attributes
             +--rw interface-switching-capability*
                    [switching-capability encoding]
    
    +--rw nt:termination-point* [tp-id]
       +--rw otnt:client-svc!
       +--rw tet:te!
          +--rw interface-switching-capability*
                  [switching-capability encoding]
  2. c/otn-ltp/otn-link-tp/ since the acronym ltp is not used in RFC8795 for YANG attribute names

  3. c/otn/otn-bandwidth/ for bandwidth augmentations for alignment with MW topology and other cases (otn-node, otn-link, otn-link-tp, otn-label-range)

italobusi commented 9 months ago

Update the YANG and the document then send a mail to the CCAMP WG mailing list to inform about the changes

italobusi commented 8 months ago

Update the YANG and the document then send a mail to the CCAMP WG mailing list to inform about the changes

  • [x] @italobusi : update the YANG model
  • [ ] @danielkinguk : update the I-D and send a mail to the CCAMP WG mailing list to inform about the changes

YANG model updated with PR #156

italobusi commented 5 months ago

Comments 2 and 3 fixed in draft-ietf-ccamp-layer1-types-18

Comment 1 superseded by issue #159