Skip to content
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 Exception handling in Agenta Observability #1847

Open
jp-agenta opened this issue Jul 5, 2024 · 1 comment
Open

Add Exception handling in Agenta Observability #1847

jp-agenta opened this issue Jul 5, 2024 · 1 comment

Comments

@jp-agenta
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Exceptions are currently not handled as per OpenTelemetry specifications.

Describe the solution you'd like
This feature would require changes in the SDK (for instance self.tracing.add_event() and self.tracing.record_exception(), at least, if the solution uses Events as per OTel specs), as well as changes in the UI to handle events (again, if the solution uses Events). Also, Exceptions would not be part of the trace outputsanymore.

Describe alternatives you've considered

  1. Implementing Events (proper solution but with more moving parts)
  2. Adding exceptions as another span attribute (non-standard solution but with fewer moving parts)

Additional context
https://github.com/open-telemetry/opentelemetry-specification/blob/v1.33.0/specification/trace/api.md
https://github.com/open-telemetry/opentelemetry-specification/blob/v1.33.0/specification/trace/exceptions.md
https://opentelemetry.io/docs/specs/semconv/exceptions/exceptions-spans/
https://opentelemetry.io/docs/concepts/signals/traces/

@jp-agenta
Copy link
Contributor Author

Related to AGE-389

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant