Skip to content

feat: add heap sensors - #108

Merged
broglep merged 1 commit into
meshtastic:devfrom
skgsergio:feat/gather-heap-metrics
Oct 8, 2025
Merged

feat: add heap sensors#108
broglep merged 1 commit into
meshtastic:devfrom
skgsergio:feat/gather-heap-metrics

Conversation

@skgsergio

Copy link
Copy Markdown

I'm trying to see the behavior over time of the heap usage as I have some issues with my node so I thought it was a good idea to add it as a sensor in Home Assistant.

  • Bump protobuf to version 2.6.9 (latest stable version) to support the heap_{free,total}_bytes fields in LocalStats.
  • Add Heap Total and Heap Free Home Assistant sensors.
image

@CLAassistant

CLAassistant commented Oct 3, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@broglep
broglep changed the base branch from main to dev October 8, 2025 20:11
@broglep

broglep commented Oct 8, 2025

Copy link
Copy Markdown
Collaborator

Can you rebase onto dev? only the new sensor code is really needed, dev already has newer protobufs that should contain the heap sensors.

If you can't at the moment, I'll make sure to do the rebase for a next release

@skgsergio
skgsergio force-pushed the feat/gather-heap-metrics branch from 21e11cf to c934814 Compare October 8, 2025 20:17
@skgsergio

Copy link
Copy Markdown
Author

Done @broglep, didn't know that I should target dev :)

@broglep

broglep commented Oct 8, 2025

Copy link
Copy Markdown
Collaborator

Done @broglep, didn't know that I should target dev :)

it is not that obvious unfortunately, and usually there is no dev branch, only when pre-release is in the making. didn't find time yet to finalise that release, so dev is lying around a bit too long

@broglep
broglep merged commit 0a49f01 into meshtastic:dev Oct 8, 2025
4 checks passed
@skgsergio
skgsergio deleted the feat/gather-heap-metrics branch October 9, 2025 17:11
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