Skip to content

Commit

Permalink
tetragon: Switch off kprobe multi through spec in TestKprobeOverrideS…
Browse files Browse the repository at this point in the history
…ecurity

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 <[email protected]>
  • Loading branch information
olsajiri committed Oct 20, 2023
1 parent 37e26a8 commit 2db823a
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 2db823a

Please sign in to comment.