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

Support conditional rule loading based on sysdig capabilities #345

Closed
mstemm opened this issue Apr 3, 2018 · 0 comments · Fixed by #364
Closed

Support conditional rule loading based on sysdig capabilities #345

mstemm opened this issue Apr 3, 2018 · 0 comments · Fixed by #364
Assignees

Comments

@mstemm
Copy link
Contributor

mstemm commented Apr 3, 2018

We often have new falco rules that depend on a new event type or filtercheck. If someone is simply using the released version of falco, the rules change will always come alongside a sysdig version that supports the event type/etc in the rule.

However, some users want to pick up the latest rules changes independent from a given falco release. In this case, their falco + sysdig version might not necessarily have the new event type/filtercheck that the rule uses.

It would be nice if the rules format allowed for loading a given rule only if a given event type/filtercheck/other capability were supported by the underlying falco + sysdig software. This would allow for some flexibility without having to deal with full versioning between sysdig and the rules file.

If this ends up not being feasible, we could always add a best-effort rules loading method that simply skipped individual rules that contained condition or output fields that can't be parsed. This isn't a great solution, however, as it could mask unintended problems such as typos, etc. in the condition/output field.

@mstemm mstemm self-assigned this May 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant