bug: host_metrics host_physical_cpus is capped at 10 cpus

[kaspar030]

A note for the community

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Problem

The host_metrics host_physical_cpus never shows more than 10 cpus.
I tracked that down to a bug in the used heim crate, and opened a fix there.

Configuration

[sources.host-metrics]
type = "host_metrics"

Version

timberio/vector:0.28.0-alpine

Debug Output

No response

Example Data

No response

Additional Context

No response

References

Upstream bug fix: heim-rs/heim#366

[jszwedko]

Thanks for tracking that down and opening the fix upstream! Unfortunately that crate hasn't seemed to be maintained since 2021. I think we'll need to pull the fix into the fork that we have: https://github.com/vectordotdev/heim

[kaspar030]

Let me know if you'd like me to open the PR in https://github.com/vectordotdev/heim!

[jszwedko]

Let me know if you'd like me to open the PR in https://github.com/vectordotdev/heim!

That'd be great if you feel motivated!

[kaspar030]

Here you are: vectordotdev/heim#1

[jszwedko]

Reopening since we need to pull the change into Vector to resolve.

[woodsb02]

Did this fix ever get pulled into vector?

It looks like the vector cargo.toml uses the update_nix branch here, which doesn’t appear to include this fix.
https://github.com/vectordotdev/heim/commits/update-nix

[pront]

I cherry-picked the fix: vectordotdev/heim@f3537d9

P.S. Tracker for dropping the heim dependency: #21861.