Skip to content

Commit 1b5e49e

Browse files
committed
Lint fix + Unit test fix
1 parent b49118e commit 1b5e49e

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

aws-opentelemetry-distro/src/amazon/opentelemetry/distro/aws_opentelemetry_distro.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,13 @@
1919
OTEL_TRACES_SAMPLER,
2020
)
2121
from amazon.opentelemetry.distro.patches._instrumentation_patch import apply_instrumentation_patches
22+
from opentelemetry import propagate
2223
from opentelemetry.distro import OpenTelemetryDistro
2324
from opentelemetry.environment_variables import OTEL_PROPAGATORS, OTEL_PYTHON_ID_GENERATOR
2425
from opentelemetry.sdk.environment_variables import (
2526
OTEL_EXPORTER_OTLP_METRICS_DEFAULT_HISTOGRAM_AGGREGATION,
2627
OTEL_EXPORTER_OTLP_PROTOCOL,
2728
)
28-
from opentelemetry import propagate
2929

3030
_logger: Logger = getLogger(__name__)
3131

aws-opentelemetry-distro/tests/amazon/opentelemetry/distro/test_aws_opentelementry_configurator.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1262,7 +1262,7 @@ def validate_distro_environ():
12621262
tc.assertEqual(
12631263
"base2_exponential_bucket_histogram", os.environ.get("OTEL_EXPORTER_OTLP_METRICS_DEFAULT_HISTOGRAM_AGGREGATION")
12641264
)
1265-
tc.assertEqual("xray,tracecontext,b3,b3multi", os.environ.get("OTEL_PROPAGATORS"))
1265+
tc.assertEqual("tracecontext,baggage,xray", os.environ.get("OTEL_PROPAGATORS"))
12661266
tc.assertEqual("xray", os.environ.get("OTEL_PYTHON_ID_GENERATOR"))
12671267

12681268
# Not set

aws-opentelemetry-distro/tests/amazon/opentelemetry/distro/test_aws_opentelemetry_distro.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
22
# SPDX-License-Identifier: Apache-2.0
3-
import os
43
import importlib
4+
import os
55
import sys
66
from importlib.metadata import PackageNotFoundError, version
77
from unittest import TestCase
@@ -102,7 +102,7 @@ def test_configure_without_patches(self, mock_super_configure, mock_apply_patche
102102
@patch("amazon.opentelemetry.distro.aws_opentelemetry_distro.apply_instrumentation_patches")
103103
@patch("amazon.opentelemetry.distro.aws_opentelemetry_distro.OpenTelemetryDistro._configure")
104104
def test_configure_with_agent_observability_enabled(
105-
self, mock_super_configure, mock_apply_patches, mock_is_agent_observability, mock_get_aws_region
105+
self, mock_super_configure, mock_apply_patches, mock_is_agent_observability, mock_get_aws_region
106106
):
107107
"""Test that _configure sets agent observability defaults when enabled"""
108108
mock_is_agent_observability.return_value = True
@@ -241,6 +241,7 @@ def test_user_defined_propagators(self):
241241
# Force the reload of the propagate module otherwise the above environment
242242
# variable doesn't taker effect.
243243
from opentelemetry import propagate
244+
244245
importlib.reload(propagate)
245246

246247
distro = AwsOpenTelemetryDistro()
@@ -263,6 +264,7 @@ def test_otel_propagators_added_when_not_user_defined(self):
263264

264265
# Verify that the propagators are set correctly by ADOT
265266
from opentelemetry import propagate
267+
266268
propagators = propagate.get_global_textmap()
267269

268270
self.assertTrue(isinstance(propagators, CompositePropagator))

0 commit comments

Comments
 (0)