Build a statically linked version of kmod #3981
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue number:
Updates #3968
Description of changes:
This change builds a statically linked version of
kmod
.This newkmod
is unlinked from the existing/bin/kmod
and its symlinks. It is installed in/usr/libexec/kmod
This new version of
kmod
replaces the existing one..so
files are still provided for dependenciesBy providing a statically linked kmod, containers can mount it and load kernel modules without compatibility issues. This is a first step to unblocking #3968
Testing done:
Tested on an admin container
Sample pod definition that used
Based on the helm charts I see on Cilium's github they do use
privileged: true
on some of their containers. They also mount the/lib/modules/
in the same wayTerms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.