-
Notifications
You must be signed in to change notification settings - Fork 166
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
Use semconv span naming in ActionPack #961
Comments
+1 for ActionDispatch::Request#route_uri_pattern Personally I would like to keep the existing naming convention as config opt if possible, it's just such a useful railism and Action Pack is rails instrumentation, just saying 😄 |
I've heard similar from other Rails folks instrumenting with OTel. The Rails world currently thinks in the names Rails has given and not so much thinking in the OTel semantics. Maybe that will change as more people adopt OTel and polyglot shops start wondering why their Rails web request handling spans are named differently than the spans from other services in their larger system. |
Shouldn't >= 7.1 be |
@karmingc yes! |
👋 This issue 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 |
action_pack
currently overrides theserver
span name to a value that does not align with semantic conventions. Per the specification1In this case the span name is set to RubyDoc naming style, with the class name of the controller followed by an
#
then the action:opentelemetry-ruby-contrib/instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers/action_controller.rb
Line 29 in 10e766e
I propose the span name be set to either of the following values:
>= 7.1
ActionDispatch::Request#route_uri_pattern
< 7.1
#{method} /#{underscore(controller)}/#{action}
Footnotes
https://opentelemetry.io/docs/specs/semconv/http/http-spans/#name ↩
The text was updated successfully, but these errors were encountered: