Skip to content

Fix auto instrumentation entry point for mysql#858

Merged
toumorokoshi merged 4 commits intoopen-telemetry:masterfrom
mariojonke:mysql-auto-instrumentation-fix
Jun 30, 2020
Merged

Fix auto instrumentation entry point for mysql#858
toumorokoshi merged 4 commits intoopen-telemetry:masterfrom
mariojonke:mysql-auto-instrumentation-fix

Conversation

@mariojonke
Copy link
Contributor

entry point for the mysql instrumentation pointed to pymysql instrumentation which caused loading of the instrumentation to fail since it does not exist.

@mariojonke mariojonke requested a review from a team June 26, 2020 09:38
Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Hey @mariojonke! Thanks for the fix. Could you add a note to the CHANGELOG, i can approve and merged once it's in there.

@mariojonke mariojonke requested a review from codeboten June 29, 2020 07:02
Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Thanks for updating the log!

@toumorokoshi toumorokoshi merged commit 0c102a1 into open-telemetry:master Jun 30, 2020
@mariojonke mariojonke deleted the mysql-auto-instrumentation-fix branch July 1, 2020 05:08
cnnradams pushed a commit to cnnradams/opentelemetry-python that referenced this pull request Jul 2, 2020
entry point for the mysql instrumentation pointed to pymysql instrumentation which caused loading of the instrumentation to fail since it does not exist.

Co-authored-by: alrex <aboten@lightstep.com>
Co-authored-by: Yusuke Tsutsumi <yusuke@tsutsumi.io>
codeboten pushed a commit that referenced this pull request Jul 7, 2020
entry point for the mysql instrumentation pointed to pymysql instrumentation which caused loading of the instrumentation to fail since it does not exist.

Co-authored-by: alrex <aboten@lightstep.com>
Co-authored-by: Yusuke Tsutsumi <yusuke@tsutsumi.io>
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this pull request Nov 1, 2020
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.

5 participants