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

bzip in backup script #491

Closed
mr-tron opened this issue Mar 12, 2014 · 4 comments
Closed

bzip in backup script #491

mr-tron opened this issue Mar 12, 2014 · 4 comments

Comments

@mr-tron
Copy link

mr-tron commented Mar 12, 2014

Need add into "class mysql::server::backup" resource package {"mysql_require_bzip2": name => bzip2, ensure => latest}"
or change bzip to gzip in backup scripts.

@igalic
Copy link
Contributor

igalic commented Mar 12, 2014

As we see with many, many other modules, managing trivial dependencies in the module is a good idea for some people, but a bad idea for most others.

@mr-tron
Copy link
Author

mr-tron commented Mar 12, 2014

Ok. I dislike this variant too. Сhanging script is more attractive.

@igalic
Copy link
Contributor

igalic commented Mar 12, 2014

The problem is we can't change the scripts because we'd be breaking the default, and, arguably, the compatibility with the module so far.

Adding the compression method as an option is also quite bothersome, well, … because it's one more option…
…but probably our best shot.

@mr-tron mr-tron closed this as completed Mar 12, 2014
@igalic
Copy link
Contributor

igalic commented Mar 12, 2014

I didn't mean to discourage you entirely… ¯(°_o)/¯

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

No branches or pull requests

2 participants