Skip to content

Commit

Permalink
Test capabilities and ensure tracer is only instantiated when required
Browse files Browse the repository at this point in the history
  • Loading branch information
rafaelroquetto committed Oct 17, 2024
1 parent 896f002 commit 538f800
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
4 changes: 4 additions & 0 deletions pkg/beyla/os.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,10 @@ func checkCapabilitiesForSetOptions(config *Config, caps *helpers.OSCapabilities
if config.Enabled(FeatureAppO11y) {
testAndSet(caps, capError, unix.CAP_CHECKPOINT_RESTORE)
testAndSet(caps, capError, unix.CAP_SYS_PTRACE)

if config.EBPF.UseLinuxTC {
testAndSet(caps, capError, unix.CAP_NET_ADMIN)
}
}

if config.Enabled(FeatureNetO11y) {
Expand Down
6 changes: 5 additions & 1 deletion pkg/internal/discover/finder.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,5 +98,9 @@ func newGoTracersGroup(cfg *beyla.Config, metrics imetrics.Reporter) []ebpf.Trac
}

func newGenericTracersGroup(cfg *beyla.Config, metrics imetrics.Reporter) []ebpf.Tracer {
return []ebpf.Tracer{generictracer.New(cfg, metrics), tctracer.New(cfg)}
if cfg.EBPF.UseLinuxTC {
return []ebpf.Tracer{generictracer.New(cfg, metrics), tctracer.New(cfg)}
}

return []ebpf.Tracer{generictracer.New(cfg, metrics)}
}

0 comments on commit 538f800

Please sign in to comment.