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: map changes #3328

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

tetragon: map changes #3328

wants to merge 6 commits into from

Conversation

olsajiri
Copy link
Contributor

@olsajiri olsajiri commented Jan 23, 2025

Adding execve-map-entries option to setup entries of execve_map map, plus related loader changes.

Copy link

netlify bot commented Jan 23, 2025

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit e6455a4
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/6793c562edc3310008bca643
😎 Deploy Preview https://deploy-preview-3328--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 23, 2025
@olsajiri olsajiri force-pushed the pr/olsajiri/maps branch 2 times, most recently from e6455a4 to 96eade9 Compare January 26, 2025 21:44
pkg/sensors/base/base.go Dismissed Show dismissed Hide dismissed
@olsajiri olsajiri marked this pull request as ready for review January 28, 2025 13:21
@olsajiri olsajiri requested review from mtardy and a team as code owners January 28, 2025 13:21
Adding Maps array to loader LoadOpts object as additional source
of user maps for loader maps resolving.

This will allow program loader to use user maps pinned to another
program and will ease up maps sharing for sensors in following
changes.

At the moment for simplicity we will pass all the sensors maps,
so we filter out user maps before using the array.

Signed-off-by: Jiri Olsa <[email protected]>
Passing sensors maps to program loader so the loader
has access to all sensor maps.

Signed-off-by: Jiri Olsa <[email protected]>
Now that the loader can access all sensor maps we no longer need
to pin user maps to programs.

Also for static maps we can add new program.MapUserFrom interface
where it's enough just to pass map pointer to create its user map.

Signed-off-by: Jiri Olsa <[email protected]>
Adding owner info when printing map object, like:
  Map{Name:m1 PinPath:m1 Owner: false}

Signed-off-by: Jiri Olsa <[email protected]>
Adding support to allow to setup execve_map max entries,
so we can control the size of this map.

Signed-off-by: Jiri Olsa <[email protected]>
Adding execve-map-entries option to setup entries of execve_map map.

Signed-off-by: Jiri Olsa <[email protected]>
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.

1 participant