Fix lint: remove unused import and apply ruff format

Author: ocelotl

opentelemetry-configuration/src/opentelemetry/configuration/_logger_provider.py

@@ -297,20 +297,14 @@ def _create_log_record_limits(
 
     attribute_value_length_limit = config.attribute_value_length_limit
     if attribute_value_length_limit is None and global_limits is not None:
-        attribute_value_length_limit = (
-            global_limits.attribute_value_length_limit
-        )
+        attribute_value_length_limit = global_limits.attribute_value_length_limit
 
     return LogRecordLimits(
         max_attributes=(
-            attribute_count_limit
-            if attribute_count_limit is not None
-            else _DEFAULT_OTEL_LOG_ATTRIBUTE_COUNT_LIMIT
+            attribute_count_limit if attribute_count_limit is not None else _DEFAULT_OTEL_LOG_ATTRIBUTE_COUNT_LIMIT
         ),
         max_attribute_length=(
-            attribute_value_length_limit
-            if attribute_value_length_limit is not None
-            else LogRecordLimits.UNSET
+            attribute_value_length_limit if attribute_value_length_limit is not None else LogRecordLimits.UNSET
         ),
     )
 
@@ -341,16 +335,12 @@ def create_logger_provider(
     )
 
     if config is not None and config.limits is not None:
-
         limits = config.limits
 
     else:
-
         limits = LogRecordLimitsConfig()
 
-    log_record_limits = _create_log_record_limits(
-        limits, global_attribute_limits
-    )
+    log_record_limits = _create_log_record_limits(limits, global_attribute_limits)
 
     provider = LoggerProvider(
         resource=resource,
@@ -385,6 +375,4 @@ def configure_logger_provider(
     """
     if config is None:
         return
-    set_logger_provider(
-        create_logger_provider(config, resource, global_attribute_limits)
-    )
+    set_logger_provider(create_logger_provider(config, resource, global_attribute_limits))

opentelemetry-configuration/src/opentelemetry/configuration/_sdk.py

@@ -27,7 +27,6 @@
     configure_instrumentation,
 )
 from opentelemetry.configuration.models import (
-    AttributeLimits,
     OpenTelemetryConfiguration,
     SeverityNumber,
 )
@@ -102,12 +101,8 @@ def configure_sdk(config: OpenTelemetryConfiguration) -> None:
 
     global_attribute_limits = config.attribute_limits
     resource = create_resource(config.resource)
-    configure_tracer_provider(
-        config.tracer_provider, resource, global_attribute_limits
-    )
+    configure_tracer_provider(config.tracer_provider, resource, global_attribute_limits)
     configure_meter_provider(config.meter_provider, resource)
-    configure_logger_provider(
-        config.logger_provider, resource, global_attribute_limits
-    )
+    configure_logger_provider(config.logger_provider, resource, global_attribute_limits)
     configure_propagator(config.propagator)
     configure_instrumentation(config.instrumentation_development)

opentelemetry-configuration/src/opentelemetry/configuration/_tracer_provider.py

@@ -385,15 +385,11 @@ def _create_span_limits(
 
     attribute_value_length_limit = config.attribute_value_length_limit
     if attribute_value_length_limit is None and global_limits is not None:
-        attribute_value_length_limit = (
-            global_limits.attribute_value_length_limit
-        )
+        attribute_value_length_limit = global_limits.attribute_value_length_limit
 
     return SpanLimits(
         max_span_attributes=(
-            attribute_count_limit
-            if attribute_count_limit is not None
-            else _DEFAULT_OTEL_SPAN_ATTRIBUTE_COUNT_LIMIT
+            attribute_count_limit if attribute_count_limit is not None else _DEFAULT_OTEL_SPAN_ATTRIBUTE_COUNT_LIMIT
         ),
         max_events=(
             config.event_count_limit if config.event_count_limit is not None else _DEFAULT_OTEL_SPAN_EVENT_COUNT_LIMIT
@@ -412,14 +408,10 @@ def _create_span_limits(
             else _DEFAULT_OTEL_LINK_ATTRIBUTE_COUNT_LIMIT
         ),
         max_attribute_length=(
-            attribute_value_length_limit
-            if attribute_value_length_limit is not None
-            else SpanLimits.UNSET
+            attribute_value_length_limit if attribute_value_length_limit is not None else SpanLimits.UNSET
         ),
         max_span_attribute_length=(
-            attribute_value_length_limit
-            if attribute_value_length_limit is not None
-            else SpanLimits.UNSET
+            attribute_value_length_limit if attribute_value_length_limit is not None else SpanLimits.UNSET
         ),
     )
 
@@ -482,16 +474,12 @@ def create_tracer_provider(
         _create_id_generator(config.id_generator) if config is not None and config.id_generator is not None else None
     )
     if config is not None and config.limits is not None:
-
         limits = config.limits
 
     else:
-
         limits = SpanLimitsConfig()
 
-    span_limits = _create_span_limits(
-        limits, global_attribute_limits
-    )
+    span_limits = _create_span_limits(limits, global_attribute_limits)
 
     tracer_configurator = (
         _create_tracer_configurator(config.tracer_configurator_development)
@@ -533,6 +521,4 @@ def configure_tracer_provider(
     """
     if config is None:
         return
-    trace.set_tracer_provider(
-        create_tracer_provider(config, resource, global_attribute_limits)
-    )
+    trace.set_tracer_provider(create_tracer_provider(config, resource, global_attribute_limits))

opentelemetry-configuration/tests/test_logger_provider.py

@@ -26,6 +26,7 @@
 from opentelemetry.configuration.models import (
     AttributeLimits,
     NameStringValuePair,
+    SeverityNumber,
 )
 from opentelemetry.configuration.models import (
     BatchLogRecordProcessor as BatchLogRecordProcessorConfig,
@@ -54,10 +55,6 @@
 from opentelemetry.configuration.models import (
     LogRecordProcessor as LogRecordProcessorConfig,
 )
-from opentelemetry.configuration.models import (
-    NameStringValuePair,
-    SeverityNumber,
-)
 from opentelemetry.configuration.models import (
     OtlpGrpcExporter as OtlpGrpcExporterConfig,
 )
@@ -471,34 +468,26 @@ def test_no_limits_no_warning():
 
     def test_global_attribute_count_limit_used_when_no_per_signal_limits(self):
         global_limits = AttributeLimits(attribute_count_limit=42)
-        provider = create_logger_provider(
-            None, global_attribute_limits=global_limits
-        )
+        provider = create_logger_provider(None, global_attribute_limits=global_limits)
         self.assertEqual(provider._log_record_limits.max_attributes, 42)
 
     def test_global_attribute_value_length_limit_used_when_no_per_signal_limits(
         self,
     ):
         global_limits = AttributeLimits(attribute_value_length_limit=64)
-        provider = create_logger_provider(
-            None, global_attribute_limits=global_limits
-        )
+        provider = create_logger_provider(None, global_attribute_limits=global_limits)
         self.assertEqual(provider._log_record_limits.max_attribute_length, 64)
 
     def test_per_signal_limits_override_global(self):
-        global_limits = AttributeLimits(
-            attribute_count_limit=100, attribute_value_length_limit=200
-        )
+        global_limits = AttributeLimits(attribute_count_limit=100, attribute_value_length_limit=200)
         config = LoggerProviderConfig(
             processors=[],
             limits=LogRecordLimitsConfig(
                 attribute_count_limit=7,
                 attribute_value_length_limit=16,
             ),
         )
-        provider = create_logger_provider(
-            config, global_attribute_limits=global_limits
-        )
+        provider = create_logger_provider(config, global_attribute_limits=global_limits)
         self.assertEqual(provider._log_record_limits.max_attributes, 7)
         self.assertEqual(provider._log_record_limits.max_attribute_length, 16)
 

opentelemetry-configuration/tests/test_sdk.py

@@ -70,9 +70,7 @@ def test_calls_each_signal_with_resource(
         configure_sdk(config)
 
         mock_create_resource.assert_called_once_with(resource_cfg)
-        mock_tracer.assert_called_once_with(
-            tracer_cfg, sentinel_resource, None
-        )
+        mock_tracer.assert_called_once_with(tracer_cfg, sentinel_resource, None)
         mock_meter.assert_called_once_with(None, sentinel_resource)
         mock_logger.assert_called_once_with(None, sentinel_resource, None)
         mock_propagator.assert_called_once_with(propagator_cfg)