Skip to content

Conversation

@a-gave
Copy link
Contributor

@a-gave a-gave commented Dec 8, 2025

This collector currently doesn't work with profile-libremesh-suggested-packages-tiny
due to the lack of shared-state-bat_hosts (since this is based on shared-state-async which is too big for 8-64 devices)

This commit allows the collector to print hostnames taking the info from /etc/babeld-hosts
It also introduces a check to verify the existence of /tmp/dhcp.leases,
this in case of a setup without anygw or another dhcp server.

Copy link
Member

@ilario ilario left a comment

Choose a reason for hiding this comment

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

Didn't test but the code and the concept look good to me. Feel free to merge this if you tested it.

This collector currently doesn't work with profile-libremesh-suggested-packages-tiny
due to the lack of shared-state-bat_hosts (since this is based on shared-state-async
which is too big for 8/64 devices)

This commit allows the collector to print hostnames taking the info from /etc/babeld-hosts
It also introduces a check to verify the existence of /tmp/dhcp.leases,
this in case of a setup without anygw or another dhcp server.
@a-gave a-gave force-pushed the packages/prometheus-node-exporter-lua-wifi-stations-extra branch from 119b0c2 to 937d4ef Compare December 8, 2025 20:39
@a-gave
Copy link
Contributor Author

a-gave commented Dec 8, 2025

Thanks, yes tested on a ubnt_nanostation-m-xw running latest lime code and openwrt 24.10.4.
Fixed the wording.

@a-gave a-gave merged commit c38b2b4 into libremesh:master Dec 8, 2025
1 check passed
@a-gave
Copy link
Contributor Author

a-gave commented Dec 10, 2025

Mmm.. this was not sufficient. I tested it on two ubnt_nanostation-m-xw but they both were connected to some tplink_cpe510-v3 which is the only device which has the same mac for wlan0-mesh and br-lan so the match via /etc/babeld-hosts works. Doesn't work with different devices.
Still need to figure out a better way to retrieve hostnames. Maybe parsing the nodes_and_links.json

@ilario
Copy link
Member

ilario commented Dec 11, 2025

If you consider so, open an issue so that we remember this :)

@javierbrk
Copy link
Collaborator

due to the lack of shared-state-bat_hosts (since this is based on shared-state-async which is too big for 8-64 devices)
do you think that the static building is enough ? #1218 (comment)
that pull requires two pull requests approval, one in the shared state async repo and another one in the lime packages, I wold like to confirm that someone else has tested it to approve both.

@a-gave
Copy link
Contributor Author

a-gave commented Dec 17, 2025

due to the lack of shared-state-bat_hosts (since this is based on shared-state-async which is too big for 8-64 devices)
do you think that the static building is enough ? #1218 (comment)

Yes, totally, I was awaiting for the merge to test if it fits in outdoor 8/64 devices both in openwrt-24.10.4 and SNAPSHOT

that pull requires two pull requests approval, one in the shared state async repo and another one in the lime packages, I wold like to confirm that someone else has tested it to approve both.

If some tests could speed the time of merge I'll try to test it in the next days
Still need to go further with https://github.com/a-gave/openwrt-tests and add tests for libremesh

@javierbrk
Copy link
Collaborator

javierbrk commented Dec 19, 2025 via email

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