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

tunneldigger: add L2TPv3 tunneling #21308

Merged

Conversation

PolynomialDivision
Copy link
Member

@PolynomialDivision PolynomialDivision commented Jun 8, 2023

tunneldigger-broker: add broker for tunneldigger

In mesh communities, tunneldigger is widely used to create L2TPv3 tunnels
and mesh via them. Since the broker is typically installed on other
distributions, the openwrt broker package has not received any
maintenance in recent years [0]. I  take now care of the further maintaince
of this package. Furthermore, I consulted with the maintainers to ensure
that they were comfortable with the change [1].

This PR is just a refactoring of the already existing opkg package from
wlanslovenija. It fixes config parsing and in general the config, adapts
to the new python syntax and fixes dependency handling.

- [0] https://github.com/wlanslovenija/firmware-packages-opkg/tree/master/net/tunneldigger-broker
- [1] https://github.com/wlanslovenija/firmware-packages-opkg/issues/24
tunneldigger: add package for establishing L2TPv3 tunnels over UDP

In the previous commit we already added tunneldigger-broker. Add the
corresponding client.

This PR is just a refactoring of the already existing opkg package from
wlanslovenija [0].

[0] - https://github.com/wlanslovenija/firmware-packages-opkg/tree/master/net/tunneldigger

Maintainer: me
Compile tested: mt7621
Run tested: mt7621

@PolynomialDivision PolynomialDivision force-pushed the tunneldigger-broker branch 4 times, most recently from b7de7fe to 482e420 Compare June 8, 2023 12:12
@PolynomialDivision PolynomialDivision changed the title [WIP] tunneldigger-broker: add broker for tunneldigger [WIP] tunneldigger: add L2TPv3 tunneling Jun 8, 2023
@PolynomialDivision PolynomialDivision force-pushed the tunneldigger-broker branch 2 times, most recently from f92dc38 to 5bf3081 Compare June 8, 2023 12:50
@PolynomialDivision PolynomialDivision changed the title [WIP] tunneldigger: add L2TPv3 tunneling tunneldigger: add L2TPv3 tunneling Jun 11, 2023
@PolynomialDivision PolynomialDivision marked this pull request as ready for review June 11, 2023 10:06
@PolynomialDivision
Copy link
Member Author

CI fails because of some issue that is not directly related with the packages:

 Collected errors:
 * pkg_hash_check_unresolved: cannot find dependency libxtables12 for tc-mod-iptables
 * pkg_hash_fetch_best_installation_candidate: Packages for tc-mod-iptables found, but incompatible with the architectures configured
 * pkg_hash_check_unresolved: cannot find dependency libxtables12 for tc-full
 * pkg_hash_fetch_best_installation_candidate: Packages for tc-full found, but incompatible with the architectures configured
 * satisfy_dependencies_for: Cannot satisfy the following dependencies for tunneldigger-broker:
 * 	libxtables12
 * opkg_install_cmd: Cannot install package tunneldigger-broker.

net/tunneldigger-broker/Makefile Outdated Show resolved Hide resolved
net/tunneldigger/Makefile Outdated Show resolved Hide resolved
net/tunneldigger/files/tunneldigger.init Outdated Show resolved Hide resolved
@PolynomialDivision PolynomialDivision force-pushed the tunneldigger-broker branch 3 times, most recently from 64d4e0e to 9419164 Compare June 12, 2023 08:22
In mesh communities, tunneldigger is widely used to create L2TPv3 tunnels
and mesh via them. Since the broker is typically installed on other
distributions, the openwrt broker package has not received any
maintenance in recent years [0]. I  take now care of the further maintaince
of this package. Furthermore, I consulted with the maintainers to ensure
that they were comfortable with the change [1].

This PR is just a refactoring of the already existing opkg package from
wlanslovenija. It fixes config parsing and in general the config, adapts
to the new python syntax and fixes dependency handling.

- [0] https://github.com/wlanslovenija/firmware-packages-opkg/tree/master/net/tunneldigger-broker
- [1] wlanslovenija/firmware-packages-opkg#24

Signed-off-by: Nick Hainke <vincent@systemli.org>
In the previous commit we already added tunneldigger-broker. Add the
corresponding client.

This PR is just a refactoring of the already existing opkg package from
wlanslovenija [0].

[0] - https://github.com/wlanslovenija/firmware-packages-opkg/tree/master/net/tunneldigger

Signed-off-by: Nick Hainke <vincent@systemli.org>
@1715173329
Copy link
Member

lgtm ;)

@PolynomialDivision
Copy link
Member Author

lgtm ;)

Thanks a lot for your review. :)

@PolynomialDivision PolynomialDivision merged commit bd2b4f3 into openwrt:master Jun 12, 2023
10 of 12 checks passed
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.

None yet

3 participants