Update README for Alpha#1282
Conversation
| The agent loads the eBPF program and its maps, starts unwinding and reports | ||
| captured traces to the backend. | ||
|
|
||
| ## Open Source Backends |
There was a problem hiding this comment.
The discussion around the backend should not be mixed with the Update around the promotion of the signal to Alpha. Can the be part of a separate PR?
There was a problem hiding this comment.
Isn't it faster to do both in one PR (they're semantically linked together as devfiler is part of the "getting started story" we want to tell) ?
There was a problem hiding this comment.
As the focus of OTel is not about the backend and visualization, I think it should not get mixed.
If something should be named to get started, then more open source options should be named.
There was a problem hiding this comment.
We are mentioning Devfiler as part of the upcoming blog post so I think it's only natural to also mention it here. I named the section "Open Source Backends" so that we can also add additional backends here (folks can open PRs to make that happen).
We've also previously discussed this (e.g. see here and here).
There was a problem hiding this comment.
This PR updates various aspects of the Readme. I don't see any problem combining all these as it's just updating docs. Maybe rename the PR title to just 'Update PR' if it helps?
This was already discussed in an issue that this actually fixes. See: #514
There was a problem hiding this comment.
Similar to #514 my request is to list more alternatives.
But I will approve the PR as I want to see the changes around the promotion to Alpha happening.
There was a problem hiding this comment.
+1 to list more alternatives
There was a problem hiding this comment.
Feel free to open a PR. As long as the backend is open source and supports the Alpha protocol we can list it.
There was a problem hiding this comment.
Feel free to open a PR. As long as the backend is open source and supports the Alpha protocol we can list it.
fabled
left a comment
fabled
left a comment
There was a problem hiding this comment.
Lgtm. Needs update for dotnet not supported on arm64. Pre approving though. Thanks!
| The agent loads the eBPF program and its maps, starts unwinding and reports | ||
| captured traces to the backend. | ||
|
|
||
| ## Open Source Backends |
There was a problem hiding this comment.
This PR updates various aspects of the Readme. I don't see any problem combining all these as it's just updating docs. Maybe rename the PR title to just 'Update PR' if it helps?
This was already discussed in an issue that this actually fixes. See: #514
Co-authored-by: Timo Teräs <timo.teras@iki.fi>
5 participants
Summary
otelcol-ebpf-profilerinstructions