[WIP] Deprecate exception.escaped
attribute, update exception example
#4368
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is WIP awaiting open-telemetry/semantic-conventions#1716 to be merged and released
Changes
Follow up for open-telemetry/semantic-conventions#1716
exception.escaped
attribute is not used in practice within OTel instrumentations (checked Java, .NET, JS. Python, Go, C++ and Rust including contrib repos).OTel instrumentations, in general, don't record handled exceptions, i.e. virtually all exceptions we're recording in non-native instrumentations are expected to escape span scope.
The litmus test whether to record exception on spans is whether the instrumentation is going to set span status to
ERROR
due to this exception.A way to record all sorts of exceptions is being introduced in #4333 and leverages logs along with severity that conveys how critical the exception is in more expressive way than
exception.escaped
flag.exception.escaped
? semantic-conventions#1516CHANGELOG.md
file updated for non-trivial changesspec-compliance-matrix.md
updated if necessary