Opened 5 years ago

Closed 5 years ago

#12648 closed Bug Report - General (Invalid)

Incorrect Metric on Disk Space Summaries

Reported by: mythtv@… Owned by: Rob Smith
Priority: minor Milestone: unknown
Component: MythTV - Services API - Backend Version: 0.28-beta
Severity: medium Keywords:
Cc: Ticket locked: no


On the MythTV Status Page, Machine Information block Disk space used is listed in MB, should be GB

Change History (3)

comment:1 Changed 5 years ago by sphery

Component: Plugin - MythWebMythTV - Services API - Backend
Status: newinfoneeded_new

On my system, storage space is properly reported. The units used are always MB.

I think you're saying that you would prefer if the MythTV backend status page (returned by the MythTV backend, not part of MythWeb, so I changed the component) used a more human-friendly format for storage space reporting (like df's --human-readable argument) such that rather than showing:

Space Free: 750,749 MB

it would show:

Space Free: 733 GB

which implies that rather than showing:

Total Space: 25,262,247 MB

it would show:

Total Space: 24 TB

and so on (since GB are not appropriate for everyone's configuration any more than MB are). Is this what you're saying, or are you saying that the values presented to you are off by a factor of 1024?

And, FWIW, I won't take a side regarding the MythTV backend status page's use of MiB while showing the unit MB.

comment:2 Changed 5 years ago by mythtv@…

My old eyes mistook the ',' in the metrics for a '.'. So I thought the figures were off by 1000. There is nothing wrong with your code. Sorry for the distraction.

Please do not spend another microsecond on this, but if you wish to pursue the MB/GB/TB avenue, the "human readable" options in many standard commands (e.g. ls du tree) all implement this. Perhaps there is a common API that will perform the conversion for you.

comment:3 Changed 5 years ago by Bill Meek

Resolution: Invalid
Status: infoneeded_newclosed

Thanks for getting back.

Note: See TracTickets for help on using tickets.