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

tetragon: assorted fixes #2926

Merged
merged 5 commits into from
Sep 16, 2024
Merged

tetragon: assorted fixes #2926

merged 5 commits into from
Sep 16, 2024

Conversation

olsajiri
Copy link
Contributor

map and cgrouprate related fixes

We did not set the cgroup_rate_map properly for test, so tests might
occasionally fail if first recorded cgroup wasn't test cgroup.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
So we do not get enforcer map created when it's not needed.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Each policy can have just one enforcer, so it's ok to use
just policy name as key to global map of enforcers.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
We have the hierarchy in sysfs tree, so we don't nede the 'enforcer'
prefix for kprobe program.

Also adding 'fmod_ret_' prefix for fmod_ret programs, like:

./kill-syscalls
./kill-syscalls/__enforcer__
./kill-syscalls/__enforcer__/fmod_ret___ia32_sys_dup
./kill-syscalls/__enforcer__/fmod_ret___ia32_sys_dup/prog
./kill-syscalls/__enforcer__/fmod_ret___x64_sys_dup2
./kill-syscalls/__enforcer__/fmod_ret___x64_sys_dup2/prog
./kill-syscalls/__enforcer__/fmod_ret___x64_sys_dup
./kill-syscalls/__enforcer__/fmod_ret___x64_sys_dup/prog

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
It's no longer needed, because all sensors are now placed
in policy directory.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
@olsajiri olsajiri added the release-note/minor This PR introduces a minor user-visible change label Sep 14, 2024
Copy link

netlify bot commented Sep 14, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit fcd79d5
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/66e5e076d21c940008e97591
😎 Deploy Preview https://deploy-preview-2926--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 marked this pull request as ready for review September 14, 2024 20:41
@olsajiri olsajiri requested a review from a team as a code owner September 14, 2024 20:41
@olsajiri olsajiri requested a review from jrfastab September 14, 2024 20:41
@kkourt kkourt merged commit 7fa6b84 into main Sep 16, 2024
45 of 46 checks passed
@kkourt kkourt deleted the pr/olsajiri/fixes branch September 16, 2024 05:48
# 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