update(build): relax dkms dependencies to suggestions in DEB and RPM packages #3450
+3
−2
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.
Using Falco with the modern_ebpf driver does not require kernel build dependencies. This PR updates the cmake configuration to make the
dkms
andkernel-devel
package requirements into suggestions.What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
This makes the
kernel-devel
anddkms
dependencies optional when installing thefalco
package, significantly reducing the number of packages installed when running in modern_ebpf mode.On Alma 8, installing Falco 0.39.2 from RPM requires
dkms
andkernel-devel
, which adds many dependencies for a total of 76 packages on my minimal test VM. After converting thedkms
andkernel-devel
requirements to suggestions with rpmrebuild, no additional dependencies beyondfalco
itself were required, and Falco appears to run fine with modern_ebpf. This PR should have the same effect on packages generated by cmake.Does this PR introduce a user-facing change?:
YES