Updated mpls-te-types

Updated mpls-te-types to address Tarek's review comments as discussed during the call on 2020-07-10
tsaad-dev · Jul 10, 2020 · 6565ce7 · 6565ce7
1 parent acce44b
commit 6565ce7
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 13 deletions.
diff --git a/ietf-mpls-te-types.yang b/ietf-mpls-te-types.yang
@@ -38,7 +38,7 @@ module ietf-mpls-te-types {
      This version of this YANG module is part of RFC XXXX; see
      the RFC itself for full legal notices.";
 
-  revision 2020-07-03 {
+  revision 2020-07-10 {
     description
       "Initial Version";
     reference
@@ -86,17 +86,22 @@ module ietf-mpls-te-types {
     type enumeration {
       enum per-flow {
         description
-          "The load-balancing algorithm ensures that packets for
-           the same flow are forwarded on the same component link.
+          "The load-balancing algorithm ensures that packets
+           characterized as the same flow (e.g. based on IP 5-tuple)
+           that egress on a LAG or a bundled TE link are forwarded
+           on the same component link.
 
            Packets for different flows within the same LSP can be
            forwarded on different component links.";
       }
       enum per-top-label {
         description
-          "The load-balancing algorithm ensures that packets, which
-           are forwarding based on the same label at the top of the
-           label stack, are forwarded on the same component link.
+          "The load-balancing algorithm ensures incoming MPLS
+           packets with the same top MPLS label and that egress on
+           a on a LAG or bundled TE link are forwarded on the same
+           component link.
+
+
 
            Packets for different flows within the same LSP are
            forwarded on the same component link.";
@@ -166,10 +171,7 @@ module ietf-mpls-te-types {
     description
       "Link Bandwidth for Packet. ";
     leaf packet-bandwidth {
-      type uint64{
-        range "0..10000000000";
-      }
-      units "Kbps";
+      type bandwidth-kbps;
       description
         "Available bandwith value expressed in kilobits per
          second";

diff --git a/ietf-te-mpls-topology.tree b/ietf-te-mpls-topology.tree
@@ -25,17 +25,17 @@ module: ietf-te-mpls-topology
             /tet:te-link-attributes/tet:max-link-bandwidth
             /tet:te-bandwidth/tet:technology:
     +--:(packet)
-       +--rw packet-bandwidth?   uint64
+       +--rw packet-bandwidth?   bandwidth-kbps
   augment /nw:networks/nw:network/nt:link/tet:te
             /tet:te-link-attributes/tet:max-resv-link-bandwidth
             /tet:te-bandwidth/tet:technology:
     +--:(packet)
-       +--rw packet-bandwidth?   uint64
+       +--rw packet-bandwidth?   bandwidth-kbps
   augment /nw:networks/nw:network/nt:link/tet:te
             /tet:te-link-attributes/tet:unreserved-bandwidth
             /tet:te-bandwidth/tet:technology:
     +--:(packet)
-       +--rw packet-bandwidth?   uint64
+       +--rw packet-bandwidth?   bandwidth-kbps
   augment /nw:networks/nw:network/nw:network-types/tet:te-topology
             /tet-pkt:packet:
     +--rw mpls-topology!