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 118011d commit ae43818
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 ae43818

Please sign in to comment.