-
Notifications
You must be signed in to change notification settings - Fork 170
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
feat: Use Default Span Name Format #1039
feat: Use Default Span Name Format #1039
Conversation
It also allows users to rollback to legacy span name format. Follow up to open-telemetry#1014
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.
Looks good as-is! One small documentation suggestion.
instrumentation/action_view/lib/opentelemetry/instrumentation/action_view/instrumentation.rb
Outdated
Show resolved
Hide resolved
Co-authored-by: Kayla Reopelle (she/her) <[email protected]>
👋 This pull request has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the |
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.
Thank you!
This reverts commit f999e70.
This reverts commit f999e70.
This restores the functionality that was reverted 94c3c3b. I had originally introduced this functionality in this PR: open-telemetry#1039 cc: @mrsimo
It also allows users to rollback to legacy span name format.
Follow up to #1014