-
Notifications
You must be signed in to change notification settings - Fork 869
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
Need to map HTTP and gRPC status codes to Span.setStatus() #326
Comments
Seems like a good chance to do an audit of those mappings and make sure they make sense, as well. Not sure if anyone has actually used them yet. ;) |
If this task is about implementing the straightforward mapping described in spec doc, then I can take it. I don't quite understand what did @jkwatson mean by |
all yours 😄 |
I was just suggesting that if we run across a status that doesn't map well, that would be important feedback for the specs. |
Hoping we can close this soon if span status goes away open-telemetry/oteps#123 (comment) |
Technically we could probably close anyways I added that mapping a while ago I think. |
Then closing! |
See
https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/semantic_conventions/http.md#status
and
https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/semantic_conventions/rpc.md#status
The text was updated successfully, but these errors were encountered: