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

Fix mysql tasks when using custom socket #351

Closed
wants to merge 1 commit into from

Conversation

netzvieh
Copy link

Update all tasks interacting with the MySQL db on the system to use the
configured socket. Otherwise setting the mysql_socket variable to
something different from the default will result in errors since the
tasks can not connect to the db.

Update all tasks interacting with the MySQL db on the system to use the
configured socket. Otherwise setting the mysql_socket variable to
something different from the default will result in errors since the
tasks can not connect to the db.
@stale
Copy link

stale bot commented Mar 6, 2020

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

@stale stale bot added the stale label Mar 6, 2020
@stale
Copy link

stale bot commented Apr 5, 2020

This pull request has been closed due to inactivity. If you feel this is in error, please reopen the pull request or file a new PR with the relevant details.

@stale stale bot closed this Apr 5, 2020
@LordAro
Copy link

LordAro commented May 7, 2020

Can we do better here? There have been at least 4 separate PRs (#312, #351, #358 & #366) that I can see that attempt to fix this issue. Or specifically, ansible/ansible#47736 . Being busy is perfectly fine, but if stalebot is closing things faster than you can triage/merge them, it needs to be reconfigured or removed.

I'm pretty sure this is the cleanest one, and as I understand it the CI should work now that the Debian 10 default changes have been merged.

My specific issue here is the change in behaviour that's brought about by using pymysql as opposed to mysqldb - it no longer defaults to using the socket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants