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 5.6 bind address #504

Merged
merged 1 commit into from
Sep 22, 2015
Merged

Conversation

lukewaite
Copy link
Contributor

MySQL has been updated upstream of the PPA used for 5.6. These changes introduce new config file locations, and conf.d directories. The bind-address setting which was previously in /etc/mysql/my.cnf is now located in /etc/mysql/mysql.conf.d/mysqld.cnf.

I have tested this locally with Vaprobash set to install both "5.5" and "5.6" MySQL versions to verify I didn't break 5.5 support inadvertently.

MySQL has been updated upstream of the PPA used for 5.6. These changes introduce new config file locations, and conf.d directories. The `bind-address` setting which was previously in `/etc/mysql/my.cnf` is now located in `/etc/mysql/mysql.conf.d/mysqld.cnf`.
@fideloper
Copy link
Owner

Awesome, thanks for this (and for regression testing, that's less common than hoped :D )

fideloper added a commit that referenced this pull request Sep 22, 2015
@fideloper fideloper merged commit 8a5ef32 into fideloper:master Sep 22, 2015
@lukewaite lukewaite deleted the fix-mysql-5.6 branch September 22, 2015 18:51
@dac514
Copy link

dac514 commented Mar 3, 2016

Setting mysql_version = "5.6" I'm getting:

==> Vaprobash: Setting up mysql-common-5.6 (5.6.28-0ubuntu0.14.04.1) ...
==> Vaprobash: Processing triggers for libc-bin (2.19-0ubuntu6.7) ...
==> Vaprobash: Processing triggers for ureadahead (0.100.0-16) ...
==> Vaprobash: sed: can't read /etc/mysql/mysql.conf.d/mysqld.cnf: No such file or directory
==> Vaprobash: Warning: Using a password on the command line interface can be insecure.
==> Vaprobash: mysql stop/waiting
==> Vaprobash: mysql start/running, process 13800

Someone else did too, and submitted a fix, but where did it go? I don't see it in master.

#514

Would be nice to do it conditionally.

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.

3 participants