tetragon: Warn for --release-pinned-bpf and --keep-sensors-on-exit

The effect of having both --release-pinned-bpf and --keep-sensors-on-exit options enabled is that the previous sysfs instance will be removed early before the new config is set. Not a big problem, but better to warn.. Signed-off-by: Jiri Olsa <[email protected]>
cilium · Aug 13, 2024 · dfb4de0 · dfb4de0
1 parent b8098c3
commit dfb4de0
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/cmd/tetragon/main.go b/cmd/tetragon/main.go
@@ -256,6 +256,12 @@ func tetragonExecute() error {
 	}
 
 	if option.Config.KeepSensorsOnExit {
+		// The effect of having both --release-pinned-bpf and --keep-sensors-on-exit options
+		// enabled is that the previous sysfs instance will be removed early before the new
+		// config is set. Not a big problem, but better to warn..
+		if option.Config.ReleasePinned {
+			log.Warn("Options --release-pinned-bpf and --keep-sensors-on-exit enabled together, we will remove sysfs instance early.")
+		}
 		log.Info("Not unloading sensors on exit")
 	}