Remove unnecessary condition on if block. There is no need to guard against a state where there is a transport layer, but it is not connected

davidigandan · davidigandan · commit 621044d87453 · 2026-03-11T14:32:43.000Z
diff --git a/src/workflows/services/common_service.py b/src/workflows/services/common_service.py
@@ -195,7 +195,7 @@ def start_transport(self):
 
             # Configure OTELTracing if configuration is available
             otel_config = getattr(self.config, "_opentelemetry", None)
-            if otel_config and self.transport.connect:
+            if otel_config:
                 # Configure OTELTracing
                 resource = Resource.create(
                     {

Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ def start_transport(self):`
`195`	`195`
`196`	`196`	`# Configure OTELTracing if configuration is available`
`197`	`197`	`otel_config = getattr(self.config, "_opentelemetry", None)`
`198`		`- if otel_config and self.transport.connect:`
	`198`	`+ if otel_config:`
`199`	`199`	`# Configure OTELTracing`
`200`	`200`	`resource = Resource.create(`
`201`	`201`	`{`