tsaad-dev / te

IETF TE Tunnels YANG models
16 stars 19 forks source link

Update description of lsps-state container + add sestion to describe it (e.g. 3.4) + question #98

Closed oscargdd closed 4 years ago

oscargdd commented 4 years ago

The lsps-state container is under te and holds the complete list of LSPs associated to all tunnels (not a particular tunnel ). Current description is description "List of LSPs associated with the tunnel."; Update the description to reflect it.

Also, there is one section per main container (globals, TE, interfaces), so another one on lsps-state would make sense.

Finally, all the information in that container is a duplicate from the lsps container under the tunnel container. If the intention is to maintain the list of LSPs that have been created in the Network (or device), why not add just a list of references to the leafs in primary and secondary paths?

italobusi commented 4 years ago

2020-05-22 TEAS call

To allow retrieving the list of LSPs within a given tunnel, we can add an augmentation constrained by a feature:

     +--rw tunnels
     |  +--rw tunnel* [name]

     |     +--rw primary-paths
     |     |  +--rw primary-path* [name]
     |     |     +--rw name                             string

     |     |     +--ro lsps
     |     |     |  +--ro lsp* [lsp-id]
     |     |     |  +--leafref

augmentation in the same ietf-te module:
     |     |     |  +--.......              { feature-dependent-lsp-state }

     +--ro lsps-state
     |  +--ro lsp*
     |          [source destination tunnel-id lsp-id extended-tunnel-id]
italobusi commented 4 years ago

2020-05-22 TEAS call

Open issue (from 2020-05-15): how to avoid defining five leafrefs to point to LSP state (source destination tunnel-id lsp-id extended-tunnel-id)

Possible solution:

     +--rw tunnels
     |  +--rw tunnel* [name]

     |     +--rw primary-paths
     |     |  +--rw primary-path* [name]
     |     |     +--rw name                             string

     |     |     +--ro lsps
     |     |     |  +--ro source               leafref
     |     |     |  +--ro destination          leafref
     |     |     |  +--ro tunnel-id            leafref
     |     |     |  +--ro extended-tunnel-id   leafref
     |     |     |  +--ro lsp* [lsp-id]
     |     |     |  |  +--ro lsp-id            leafref

augmentation in the same ietf-te module:
     |     |     |  |  +--.......              { feature-dependent-lsp-state }

     +--ro lsps-state
     |  +--ro lsp*
     |          [source destination tunnel-id lsp-id extended-tunnel-id]
tsaad-dev commented 4 years ago

lsps-state list can be keyed by an index 32-bit number (on controller, unique per LSPs all over the network).

image

image