Skip to content

feat: plugin: hw_info: added human-readable flag for memory info#1587

Merged
Conan-Kudo merged 1 commit intorpm-software-management:mainfrom
k0ste:help
May 28, 2025
Merged

feat: plugin: hw_info: added human-readable flag for memory info#1587
Conan-Kudo merged 1 commit intorpm-software-management:mainfrom
k0ste:help

Conversation

@k0ste
Copy link
Contributor

@k0ste k0ste commented May 28, 2025

I'm bit confused, when I tried to debug build failure - I expected the worker memory to be less than 4GB, I was surprised when I saw 256MB. As it turned out, it was 256GB actually, I converted the bytes
The free program prints values in kibibytes by default. Not in bytes, that more than expected without flags

This PR add's --human flag, so that the plugin output for storage and memory has the same units. Currently, such a flag is set for storage

Show all output fields automatically scaled to shortest three digit unit and display the units of print out. Following units are used:
B = bytes
Ki = kibibyte
Mi = mebibyte
Gi = gibibyte
Ti = tebibyte
Pi = pebibyte

Copy link
Member

@xsuchy xsuchy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@Conan-Kudo Conan-Kudo merged commit 892d01f into rpm-software-management:main May 28, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants