This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 9cd75dcab83 OTel integration: align test_export_legacy_metric_names
timeout with siblings (#67170)
9cd75dcab83 is described below
commit 9cd75dcab839d30c2e27b5e0148215d1fbea3f3b
Author: Jarek Potiuk <[email protected]>
AuthorDate: Tue May 19 13:25:24 2026 +0200
OTel integration: align test_export_legacy_metric_names timeout with
siblings (#67170)
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 1b23fe0f754..4f46b37e851 100644
--- a/airflow-core/tests/integration/otel/test_otel.py
+++ b/airflow-core/tests/integration/otel/test_otel.py
@@ -382,6 +382,7 @@ class TestOtelIntegration:
)
return ti
+ @pytest.mark.execution_timeout(90)
@pytest.mark.parametrize(
("legacy_names_on_bool", "legacy_names_exported"),
[
@@ -417,6 +418,7 @@ class TestOtelIntegration:
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)
