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

tetragon: assorted fixes #3323

Merged
merged 6 commits into from
Jan 24, 2025
Merged

tetragon: assorted fixes #3323

merged 6 commits into from
Jan 24, 2025

Conversation

olsajiri
Copy link
Contributor

@olsajiri olsajiri commented Jan 21, 2025

gathered multiple assorted fixes, please check commits

Copy link

netlify bot commented Jan 21, 2025

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 53f1f90
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/67921a6cae29d00008e288a4
😎 Deploy Preview https://deploy-preview-3323--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.

@olsajiri olsajiri added the release-note/minor This PR introduces a minor user-visible change label Jan 21, 2025
@olsajiri olsajiri changed the title Pr/olsajiri/fixes tetragon: assorted fixes Jan 23, 2025
Moving override action code in do_override_action so we can
define it only for kprobe and lsm program (GENERIC_KPROBE and
GENERIC_LSM) where it belongs. Now it's also part of kretprobe.

Signed-off-by: Jiri Olsa <[email protected]>
Moving SizeWithSuffix to strutils package, so we can
use it from other places.

Signed-off-by: Jiri Olsa <[email protected]>
It's possible to omit programs when we create map in which
case we explode. Adding check for that.

Signed-off-by: Jiri Olsa <[email protected]>
Adding TAIL_CALL_SETUP tail call index, which was missing
in the TAIL_CALL_* enum values.

Signed-off-by: Jiri Olsa <[email protected]>
Switch 'enter' check to '!enter' and decrease indent of the
following code, which makes the function more readable and
will ease up following changes.

There's no functional change.

Signed-off-by: Jiri Olsa <[email protected]>
Passing execve_map_value directly to match_binaries to
eliminate superfluous event_find_curr in it.

Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri marked this pull request as ready for review January 23, 2025 12:25
@olsajiri olsajiri requested a review from a team as a code owner January 23, 2025 12:25
@olsajiri olsajiri requested review from tixxdz and kkourt January 23, 2025 12:25
@olsajiri olsajiri merged commit 6ed124a into main Jan 24, 2025
44 checks passed
@olsajiri olsajiri deleted the pr/olsajiri/fixes branch January 24, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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