Skip to content

[v3-1-test] Flush in-memory OTel metrics at process shutdown (#61808)#61869

Merged
potiuk merged 3 commits intoapache:v3-1-testfrom
xBis7:backport-a9df122-v3-1-test
Feb 17, 2026
Merged

[v3-1-test] Flush in-memory OTel metrics at process shutdown (#61808)#61869
potiuk merged 3 commits intoapache:v3-1-testfrom
xBis7:backport-a9df122-v3-1-test

Conversation

@xBis7
Copy link
Contributor

@xBis7 xBis7 commented Feb 13, 2026

Manual backport for #61808


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@potiuk potiuk force-pushed the backport-a9df122-v3-1-test branch from 19d54af to b998394 Compare February 15, 2026 23:34
@potiuk
Copy link
Member

potiuk commented Feb 15, 2026

Rebased - but there were some strange errors?

@xBis7
Copy link
Contributor Author

xBis7 commented Feb 16, 2026

It's the new unit test that I added.

https://github.com/apache/airflow/pull/61869/changes#diff-62c84d713d57cf60b8db5b418f5c37f2490555fddf66e0b88e8f6542f1c7e076R320-R321

test_module_name = "unit.core.test_otel_logger"
function_call_str = f"import {test_module_name} as m; m.mock_service_run()"

When it tries to execute the import, we get an error that the module unit.core isn't found.

@xBis7 xBis7 force-pushed the backport-a9df122-v3-1-test branch from a9d887d to 13a2218 Compare February 16, 2026 08:48
@xBis7
Copy link
Contributor Author

xBis7 commented Feb 16, 2026

@potiuk I fixed it and also did another rebase. The CI is green. Can you take another look?

@potiuk potiuk merged commit bb00374 into apache:v3-1-test Feb 17, 2026
87 checks passed
@potiuk
Copy link
Member

potiuk commented Feb 17, 2026

Cool! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants