-
Notifications
You must be signed in to change notification settings - Fork 73
feat(ebpf): move the core ebpf code from agent repo #784
Conversation
ebpf/go.mod
Outdated
@@ -0,0 +1,23 @@ | |||
module github.com/grafana/phlare/ebpf | |||
|
|||
go 1.20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
go 1.20 | |
go 1.19 |
To match the other modules in the repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ugh unfortunately a piece of my code does not compile
type Resource interface {
comparable
Refresh()
Cleanup()
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will try to workaround
type TargetsOptions struct { | ||
Targets []DiscoveryTarget | ||
TargetsOnly bool | ||
DefaultTarget DiscoveryTarget |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Following up on grafana/agent#3953 (comment), +1 for keeping this while there is not other way to profile the whole system. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 for keeping it undocumented until we have other options, let's see what others think
6e36ade
to
57fce5b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ed, do not append empty stacktrace to pprof
Great work! |
grafana/agent#3953