-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Go Auto-Instrumentation Beta Announcement Blog Post #6079
Conversation
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.
🎉 🎉 🚀
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.
🚀
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.
thanks, some inline comments
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
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! Looks great, I just had one minor comment about the benefit of the Go Auto instrumentation that might help with clarifying the differences between eBPF based instrumentations and compile time instrumentations. One could argue that compile time instrumentations also don't require manual code changes, but they do require that the build process of the project is changed.
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.
Some copy edits. They're mostly just changing headings to sentence case (our preferred style). Thanks!
Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com>
/fix:all |
You triggered fix:all action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/13014157608 |
IMPORTANT: (RE-)RUN
|
FYI, we plan to have our release out before this is merged and published. |
/fix:all |
You triggered fix:all action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/13036930878 |
IMPORTANT: (RE-)RUN
|
The beta release has been made. This is ready to merge when approved. |
…ry#6079) Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de> Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com> Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
Add a blog post announcing the upcoming beta release of the OpenTelemetry Go auto-instrumentation.
cc @open-telemetry/go-instrumentation-approvers
Closes open-telemetry/opentelemetry-go-instrumentation#236