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
Fix PyMySQL library to respect my.cnf
parameter
#9526
Conversation
Manage this branch in SquashTest this branch here: https://anirudhdgglissue-7518-y6t6n.squash.io |
collectors/python.d.plugin/python_modules/bases/FrameworkServices/MySQLService.py
Show resolved
Hide resolved
@anirudhdggl we need you to sign the CLA to get this PR merged |
SIgned. |
I guess this is not hard requirement. Indeed old versions of pymysql didnt support loading from @anirudhdggl thx for contribution! |
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.
LGTM!
i see pymysql has |
my.cnf
parameter using PyMySQL library
my.cnf
parameter using PyMySQL librarymy.cnf
parameter
Summary
Fixes #7518
Component Name
MySQLService.py
Test Plan
Have original users who raised the issue verify it
Additional Information
As mentioned in one of the issue comment, the requisite code block responsible for the issue has been removed so that no one faces this issue again