Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bandwidth in uint64 or bandwidth-scientific-notation #221

Closed
italobusi opened this issue Mar 10, 2023 · 3 comments · Fixed by #232
Closed

Bandwidth in uint64 or bandwidth-scientific-notation #221

italobusi opened this issue Mar 10, 2023 · 3 comments · Fixed by #232

Comments

@italobusi
Copy link
Collaborator

The bandwidth profile parameters (cir, eir, cbs, ebs) are defined as uint64

Should they be re-defined using the bandwidth-scientific-notation?

The units for cbs, ebs are Kbytes: should we generalize the bandwidth-scientific-notation with number-scientific-notation?

@tsaad-dev
Copy link
Owner

Which YANG models are using scientific notation? Why do we need it?

@italobusi
Copy link
Collaborator Author

The bandwidth profile parameters (cir, eir, cbs, ebs) are defined as uint64

Should they be re-defined using the bandwidth-scientific-notation?

The units for cbs, ebs are Kbytes: should we generalize the bandwidth-scientific-notation with number-scientific-notation?

See past discussion in #116 and ietf-ccamp-wg/draft-ietf-ccamp-client-signal-yang#7

The solution (bandwidth-scientific-notation as defined in RFC8776-bis I-D) solves the problem with CIR, EIR and PIR but not for CBS, EBS and PBS attributes ...

@italobusi
Copy link
Collaborator Author

2023-06-23

Agreements:

  1. to have a consistent representation of packet bandwidth:

               type uint64;
               units "bits/second";
    
  2. remove bandwidth-scientific-notation data type

  3. to have a consistent representation of burst size

               type uint64;
               units "bytes";
    

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants