Skip to content

ensure that the elasticsearch instrumentation handles DroppedSpans correctly #1190

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

Merged
merged 1 commit into from
Jul 12, 2021

Conversation

beniwohli
Copy link
Contributor

What does this pull request do?

If the elasticsearch span is dropped for some reason, the context object
is None, which, if unhandled, leads to an exception

Related issues

closes #1188

…rrectly

If the elasticsearch span is dropped for some reason, the context object
is None, which, if unhandled, leads to an exception

fixes elastic#1188
@apmmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-07-12T10:31:24.761+0000

  • Duration: 27 min 22 sec

  • Commit: 7ab6d94

Test stats 🧪

Test Results
Failed 0
Passed 9507
Skipped 8604
Total 18111

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 9507
Skipped 8604
Total 18111

@beniwohli beniwohli merged commit 493ee0e into elastic:master Jul 12, 2021
@beniwohli beniwohli deleted the fix-1188 branch July 12, 2021 14:45
beniwohli added a commit to beniwohli/apm-agent-python that referenced this pull request Sep 14, 2021
…rrectly (elastic#1190)

If the elasticsearch span is dropped for some reason, the context object
is None, which, if unhandled, leads to an exception

fixes elastic#1188
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
close