Wrong memory percentage on /admin/reports/redis

+1, just adding a patch here for people using composer patches.

Hopefully this can get merged!

Patch from #10 fixed the issue in Drupal Version 9.5.10 and redis 1.7.0.
Can this be merged into the next stable release?
Thank you

I feel like this solution is not very future-proof, as it casts any data type to an integer. This would mean that the percentage would also break in the event that you set the max memory to 1GB and the amount used is in MBs. I feel like the solution would be to first parse the values to their count in bytes.

Hi @Dobefu the module was already performing this cast so it wouldn't break anything, see the diff. It's just correcting a math error in the original design.

Other cases not handled can be tackled in follow up issues.

That's probably fair. I do feel like the cast shouldn't happen, but I agree that this isn't the issue to be discussing that

the cast is applied to the result of the divide, to avoid displaying a float, it's basically a round().

However, the MR doesn't apply anymore.

Rebased and added new test coverage for the status report page, including the percentage and also discovered a bug in another recently committed issue that I had to revert.

Merged.

Automatically closed - issue fixed for 2 weeks with no activity.