Skip to content

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

Merged
potiuk merged 3 commits into
apache: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 into
apache:v3-1-testfrom
xBis7:backport-a9df122-v3-1-test

Conversation

@xBis7

@xBis7 xBis7 commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

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

potiuk commented Feb 15, 2026

Copy link
Copy Markdown
Member

Rebased - but there were some strange errors?

@xBis7

xBis7 commented Feb 16, 2026

Copy link
Copy Markdown
Contributor Author

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

xBis7 commented Feb 16, 2026

Copy link
Copy Markdown
Contributor Author

@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

potiuk commented Feb 17, 2026

Copy link
Copy Markdown
Member

Cool! Thanks!

@xBis7 xBis7 deleted the backport-a9df122-v3-1-test branch February 17, 2026 07:00
@xBis7

xBis7 commented Feb 17, 2026

Copy link
Copy Markdown
Contributor Author

@potiuk Thanks!

@vatsrahul1001 vatsrahul1001 added this to the Airflow 3.1.8 milestone Mar 1, 2026
@vatsrahul1001 vatsrahul1001 added the type:bug-fix Changelog: Bug Fixes label Mar 1, 2026
vatsrahul1001 pushed a commit that referenced this pull request Mar 4, 2026
…#61869)

* manually backport and resolve conflicts

* fix mypy-airflow-core error

* make package unit.core importable for the test subprocess
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants