Add capture the fully qualified type name for raised exceptions in spans

opentelemetry-sdk/src/opentelemetry/sdk/trace/__init__.py

@@ -995,8 +995,18 @@ def record_exception(
                 type(exception), value=exception, tb=exception.__traceback__
             )
         )
+        module = (
+            exception.__module__
+            if type(exception).__module__ != "builtins"
+            else ""
+        )
+        qualname = type(exception).__qualname__
+        exc_type = (
+            f"{module}.{qualname}"
+            if module else qualname
+        )
         _attributes: MutableMapping[str, types.AttributeValue] = {
-            "exception.type": exception.__class__.__name__,
+            "exception.type": exc_type,
             "exception.message": str(exception),
             "exception.stacktrace": stacktrace,
             "exception.escaped": str(escaped),

opentelemetry-sdk/tests/trace/test_trace.py

@@ -636,6 +636,10 @@ def test_events(self):
         self.assertEqual(span.events, tuple(events))
 
 
+class DummyError(Exception):
+    pass
+
+
 class TestSpan(unittest.TestCase):
     # pylint: disable=too-many-public-methods
 
@@ -1144,6 +1148,25 @@ def error_status_test(context):
             .start_as_current_span("root")
         )
 
+    def test_record_exception_fqn(self):
+        span = trace._Span("name", mock.Mock(spec=trace_api.SpanContext))
+        exception = DummyError("error")
+        exception_type = f"{exception.__module__}.{type(exception).__qualname__}"
+        span.record_exception(exception)
+        exception_event = span.events[0]
+        self.assertEqual("exception", exception_event.name)
+        self.assertEqual(
+            "error", exception_event.attributes["exception.message"]
+        )
+        self.assertEqual(
+            exception_type,
+            exception_event.attributes["exception.type"],
+        )
+        self.assertIn(
+            "DummyError: error",
+            exception_event.attributes["exception.stacktrace"],
+        )
+
     def test_record_exception(self):
         span = trace._Span("name", mock.Mock(spec=trace_api.SpanContext))
         try: