Unify top-level ecto transactions and ecto spans

Author: solnic

lib/sentry/opentelemetry/span_processor.ex

@@ -53,7 +53,7 @@ defmodule Sentry.Opentelemetry.SpanProcessor do
       start_timestamp: root_span.start_time,
       timestamp: root_span.end_time,
       transaction_info: %{
-        source: "db"
+        source: "component"
       },
       contexts: %{
         trace: build_trace_context(root_span)
@@ -156,6 +156,7 @@ defmodule Sentry.Opentelemetry.SpanProcessor do
       span_id: root_span.span_id,
       parent_span_id: root_span.parent_span_id,
       op: "db.#{attributes["db.type"]}.ecto",
+      description: attributes["db.statement"] || root_span.name,
       origin: root_span.origin,
       data: attributes
     }

test_integrations/phoenix_app/test/phoenix_app/repo_test.ex renamed to test_integrations/phoenix_app/test/phoenix_app/repo_test.exs

@@ -1,7 +1,7 @@
 defmodule PhoenixApp.RepoTest do
   use PhoenixApp.DataCase
 
-  alias PhoenixApp.{Repo, User}
+  alias PhoenixApp.{Repo, Accounts.User}
 
   import Sentry.TestHelpers
 
@@ -20,8 +20,9 @@ defmodule PhoenixApp.RepoTest do
 
     assert [transaction] = transactions
 
-    assert transaction.transaction_info == %{source: "db"}
+    assert transaction.transaction_info == %{source: "component"}
     assert transaction.contexts.trace.op == "db.sql.ecto"
+    assert String.starts_with?(transaction.contexts.trace.description, "SELECT")
     assert transaction.contexts.trace.data["db.system"] == :sqlite
   end
 end