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

Add Option to Use Base 10 for Quota Warnings #3650

Open
jarulsamy opened this issue Jun 28, 2024 · 1 comment
Open

Add Option to Use Base 10 for Quota Warnings #3650

jarulsamy opened this issue Jun 28, 2024 · 1 comment

Comments

@jarulsamy
Copy link

My institution uses base 10 for all storage quotas. When quotas are calculated from our storage system, we convert the raw quota from bytes to KiB by dividing by 1024 to generate the JSON file for OOD quota warnings. However, this leads to misleading discrepancies between our internal quota reporting for billing (which uses base 10) and OOD.

Also, it is worth mentioning that the quota warnings displayed within OOD are suffixed with KB, MB, GB, etc. are calculated based on 1024 bytes per KB, 1024 KB per MB and so on. These are also generally referred to as Kibibytes (KiB), Mibibytes (MiB) and so on[1] while the metric variants (base 10) are referred to as -byte to avoid ambiguity. Unfortunately, there are three competing standards for this, Metric, IEC, and JEDEC, so some feedback on which standards to follow would be appreciated.

I propose that we:

  • Add an option to use base 10 instead of base 2 for the block calculations.
  • Use the correct suffix depending on the base. [K,M,G,T,P]iB for base 2, [K,M,G,T,P]B for base 10.

[1] - SI and international standard IEC 80000-13 consider base 2 as -bibyte - https://en.wikipedia.org/wiki/Byte#Multiple-byte_units

@osc-bot osc-bot added this to the Backlog milestone Jun 28, 2024
@johrstrom
Copy link
Contributor

Hi, sorry for not commenting on this sooner.

I propose that we:
Add an option to use base 10 instead of base 2 for the block calculations.
Use the correct suffix depending on the base. [K,M,G,T,P]iB for base 2, [K,M,G,T,P]B for base 10.

Yea, I think you're right. We can definitely correct the suffix soon and adding an option for base 10 sounds reasonable.

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

No branches or pull requests

3 participants