From 2b8ca97f91adce38a05728eb7f1fd18108d4afbf Mon Sep 17 00:00:00 2001 From: Tammy Baylis <96076570+tammy-baylis-swi@users.noreply.github.com> Date: Wed, 25 Feb 2026 03:25:41 -0800 Subject: [PATCH] Fix docker-tests assumption by Postgres-Sqlalchemy case about scope of metrics (#4258) * Fix postgres-sqlalchemy docker-test assumption about metrics * Changelog --- CHANGELOG.md | 2 ++ .../tests/sqlalchemy_tests/test_postgres.py | 4 +++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0e01dbe8b..9bbaae300 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -113,6 +113,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ([#4078](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/4171)) - `opentelemetry-instrumentation-aiohttp-server`: fix HTTP error inconsistencies ([#4175](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/4175)) +- `opentelemetry-docker-tests` Fix docker-tests assumption by Postgres-Sqlalchemy case about scope of metrics + ([#4258](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/4258)) ### Breaking changes diff --git a/tests/opentelemetry-docker-tests/tests/sqlalchemy_tests/test_postgres.py b/tests/opentelemetry-docker-tests/tests/sqlalchemy_tests/test_postgres.py index ff664091c..d0b659ec8 100644 --- a/tests/opentelemetry-docker-tests/tests/sqlalchemy_tests/test_postgres.py +++ b/tests/opentelemetry-docker-tests/tests/sqlalchemy_tests/test_postgres.py @@ -23,6 +23,8 @@ from opentelemetry.semconv.trace import SpanAttributes from .mixins import SQLAlchemyTestMixin +SCOPE = "opentelemetry.instrumentation.sqlalchemy" + POSTGRES_CONFIG = { "host": "127.0.0.1", "port": int(os.getenv("TEST_POSTGRES_PORT", "5432")), @@ -117,7 +119,7 @@ class PostgresMetricsTestCase(PostgresTestCase): POSTGRES_CONFIG["port"], self.SQL_DB, ) - metrics = self.get_sorted_metrics() + metrics = self.get_sorted_metrics(SCOPE) self.assertEqual(len(metrics), 1) self.assert_metric_expected( metrics[0],