mrjackwills / oxker

A simple tui to view & control docker containers
MIT License
694 stars 24 forks source link

[BUG] - Memory displayed incorrect #33

Closed tanrak closed 8 months ago

tanrak commented 8 months ago

Describe the bug Hi, the memory stats does not match with the original docker stats. Dozzle for example shows the correct stats but ctop (which seems abandoned) has a similar problem with the memory stats as oxker.

To Reproduce Steps to reproduce the behavior:

  1. start oxker
  2. show memory stats
  3. compare with docker stats

Expected behavior The memory stats should be the same as docker stats.

Desktop (please complete the following information):

mrjackwills commented 8 months ago

Thanks, I'll have a look at it.

I think docker stats reports in mebibytes and oxker hopefully correctly converts to Megabytes, maybe that is also causing some issue?

tanrak commented 8 months ago

I think docker stats reports in mebibytes and oxker hopefully correctly converts to Megabytes, maybe that is also causing some issue?

I don't know. But for example my Syncthing-Container shows 37.46MiB per docker stats and oxker shows 43.67. If that were a problem with the conversion, the result would have to be 39.28?!

mrjackwills commented 8 months ago

Yeah sounds like it needs fixing, will have a look at it shortly