Changing type for NTP fields in openconfig-system.yang

[kbchandradeep]

Changing type for root-delay, root-dispersion and offset from uint32, uint64 and uint64 respectively to oc-types:ieeefloat32

• type has been changed for 3 NTP fields in openconfig-system.yang https://github.com/openconfig/public/blob/master/release/models/system/openconfig-system.yang

• Implementation A: In NTP RFC 5905, these fields are defined as floating point numbers and implemented as floating point numbers. https://datatracker.ietf.org/doc/html/rfc5905

This difference in type has resulted in implementations to deviate these fields as not-supported.

[google-cla[bot]]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[OpenConfigBot]

No major YANG version changes in commit 4a84d0fd418743f496398ee011af6665e6dbe69d

[jsterne]

One downside of ieeefloat32 type is that the underlying YANG type is binary. The native output of this in many clients will likely just be a raw byte dump (maybe in hex) and it will be pretty non-intuitive for humans looking at the values (unless the various clients or output functions add specific support for this ieeefloat32 type). Maybe that's not a top priority here but something to consider.