-
Notifications
You must be signed in to change notification settings - Fork 376
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
tetra: Add CEL filter to the CLI #3124
base: main
Are you sure you want to change the base?
Conversation
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.
Hi,
Thanks for the PR!
It seems to me that this PR attempts to add support for CEL filters in the export filters rather than the CLI (tetra
). Could you please add more details about what the PR attempts to do? An example of how it works in the commit message would be very helpful in that regard.
Sorry, my mistake. I picked this up at the Contribfest, found that these filters were missing the CEL integration and submitted the fix. I'll tidy this up, and fix the CLI issue, some time over the next few days. In the meantime I've converted this to a draft |
192ee0f
to
4fab7fe
Compare
✅ Deploy Preview for tetragon ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@kkourt I've updated the PR to also include the CLI integration, the |
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.
Thanks!
Can you please add an example usage in the commit message?
Also, can you put the "Fixes: " in the PR message" rather than the commit title?
4fab7fe
to
e96ffe8
Compare
Add the ability to filter events using CEL expressions, for example: # tetra getevents --cel-expression "process_exec.process.binary == '/bin/sh'" Signed-off-by: Kevin Conner <[email protected]>
e96ffe8
to
4d81dac
Compare
Commit message should now be updated |
Fixes #3112
Description
Integrate the existing CEL filtering capability into the CLI
Changelog