Skip to content

helm: Use MYSQL_FLAVOR to set flavor instead of EXTRA_MY_CNF#4473

Merged
dkhenry merged 1 commit intovitessio:masterfrom
ssup2:master
Dec 20, 2018
Merged

helm: Use MYSQL_FLAVOR to set flavor instead of EXTRA_MY_CNF#4473
dkhenry merged 1 commit intovitessio:masterfrom
ssup2:master

Conversation

@ssup2
Copy link
Copy Markdown
Contributor

@ssup2 ssup2 commented Dec 19, 2018

mysqlctl uses env MYSQL_FLAVOR to set MySQL Flavor.
But helper template use EXTRA_MY_CNF to set MySQL Flavor.

This cause a bug when use mariadb or mariadb103.
mysqlctl set mariadb.conf or mariadb103.conf and 'mysql56.conf' in my.cnf.
And this wrong config cause failure to run mariadb or mariadb103.

Signed-off-by: Jungsub Shin supsup5642@gmail.com

mysqlctl uses env MYSQL_FLAVOR to set MySQL Flavor.
But helper template use EXTRA_MY_CNF to set MySQL Flavor.

This cause a bug when use mariadb or mariadb103.
mysqlctl set mariadb.conf or mariadb103.conf and 'mysql56.conf' in my.cnf.
And this wrong config cause failure to run mariadb or mariadb103.

Signed-off-by: Jungsub Shin <supsup5642@gmail.com>
@dkhenry
Copy link
Copy Markdown
Contributor

dkhenry commented Dec 19, 2018

@ssup2 I am still getting errors on this branch trying to start a cluster with MariaDB 10.3

@ssup2
Copy link
Copy Markdown
Contributor Author

ssup2 commented Dec 19, 2018

@dkhenry I tested my local kubernetes cluster with MariaDB 10.2 and MariaDB 10.3. And it works.

@ssup2
Copy link
Copy Markdown
Contributor Author

ssup2 commented Dec 19, 2018

@dkhenry Could I get the log of mysqld in a replica pod??

@dkhenry
Copy link
Copy Markdown
Contributor

dkhenry commented Dec 20, 2018

I think the error was with my Persistent volumes not getting deleted between my tests. I deleted everything and just ran with MariaDB 10.3 and it all worked.

Copy link
Copy Markdown
Contributor

@dkhenry dkhenry left a comment

Choose a reason for hiding this comment

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

LGTM

@dkhenry dkhenry merged commit 345d3bd into vitessio:master Dec 20, 2018
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.

2 participants