Skip to content

Conversation

@thespad
Copy link
Member

@thespad thespad commented Oct 9, 2022

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

Adds an Alpine-based branch for the wireguard container on the same basis as the existing Ubuntu branch but without the ability to compile the wireguard kernel module.

Benefits of this PR and context:

How Has This Been Tested?

Source / References:

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for opening this pull request! Be sure to follow the pull request template!

@thespad thespad changed the title Initial commit Add Alpine branch Oct 9, 2022
@thespad thespad marked this pull request as ready for review October 10, 2022 14:04
@thespad thespad self-assigned this Oct 10, 2022
@thespad thespad requested a review from a team October 10, 2022 21:32
Copy link
Member

@drizuid drizuid left a comment

Choose a reason for hiding this comment

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

lgtm, im excited to see the size difference. Is the standard assumption now that kernels on systems we care to support will have non-module support for wireguard (sans dkms)?

@thespad
Copy link
Member Author

thespad commented Oct 11, 2022

lgtm, im excited to see the size difference. Is the standard assumption now that kernels on systems we care to support will have non-module support for wireguard (sans dkms)?

Yeah, anything with kernel 5.6 or newer should have WireGuard baked in, as well as some older ones. If people really need the build support because they love their Debian 4.x kernel, the Ubuntu branch is still there.

@thespad thespad mentioned this pull request Oct 12, 2022
1 task
@thespad thespad merged commit b3b19bf into alpine Oct 12, 2022
@thespad thespad deleted the alpine-initial branch October 12, 2022 12:48
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