potiuk commented on code in PR #56187: URL: https://github.com/apache/airflow/pull/56187#discussion_r2569453116
########## shared/observability/tests/observability/traces/test_otel_tracer.py: ########## @@ -25,11 +25,12 @@ from opentelemetry.sdk import util from opentelemetry.sdk.trace.export.in_memory_span_exporter import InMemorySpanExporter +from airflow._shared.observability.traces.base_tracer import EmptyTrace +from airflow._shared.observability.traces.otel_tracer import OtelTrace +from airflow._shared.observability.traces.utils import datetime_to_nano from airflow.configuration import conf Review Comment: I think that one should also be `from.airflow_shared.configuration`. This is test code and it should have access to all the `arflow._shared` folders ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
