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

Fix openwrt dnsmasq/dhcp on boot #773

Merged
merged 1 commit into from
Jul 14, 2023
Merged

Fix openwrt dnsmasq/dhcp on boot #773

merged 1 commit into from
Jul 14, 2023

Conversation

mgziminsky
Copy link
Contributor

@mgziminsky mgziminsky commented Jul 13, 2023

Use noop instead of comment to remove calls to procd_add_jail so usages inside a shell pipeline don't result in syntax errors.

openwrt/openwrt@936df71 added a call to procd_add_jail_mount as part of a short-circuit test. Using a comment breaks the script.

Tested in a proxmox VM

Use noop instead of comment to remove calls to `procd_add_jail` so usages inside a shell pipeline don't result in syntax errors

Signed-off-by: Michael Ziminsky (Z) <mgziminsky@users.noreply.github.com>
@stgraber stgraber merged commit fd23290 into lxc:master Jul 14, 2023
4 checks passed
@imEgo
Copy link

imEgo commented Jan 16, 2024

lxc.cap.drop: sys_admin makes dnsmasq not working, this cap dropped in openwrt.common.conf, after commented this, dnsmasq working on PVE 8.1.4 with privileged Container

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