[exporter/utils] mem conversion float err

rivosinc / prometheus-slurm-exporter

Export select slurm metrics to prometheus

Apache License 2.0

37 stars 12 forks source link

[exporter/utils] mem conversion float err #72

Closed abhinavDhulipala closed 3 months ago

abhinavDhulipala commented 3 months ago

Change the datatype for mem string conversion. The map shouldn't really have ever had int value types as we cast and return a float anyway.

resolves #71

abhinavDhulipala commented 3 months ago

I'm pretty sure the reason it failed is because the golang type int means different things to different platforms, based in the kernel. But if we use int32/64 or float32/64 golang will make sure to use a consistent underlying data type. Please test out the branch by installing and building the branch.