Skip to content

Conversation

@4ndv
Copy link
Contributor

@4ndv 4ndv commented Jan 6, 2026

Adds support for sysctls key, which is useful for VPN and other network-related containers (I have a separate VPS for VPN which I want to include in my uncloud cluster)

Tested same as #238: deploy an app without sysctls, redeploy the same app with sysctls, watch containers to be recreated with all the keys included:

lynx@uncloud:~$ sudo docker inspect dde4712ea39c | grep Sysctls -A 3
            "Sysctls": {
                "net.ipv4.conf.all.src_valid_mark": "1",
                "net.ipv4.ip_forward": "1"
            },

Copy link
Owner

@psviderski psviderski left a comment

Choose a reason for hiding this comment

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

Awesome, thank you! There are some merge conflicts with the previous PR but looks like I can resolve them even when I'm not the PR author.

@psviderski psviderski merged commit 06c3fba into psviderski:main Jan 10, 2026
4 checks passed
@4ndv
Copy link
Contributor Author

4ndv commented Jan 10, 2026

Thank you!

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