Skip to content

Commit

Permalink
tetragon: Switch off kprobe multi via spec in security_ test
Browse files Browse the repository at this point in the history
We can now localy disable kprobe multi, let's use it for
security_ override test, which won't work with kprobe multi.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
  • Loading branch information
olsajiri committed Oct 24, 2023
1 parent bf34790 commit 33a8706
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions pkg/sensors/tracing/kprobe_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2212,10 +2212,6 @@ func TestKprobeOverrideSecurity(t *testing.T) {
t.Skip("skipping fmod_ret support is not available")
}

if !option.Config.DisableKprobeMulti && bpf.HasKprobeMulti() {
t.Skip("skipping fmod_ret does not work with kprobe multi")
}

pidStr := strconv.Itoa(int(observertesthelper.GetMyPid()))

file, err := os.CreateTemp(t.TempDir(), "kprobe-override-")
Expand All @@ -2230,6 +2226,9 @@ kind: TracingPolicy
metadata:
name: "sys-openat-override"
spec:
options:
- name: "disable-kprobe-multi"
value: "1"
kprobes:
- call: "security_file_open"
syscall: false
Expand Down

0 comments on commit 33a8706

Please sign in to comment.