Skip to content
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

libremesh: add tiny version of suggested-packages #89

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

a-gave
Copy link
Contributor

@a-gave a-gave commented Jun 20, 2023

libremesh: split suggested-packages to explicitly select batctl-default or batctl-tiny; move lime-docs-minimal to tiny version.
See libremesh/lime-packages#1028 (comment)
Tested with both profiles on ramips-mt76x8-tplink_tl-mr6400-v4

@ilario
Copy link
Member

ilario commented Jun 20, 2023

Thanks!!
I would remove more packages from the tiny version:

  • babeld-auto-gw-mode
  • check-date-http
  • lime-hwd-ground-routing
  • lime-debug

and we should also check if shared-state-nodes_and_links is used only for lime-app or for something else also.

@ilario
Copy link
Member

ilario commented Jul 7, 2023

I am not sure that removing lime-hwd-openwrt-wan is a good idea. Users then will have no WAN port and, if they want it, they will have to configure it in lime-node config file.
Do you consider that these low mem devices will not be used as a gateway?
It could be right...

@a-gave
Copy link
Contributor Author

a-gave commented Jul 7, 2023

I think you're right, lime-hwd-openwrt-wan should stay, primarily for the zero configuration for newcomers. I was confusing the goal of this profile with a cli one or similar, also because with some devices that have a DSL-WAN physical port the lime-hwd-openwrt-wan does not configure a wan port predefined port among the availables, and you need to configure a port as WAN manually.

In a valley in our cn we tried to keep a poor gateway, ramips-mt76x8-tplink_tl-mr6400-v4, for a while, but we had to change it after a short time because it was going off (~15 people). But a smaller group could still use it effectively.

shared-state-nodes_and_links is used only for lime-app or for something else also
I will do more checks on this

@ilario
Copy link
Member

ilario commented Jan 26, 2024

In a previous project meeting we discussed which packages would be included in the mini-image: https://pad.cas.cat/LibreMesh_meetup_2024#Discussion---Mini-image

Comparing the discussion with the current list of packages, seems that these packages should be removed from this pull request:

  • batctl-tiny
  • shared-state-bat_hosts
  • shared-state-nodes_and_links

And during the meeting we also decided to include babeld-auto-gw-mode, but maybe it is not really needed...
On the other hand, it should not pull too many dependencies: hotplug-initd-services gets selected anyway as a dependency of shared-state-babeld_hosts; while watchping would be added, but it seems rather small (20 kB) and does not have any further dependency.
Its function is to disable the internet announcement in gateways where internet connection is not working. So it is useful only in networks with more than one gateway.
Opinions?

@a-gave a-gave force-pushed the libremesh/suggested-packages branch from be289be to 2e5e69a Compare January 31, 2024 13:35
@a-gave
Copy link
Contributor Author

a-gave commented Jan 31, 2024

Opinions?

Seems fine to me, I've done the requested changes

@ilario ilario merged commit aca10e1 into libremesh:master Jan 31, 2024
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.

2 participants