Skip to content

Commit 03ad66a

Browse files
committed
Fix ruff
1 parent 80f8a78 commit 03ad66a

File tree

2 files changed

+22
-8
lines changed

2 files changed

+22
-8
lines changed

opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/__init__.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -869,7 +869,9 @@ def emit(
869869
if self._config.disabled:
870870
return
871871

872-
if is_less_than_min_severity(record, self._config.minimum_severity_level):
872+
if is_less_than_min_severity(
873+
record, self._config.minimum_severity_level
874+
):
873875
return
874876

875877
if should_drop_logs_for_unsampled_traces(

opentelemetry-sdk/tests/logs/test_logs.py

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,8 @@ def test_emit_logrecord_trace_filtering_edge_cases(self):
348348
def test_emit_both_min_severity_and_trace_based_sampling_filtering(self):
349349
"""Test that both min severity and trace-based filtering work together"""
350350
config = LoggerConfig(
351-
minimum_severity_level=SeverityNumber.WARN, trace_based_sampling=True
351+
minimum_severity_level=SeverityNumber.WARN,
352+
trace_based_sampling=True,
352353
)
353354
logger, log_record_processor_mock = self._get_logger(config)
354355

@@ -414,7 +415,9 @@ def test_logger_config_property(self):
414415
logger, _ = self._get_logger(config)
415416

416417
self.assertEqual(logger.config.disabled, True)
417-
self.assertEqual(logger.config.minimum_severity_level, SeverityNumber.WARN)
418+
self.assertEqual(
419+
logger.config.minimum_severity_level, SeverityNumber.WARN
420+
)
418421
self.assertEqual(logger.config.trace_based_sampling, True)
419422

420423
def test_logger_configurator_behavior(self):
@@ -447,7 +450,8 @@ def test_logger_configurator_behavior(self):
447450

448451
other_logger = provider.get_logger("test.other")
449452
self.assertEqual(
450-
other_logger.config.minimum_severity_level, SeverityNumber.UNSPECIFIED
453+
other_logger.config.minimum_severity_level,
454+
SeverityNumber.UNSPECIFIED,
451455
)
452456
self.assertFalse(other_logger.config.disabled)
453457
self.assertFalse(other_logger.config.trace_based_sampling)
@@ -486,7 +490,9 @@ def test_logger_configurator_pattern_matching(self):
486490
def test_logger_configurator_dynamic_updates(self):
487491
"""Test that LoggerConfigurator updates apply to existing loggers"""
488492
initial_configs = {
489-
"test.module": LoggerConfig(minimum_severity_level=SeverityNumber.INFO)
493+
"test.module": LoggerConfig(
494+
minimum_severity_level=SeverityNumber.INFO
495+
)
490496
}
491497

492498
initial_configurator = create_logger_configurator_by_name(
@@ -496,7 +502,9 @@ def test_logger_configurator_dynamic_updates(self):
496502
provider = LoggerProvider(logger_configurator=initial_configurator)
497503

498504
logger = provider.get_logger("test.module")
499-
self.assertEqual(logger.config.minimum_severity_level, SeverityNumber.INFO)
505+
self.assertEqual(
506+
logger.config.minimum_severity_level, SeverityNumber.INFO
507+
)
500508
self.assertFalse(logger.config.disabled)
501509

502510
updated_configs = {
@@ -510,7 +518,9 @@ def test_logger_configurator_dynamic_updates(self):
510518

511519
provider.set_logger_configurator(updated_configurator)
512520

513-
self.assertEqual(logger.config.minimum_severity_level, SeverityNumber.ERROR)
521+
self.assertEqual(
522+
logger.config.minimum_severity_level, SeverityNumber.ERROR
523+
)
514524
self.assertTrue(logger.config.disabled)
515525

516526
new_logger = provider.get_logger("test.module")
@@ -533,7 +543,9 @@ def none_configurator(scope):
533543

534544
logger = provider.get_logger("test.module")
535545

536-
self.assertEqual(logger.config.minimum_severity_level, SeverityNumber.WARN)
546+
self.assertEqual(
547+
logger.config.minimum_severity_level, SeverityNumber.WARN
548+
)
537549
self.assertTrue(logger.config.trace_based_sampling)
538550
self.assertFalse(logger.config.disabled)
539551

0 commit comments

Comments
 (0)