Specify unit as Mbps for IS-IS reference-bandwidth

[earies]

• (M) release/models/isis/openconfig-isis-lsp.yang
• (M) release/models/isis/openconfig-isis-routing.yang
• (M) release/models/isis/openconfig-isis.yang
  • Add unit to reference-bandwidth
  • Increment to version 1.7.1

Change Scope

Specify unit as Mbps for IS-IS reference-bandwidth. Current definition can lead
to implementation divergence.

Platform Implementations

NOTE: Based off current public model publishes only

Arista:

• Not supported feature, deviated in OC not supported
• No native leaf

Refs:

• https://github.com/aristanetworks/yang/blob/master/EOS-4.36.0F/release/openconfig/models/isis/arista-isis-deviations.yang#L259-L263

Cisco IOS-XR:

• OC leafs not supported
• Native leafs are uint64 but kbps

Refs:

• https://github.com/YangModels/yang/blob/main/vendor/cisco/xr/2611/Cisco-IOS-XR-um-router-isis-cfg.yang#L3865-L3871
• https://github.com/YangModels/yang/blob/main/vendor/cisco/xr/2611/cisco-xr-openconfig-network-instance-deviations.yang#L3533-L3539

Nokia SROS:

• OC leafs deviated to scope reduced range of uint32
• Native leafs are uint64 but bps

Refs:

• https://github.com/nokia/7x50_YangModels/blob/master/latest_sros_26.3/nokia-types-sros.yang#L1477-L1499
• https://github.com/nokia/7x50_YangModels/blob/master/latest_sros_26.3/openconfig/nokia-sr-openconfig-isis-deviations.yang#L90-L104

Nokia SRLinux:

• OC leafs deviated to scope reduced range of uint32 (assumes Mbps)
• Native leafs are uint64 but kbps

Refs:

• https://github.com/nokia/srlinux-yang-models/blob/v26.3/srlinux-yang-models/srl_nokia/models/network-instance/srl_nokia-isis.yang#L1460-L1477
• https://github.com/nokia/srlinux-yang-models/blob/v26.3/srlinux-yang-models/openconfig/openconfig-srl-deviations.yang#L1116-L1130

Tree View

No tree updates were made as part of this patchset.

[gemini-code-assist]

Code Review

This pull request updates the ISIS YANG models to version 1.7.1, adding a new revision entry and defining the units for the reference-bandwidth leaf. The reviewer suggested changing the units from the abbreviation 'Mbps' to the full lowercase 'megabits per second' to align with OpenConfig style conventions and recommended consolidating the description string for better readability.

[sms-juniper]

changes are valid as per Rfc

https://www.rfc-editor.org/rfc/rfc9130.html#name-is-is-yang-module

  leaf reference-bandwidth {
    when "../enabled = 'true'" {
      description
        "Only when auto-cost is enabled.";
    }
    type uint32 {
      range "1..4294967";
    }
    units "Mbits";
    description
      "Configures the reference bandwidth used to automatically
       determine the interface cost (Mbits).  The cost is the
       reference bandwidth divided by the interface speed,
       with 1 being the minimum cost.";
  }