-
-
Notifications
You must be signed in to change notification settings - Fork 693
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
Update 57 pw init #584
Update 57 pw init #584
Conversation
also removed fedora-28 from 56 yum distros, latest there was 24
@tealice would you fancy resolving the conflicts and working to get this merged in? |
Hey @tealice Any chance you could fix the issues with .kitchen so we can get tests passing etc? Thanks! |
This is exactly the same as sous-chefs#584 but squashed without the MySQL 8.0 changes, which should make it much easier to merge.
This is exactly the same as sous-chefs#584 but squashed without the MySQL 8.0 changes, which should make it much easier to merge.
Hey @tealice can you update your fork and fix the kitchen conflicts and we can ship this! Thanks, |
…ous-chefs#584 but squashed without the MySQL 8.0changes, which should make it much easier to merge.
@tealice reping |
Closing due to inactivity. If this is still wanted please reopen or open another pull request. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help! Thanks, |
Description
mysql-systemd-start
script under/usr/share/mysql/8.0
and didn't want to include too many changes here.v56plus
andv57plus
methods to useGem::Version
init_records_script
to set the root password usingALTER USER
for 5.7+, from Patched helpers password setting to work with mysql>=5.7 #518 and the issue linked below.Issues Resolved
Fixes #410
closes #518
Check List
Couple questions:
Should I update the README with which platforms have been tested with Test Kitchen? And not sure if I should add a DCO to any commits (maybe the vplus methods?) since the init_records_script modification was authored by someone else. Thanks!