From 4826539b2c6ba4be500ef6550407a17412f640d9 Mon Sep 17 00:00:00 2001 From: Jarek Potiuk Date: Tue, 19 May 2026 12:07:59 +0200 Subject: [PATCH] OTel integration: align test_export_legacy_metric_names timeout with siblings The default 60s per-test timeout in core-integration CI is tight for the heavy `dag_execution_for_testing_metrics` path (starts scheduler + apiserver, triggers a DAG, waits up to 90s for the run to finish, plus a 10s sleep for span_status processing). `test_dag_execution_succeeds` is already marked `@pytest.mark.execution_timeout(90)`; the two sibling tests sharing the same fixture were missing the marker, so they failed the moment a single parametrize variant brushed past 60s (run 26086088389, job 76701766269: 60.01s, missed by 0.01s). --- airflow-core/tests/integration/otel/test_otel.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/airflow-core/tests/integration/otel/test_otel.py b/airflow-core/tests/integration/otel/test_otel.py index 1b23fe0f7549a..4f46b37e851ed 100644 --- a/airflow-core/tests/integration/otel/test_otel.py +++ b/airflow-core/tests/integration/otel/test_otel.py @@ -382,6 +382,7 @@ def _get_ti(self, dag_id: str, run_id: str, task_id: str) -> Any | None: ) return ti + @pytest.mark.execution_timeout(90) @pytest.mark.parametrize( ("legacy_names_on_bool", "legacy_names_exported"), [ @@ -417,6 +418,7 @@ def test_export_legacy_metric_names(self, legacy_names_on_bool, legacy_names_exp if legacy_names_exported: assert set(legacy_metric_names).issubset(metrics_dict.keys()) + @pytest.mark.execution_timeout(90) def test_export_metrics_during_process_shutdown(self, capfd): out, dag = self.dag_execution_for_testing_metrics(capfd)