-
Notifications
You must be signed in to change notification settings - Fork 111
packages: prometheus...-wifi-stations-extra: use babeld if available #1230
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
packages: prometheus...-wifi-stations-extra: use babeld if available #1230
Conversation
ilario
left a comment
There was a problem hiding this 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.
119b0c2 to
937d4ef
Compare
|
Thanks, yes tested on a ubnt_nanostation-m-xw running latest lime code and openwrt 24.10.4. |
|
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. |
|
If you consider so, open an issue so that we remember this :) |
|
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
If some tests could speed the time of merge I'll try to test it in the next days |
|
Nice ! !
Yes please ! it would be great if you could test the pull request
Two students from the university are also working on the testing topic. I
have just created a matrix channel to talk about this.
Here is a brief summary
* GitHub project for activity tracking: [
https://github.com/users/francoriba/projects/3](https://github.com/users/francoriba/projects/3)
* Our fork of `openwrt-tests` (currently with one PR open, and another one
ready to be submitted, in which we already added the three Belkin routers
to the project infrastructure): [
https://github.com/francoriba/openwrt-tests/tree/feat/add-fcefyn-labnet](https://github.com/francoriba/openwrt-tests/tree/feat/add-fcefyn-labnet)
* Report repository in LaTeX: [
https://github.com/francoriba/PI_Casanueva_Riba](https://github.com/francoriba/PI_Casanueva_Riba)
* Repository with scripts/configurations for setting up our host: [
https://github.com/ccasanueva7/pi-hil-testing-utils/tree/main](https://github.com/ccasanueva7/pi-hil-testing-utils/tree/main)
* Fork of Víctor’s repository for vWiFi tests: [
https://github.com/francoriba/libremesh-virtual-mesh](https://github.com/francoriba/libremesh-virtual-mesh)
Maybe we can talk about this in a meet
Saludos
Ing. Javier Alejandro Jorge
…On Wed, 17 Dec 2025 at 08:22, a-gave ***@***.***> wrote:
*a-gave* left a comment (libremesh/lime-packages#1230)
<#1230 (comment)>
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)
<#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
—
Reply to this email directly, view it on GitHub
<#1230 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABE2I2HKTI54L42X6Y7DWDT4CE4HDAVCNFSM6AAAAACOM3V7TKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTMNRUHA4TSNRSGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
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.