-
Notifications
You must be signed in to change notification settings - Fork 2.2k
[RFC] build: add build tag to disable eBPF #4005
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
[RFC] build: add build tag to disable eBPF #4005
Conversation
we are using runc to read the cgroup configuration, if eBPF is enabled it pulls in the cilium/ebpf dependency that increases the binary size by ~1 MB. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
|
@AkihiroSuda @kolyshkin what do you think about this flag? |
|
Sorry, NACK, as eBPF is a hard dependency for cgroup v2 (in the initial UserNS) |
|
For library use, the packages should be just split and refactored to eliminate eBPF deps |
AkihiroSuda
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NACK, please see my comments above
how do you suggest splitting it? |
|
@giuseppe this is already solved last year in #3452 (in a way described by @AkihiroSuda above). So you can try using runc from HEAD and seeing if it helps with the bloat. The only problem is, we haven't made a release having it yet. Working on it. |
I've tried with the latest runc from HEAD but I still get the ebpf dependency, and as a result the podman binary is ~1MB bigger. I'll check better if I am doing anything wrong, but I cannot see any import of |
|
I'll close this PR and investigate better the proposed way |
sadly the last release did not fix the issue I have reported. It is enough to pull |
we are using runc to read the cgroup configuration, if eBPF is enabled it pulls in the cilium/ebpf dependency that increases the binary size by ~1 MB.