Skip to content

Conversation

de4np
Copy link

@de4np de4np commented Apr 21, 2021

Running wg-quick in docker will fail even though the container namespace
has net.ipv4.conf.all.src_valid_mark enabled.

Running wg-quick in docker will fail even though the container namespace
has net.ipv4.conf.all.src_valid_mark enabled.
@zx2c4-bot zx2c4-bot force-pushed the master branch 9 times, most recently from 542b7c0 to 4ad6d0d Compare June 25, 2021 20:54
@zx2c4-bot zx2c4-bot force-pushed the master branch 3 times, most recently from c39c2ea to ab264f7 Compare July 13, 2021 10:26
@zx2c4-bot zx2c4-bot force-pushed the master branch 2 times, most recently from 4574b8e to 0b7d982 Compare June 19, 2025 15:09
@Dramelac
Copy link

Any update on this ?
Tested localy, it works and would be very nice to add.
This addition avoids giving the container too many privileges when the sysctl configuration has already been set (from docker for example).

@zx2c4 If you can merge this it would be great, thank you very much !

@zx2c4
Copy link
Member

zx2c4 commented Jun 23, 2025

Needs signed-off-by and a real committer name.

@Dramelac
Copy link

Needs signed-off-by and a real committer name.

Thank you for answering this fast!

Do you want me to open a new PR to comply with this signing requirements or can you copy paste the single ligne from this PR ?

@zx2c4
Copy link
Member

zx2c4 commented Jun 23, 2025

The author of the PR needs to do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants