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

Add optionnal initContainer to fix sysctl config #162

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AlexisDucastel
Copy link

This request will add an optionnal flag to helm chart values.yaml, that let people add an initContainer to fix sysctl missing "net.ipv4.ip_forward=1" issue. (like #140)

Here is an exrtract of the documentation I added to values/yaml :

sysctlInitContainer flag adds an initContainer named "init-sysctl" to wg-access-server deployment.
The goal is to set the sysctl net.ipv4.ip_forward=1 to allow packet routing through node.
This initContainer needs to run as privileged, but this is only limited to the initContainer run time, the main container will remain unprivileged as expected.
Use case :
DNS is functionning properly through VPN but does not work for standard traffic.

NB : If you have no problem with wireguard traffic, you should not enable this initContainer

DasSkelett pushed a commit to DasSkelett/wg-access-server that referenced this pull request Apr 14, 2022
…ng-1.18.1-alpine

Bump golang from 1.18.0-alpine to 1.18.1-alpine
@SoarinFerret
Copy link

I'd love to see this merged - has bit me more than once at this point. I went ahead and published my own chart with these changes for internal use, but I'd prefer to use the upstream one long term.

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