-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[exporter/datadog] Fix canceling on createTracesExporter
#9797
Conversation
I am marking this as a blocker since it affects the Datadog exporter functionality in a bad way (missing host metadata, which can result in missing tags and incorrect dimensions). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
Ci failure is unrelated: trying to fix on #9799 |
@djaglowski we need this to be merged before Wednesday, since this addresses an important bug that would land on the next release otherwise. Could you have a look? |
Description:
Fixes a bug introduced in #9426 where a context is incorrectly canceled via
defer
instead of being canceled only when the function errors out.It was spotted by @gbbr on #9426 (comment).
Since this has not been out on any release, no changelog note is needed.