lime-packages icon indicating copy to clipboard operation
lime-packages copied to clipboard

Shared state publishing twice babeld_hosts and bat_hosts on boot

Open germanferrero opened this issue 3 years ago • 3 comments

Both: babeld_hosts at: https://github.com/libremesh/lime-packages/blob/aa55eb84d8c02d7b23949ccc24f1b7c030d63daa/packages/shared-state-babeld_hosts/files/etc/hotplug.d/initd/shared-state-babeld_hosts#L6

and bat_hosts at: https://github.com/libremesh/lime-packages/blob/aa55eb84d8c02d7b23949ccc24f1b7c030d63daa/packages/shared-state-bat_hosts/files/etc/hotplug.d/iface/shared-state-bat_hosts.sh#L2

Are scheduled to publish on boot (and interface up).

And also at: https://github.com/libremesh/lime-packages/blob/aa55eb84d8c02d7b23949ccc24f1b7c030d63daa/packages/shared-state/files/etc/init.d/shared-state#L6 Shared State schedules all publishers to publish on boot, so we are doing it twice for these two, overloading the boot process.

germanferrero avatar Mar 18 '21 20:03 germanferrero

No found tmate

https://github.com/libremesh/lime-packages/blob/aa55eb84d8c02d7b23949ccc24f1b7c030d63daa/packages/ubus-tmate/Makefile#L16

predators46 avatar Mar 18 '21 21:03 predators46

So, which is ok to remove? @spiccinini do you have idea? It would be easier to remove the hotplug.d.

ilario avatar Feb 24 '23 21:02 ilario

We should not remove the one in /etc/init.d, see https://github.com/libremesh/lime-packages/pull/1004#issuecomment-1508090185

ilario avatar Apr 14 '23 07:04 ilario