Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Common Expression Language filter #3098

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Add Common Expression Language filter #3098

merged 2 commits into from
Nov 12, 2024

Conversation

michi-covalent
Copy link
Contributor

Add Common Expression Language 1 filter in GetEventRequest. See unit tests for sample usages. This is largely based on Hubble CEL expression filter 2.

Footnotes

  1. https://cel.dev/

  2. https://github.com/cilium/cilium/pull/32147

Add two helper functions in protoc-gen-go-tetragon:

- ResponseTypeMap() returns a map from event field names
  (e.g. "process_exec") to corresponding protobuf messages
  (e.g. &tetragon.ProcessExec{}).
- ProcessEventMap() returns a map from event field names
  (e.g. "process_exec") to corresponding
  protobuf messages in a given tetragon.GetEventsResponse
  (e.g. response.GetProcessExec()).

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Add Common Expression Language [^1] filter in GetEventRequest. See unit
tests for sample usages. This is largely based on Hubble CEL expression
filter [^2].

[^1]: https://cel.dev/
[^2]: cilium/cilium#32147

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@michi-covalent michi-covalent added the release-note/minor This PR introduces a minor user-visible change label Nov 10, 2024
Copy link

netlify bot commented Nov 10, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 017697a
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/67304c1129aec4000832a1e3
😎 Deploy Preview https://deploy-preview-3098--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@michi-covalent michi-covalent marked this pull request as ready for review November 10, 2024 06:53
@michi-covalent michi-covalent requested a review from a team as a code owner November 10, 2024 06:53
Copy link
Contributor

@willfindlay willfindlay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@michi-covalent michi-covalent merged commit 05e4231 into main Nov 12, 2024
51 checks passed
@michi-covalent michi-covalent deleted the pr/michi/cel branch November 12, 2024 15:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
release-note/minor This PR introduces a minor user-visible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants