From acbe0ce22dbb0d42b06b239c85719a7647da5cce Mon Sep 17 00:00:00 2001 From: Yordis Prieto Date: Thu, 26 Feb 2026 23:14:15 -0500 Subject: [PATCH] fix: enable event store OpenTelemetry tracing by default The event_store option in Commanded.OpenTelemetry.setup/1 was the only tracing module disabled by default, inconsistent with all other modules (aggregate, aggregate_populate, application, event_handler) which default to enabled. Signed-off-by: Yordis Prieto --- lib/commanded/opentelemetry.ex | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/lib/commanded/opentelemetry.ex b/lib/commanded/opentelemetry.ex index 30d4a4ba..cdc625b7 100644 --- a/lib/commanded/opentelemetry.ex +++ b/lib/commanded/opentelemetry.ex @@ -89,9 +89,8 @@ defmodule Commanded.OpenTelemetry do ], event_store: [ type: {:in, [:disabled, []]}, - default: :disabled, - doc: - "Event store tracing configuration. Disabled by default. Use `[]` to enable." + default: [], + doc: "Event store tracing configuration. Use `:disabled` to disable." ] ) @@ -106,7 +105,7 @@ defmodule Commanded.OpenTelemetry do ## Examples - # Default setup (enables all tracing except event_store) + # Default setup (enables all tracing) Commanded.OpenTelemetry.setup() # Disable aggregate tracing @@ -121,8 +120,8 @@ defmodule Commanded.OpenTelemetry do # Disable aggregate populate tracing Commanded.OpenTelemetry.setup(aggregate_populate: :disabled) - # Enable event store tracing (disabled by default) - Commanded.OpenTelemetry.setup(event_store: []) + # Disable event store tracing + Commanded.OpenTelemetry.setup(event_store: :disabled) # Use parent-child relationships for event handlers Commanded.OpenTelemetry.setup(event_handler: [span_relationship: :child])