-
Notifications
You must be signed in to change notification settings - Fork 794
(MODULES-1143) Add package_manage parameters #617
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
(MODULES-1143) Add package_manage parameters #617
Conversation
i'm really having trouble comprehending the motivation behind this ticket/pr. ping @hunner bc you approved of the ticket ;) |
I don't have any strong feelings on this PR either way. However, I can guess that at least one of the motivations for the person who raised the ticket would have been the MySQL forks that are floating around (and will probably increase given public statements from Oracle). Percona has its own module on the Forge already, although one could argue that it differs from MySQL enough to need one. |
I would like to argue that the module is tagged with percona on the forge but is not able to work with it at this time. With this PR integrated one would be able to install percona (or MariaDB) and have it managed by this module. It would make it possible to get an existing server under Puppet control more easily which is a big plus in my eyes. |
@Micsi it's possible, even now, to install and manage (mostly…) MariaDB. @juniorsysadmin would you mind rebasing this pull request? |
@igalic unfortunately, it's not really choosable... the module uses MariaDB on some versions of some distros (as to params.pp). The user is not involved. Again, I would see this a big plus for several reasons:
To the last point: I have my systems upgrade automatically using puppetlabs-apt unattended-upgrades. Some important pieces are hold, notably the databases as upgrades of those involve a server restart |
… to start service - to ensure it gets started with correct settings and does not try to enable service before it actually exists :)
… module or in unusual places and make it play nicely with user specified package_names. Works with PR puppetlabs#617
This patch adds a package_manage parameter for both mysql::server and mysql::client
(MODULES-1143) Add package_manage parameters
Thanks @juniorsysadmin ! |
This patch adds a package_manage parameter for both mysql::server and mysql::client