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

lxc: add missing deps #25719

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

lxc: add missing deps #25719

wants to merge 2 commits into from

Conversation

graysky2
Copy link
Contributor

@graysky2 graysky2 commented Jan 10, 2025

Several deps are missing based the output of lxc-checkconfig shown below before this commit is applied.

Note - CONFIG_IP_NF_TARGET_MASQUERADE and CONFIG_IP6_NF_TARGET_MASQUERADE are only needed for lxc-net which we do not package. I included a patch in a separate commit to remove them from the check to minimize confusion of OpenWrt users.

% lxc-checkconfig | grep missing
Cgroup device: missing
CONFIG_IP_NF_TARGET_MASQUERADE: missing
CONFIG_IP6_NF_TARGET_MASQUERADE: missing
CONFIG_NETFILTER_XT_TARGET_CHECKSUM: missing
FUSE (for use with lxcfs): missing
checkpoint restore: missing
CONFIG_UNIX_DIAG: missing
CONFIG_INET_DIAG: missing
CONFIG_PACKET_DIAG: missing
CONFIG_NETLINK_DIAG: missing

Note - this PR is dependent on openwrt/openwrt#17553

Maintainer: @ratkaj

graysky2 added a commit to graysky2/openwrt that referenced this pull request Jan 10, 2025
These four symbols are needed to satisfy lxc dependencies.

Note - this needs to be merged before openwrt/packages#25719

Signed-off-by: John Audia <therealgraysky@proton.me>
@graysky2 graysky2 force-pushed the lxc-missing branch 2 times, most recently from d30a6d2 to 9873c93 Compare January 10, 2025 02:57
utils/lxc/Config.in Outdated Show resolved Hide resolved
graysky2 added a commit to graysky2/openwrt that referenced this pull request Jan 10, 2025
These four symbols are needed to satisfy lxc dependencies.

Note - this needs to be merged before openwrt/packages#25719

Signed-off-by: John Audia <therealgraysky@proton.me>
graysky2 added a commit to graysky2/openwrt that referenced this pull request Jan 10, 2025
These symbols are needed to satisfy lxc dependencies.

Note - this needs to be merged before openwrt/packages#25719

Signed-off-by: John Audia <therealgraysky@proton.me>
We do not package lxc-net or lxcfs so remove the lines within
lxc-checkconfig that looks forCONFIG_IP_NF_TARGET_MASQUERADE and
CONFIG_IP6_NF_TARGET_MASQUERADE which, as far as I know, is the only
part of lxc that needs them. Also remove the check for FUSE since we do
not pakcage lxcfs.

Without this commit, users will see these two as missing.

Signed-off-by: John Audia <therealgraysky@proton.me>
Several deps are missing based the output of lxc-checkconfig shown below
before this commit is applied.

Note - CONFIG_IP_NF_TARGET_MASQUERADE and CONFIG_IP6_NF_TARGET_MASQUERADE
are only needed for lxc-net which we do not package.

% lxc-checkconfig | grep missing
Cgroup device: missing
CONFIG_IP_NF_TARGET_MASQUERADE: missing
CONFIG_IP6_NF_TARGET_MASQUERADE: missing
CONFIG_NETFILTER_XT_TARGET_CHECKSUM: missing
FUSE (for use with lxcfs): missing
checkpoint restore: missing
CONFIG_UNIX_DIAG: missing
CONFIG_INET_DIAG: missing
CONFIG_PACKET_DIAG: missing
CONFIG_NETLINK_DIAG: missing

Note - this PR is dependent on openwrt/openwrt#17553

Signed-off-by: John Audia <therealgraysky@proton.me>
graysky2 added a commit to graysky2/openwrt that referenced this pull request Jan 12, 2025
These symbols are needed to satisfy lxc dependencies.

Note - this needs to be merged before openwrt/packages#25719

Signed-off-by: John Audia <therealgraysky@proton.me>
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