-
Notifications
You must be signed in to change notification settings - Fork 295
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
falco: allow multiple socket paths for containerd #655
Comments
Issues go stale after 90d of inactivity. Mark the issue as fresh with Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with Provide feedback via https://github.com/falcosecurity/community. /lifecycle stale |
/remove-lifecycle stale |
Related falcosecurity/falco#3266 |
This likely needs to be addressed in 0.39, since falcosecurity/falco#3266 has been merged. /assign @alacuku |
Issues go stale after 90d of inactivity. Mark the issue as fresh with Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with Provide feedback via https://github.com/falcosecurity/community. /lifecycle stale |
/remove-lifecycle stale |
Motivation
We already support multiple containerd socket paths in Falco 👇
https://github.com/falcosecurity/libs/blob/34371e18cafa429107cf5d600bad5fd0c44718ab/userspace/libsinsp/container_engine/cri.cpp#L64-L68
This is not currently supported by the Falco chart.
This would particularly enable k3s by default but also allow users to add multiple paths in case their use case requires that.
Feature
Make the
containerd.socket
accept an array of paths. For example:charts/charts/falco/values.yaml
Lines 266 to 270 in 8653678
👇
Alternatives
The alternative would be to add a config node for each specific case (e.g., k3s). However, this is overkill and will not allow flexibility.
Additional context
The text was updated successfully, but these errors were encountered: