Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

apply otel_service on metrics #42242

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

romsharon98
Copy link
Contributor

closes: #42210


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failure in the non-db tests

@romsharon98
Copy link
Contributor Author

Failure in the non-db tests

Running this test in main, and it failed with the same error:

FAILED tests/core/test_otel_logger.py::TestOtelMetrics::test_timer_start_and_stop_manually_send_true - assert 3140.0 == 3.14

Maybe @ferruzzi can help us figure it out?

@romsharon98
Copy link
Contributor Author

This PR: #42267 should fix the current failed tests

@ferruzzi
Copy link
Contributor

This PR: #42267 should fix the current failed tests

Yeah, looks like you got it sorted out. Looks like it was a side-effect of the PR which converted reporting timers from seconds to millis,

@romsharon98 romsharon98 added type:bug-fix Changelog: Bug Fixes affected_version:2.10 Issues Reported for 2.10 labels Sep 21, 2024
@romsharon98 romsharon98 merged commit e14b4ca into apache:main Sep 24, 2024
53 checks passed
@romsharon98 romsharon98 deleted the fix/otel-service-name branch September 24, 2024 12:37
romsharon98 added a commit to romsharon98/airflow that referenced this pull request Sep 24, 2024
* apply otel_service on metrics

* change version_added to 2.10.3

(cherry picked from commit e14b4ca)
@eladkal eladkal added this to the Airflow 2.10.3 milestone Sep 24, 2024
romsharon98 added a commit that referenced this pull request Sep 25, 2024
* apply otel_service on metrics

* change version_added to 2.10.3

(cherry picked from commit e14b4ca)
joaopamaral pushed a commit to joaopamaral/airflow that referenced this pull request Oct 21, 2024
* apply otel_service on metrics

* change version_added to 2.10.3
utkarsharma2 pushed a commit that referenced this pull request Oct 23, 2024
* apply otel_service on metrics

* change version_added to 2.10.3

(cherry picked from commit e14b4ca)
utkarsharma2 pushed a commit that referenced this pull request Oct 24, 2024
* apply otel_service on metrics

* change version_added to 2.10.3

(cherry picked from commit e14b4ca)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected_version:2.10 Issues Reported for 2.10 type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hardcoded Opentelemetry SERVICE_NAME
4 participants