Commit graph

213 commits

Author SHA1 Message Date
mh
6d9a846867 introduce helpers for mysql backups 2013-02-19 10:12:35 +01:00
Gabriel Filion
fc1210bea8 Too much linting for facts
ng brought a problem with my latest linting commit:

facts are _always_ strings, so unquoting the value that we're testing
against is actually wrong, even though puppet-lint complains.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>
2013-02-18 16:16:35 -05:00
Gabriel Filion
d456d0280f lint mysql::server 2013-01-16 03:30:44 +01:00
mh
9c455bb3a1 fix typo 2013-01-02 18:54:51 +01:00
mh
c1c3b11245 make trocla an optional dependency 2013-01-02 18:48:49 +01:00
mh
2e57128b06 mysql 5.0 does not yet know about :trigger_priv 2013-01-02 18:19:06 +01:00
mh
6d75999d6a get rid off trailing , 2013-01-02 18:18:39 +01:00
mh
8a96ea5cf0 Merge remote-tracking branch 'riseup/master'
Conflicts:
	manifests/client.pp
	manifests/server.pp
	manifests/server/base.pp
	manifests/server/cron/backup.pp
	manifests/server/munin/default.pp
	manifests/server/nagios.pp
2013-01-02 17:25:51 +01:00
mh
33241a38d2 linting 2013-01-02 17:13:45 +01:00
mh
dd6c24844b Merge remote-tracking branch 'shared/master'
Conflicts:
	files/config/my.cnf.Debian
	manifests/server/base.pp
	manifests/server/munin/default.pp
	manifests/server/nagios.pp
2013-01-02 17:03:18 +01:00
mh
35b62c1b3d some need the password not quoted... 2012-12-10 11:01:28 +01:00
mh
c9fda13929 simplify and correct dependencies
1. it is important that the root password works
2. we don't need to chain the dependencies
2012-12-09 16:03:39 +01:00
mh
7b2a72a976 document intend of refreshonyl for setting root pw 2012-12-09 16:01:02 +01:00
mh
de7cb43faa fix dependency issues 2012-12-09 15:52:34 +01:00
mh
828d52e667 woraround a puppet provider bug 2012-12-09 15:12:19 +01:00
mh
53d1455aa5 linting 2012-12-06 23:09:44 +01:00
Micah Anderson
9465f48a65 2.7 underscore rename: site-mysql 2012-06-27 15:47:24 -04:00
mh
60ec42a15f fix typo 2012-06-17 16:50:08 -03:00
mh
59ec83a3cc take hiera out of the params 2012-06-14 15:02:17 -03:00
mh
956edf58cc take hiera out of the params 2012-06-14 14:53:44 -03:00
o
01ab6a5315 some more hiera variables removed 2012-06-14 11:33:31 -03:00
o
cd68caf662 remove hiera 2012-06-13 20:22:45 -03:00
mh
e4ca510b7c refactor things for >2.7 2012-06-08 13:04:06 -03:00
mh
7f4ae6753f use correct backup location
Up to now we used an empty backup location... :/
2012-05-14 10:29:52 +02:00
mh
1f0784b73e Merge remote-tracking branch 'laurantb/master'
Conflicts:
	manifests/server/cron/backup.pp
2012-04-29 12:55:07 +02:00
Micah Anderson
9db98da33c fix trocla function lookup in template, without this, you get:
err: Could not retrieve catalog from remote server: Error 400 on SERVER: Failed to parse template mysql/root/my.cnf.erb: undefined method `function_trocla' for #<Puppet::Parser::Scope:0xf6457e2c> at /etc/puppet/modules/mysql/manifests/server/base.pp:50 on node
2012-04-18 13:26:57 -04:00
Micah Anderson
9e22feaca0 fix another trivial syntax error 2012-04-18 11:36:09 -04:00
Micah Anderson
a30637ad2e fix syntax error 2012-04-18 11:30:36 -04:00
Micah Anderson
05e97ab820 Merge remote-tracking branch 'lelutin/nagios_service' into shared 2012-04-11 18:28:15 -04:00
Micah Anderson
ad04b1fbe7 Merge remote-tracking branch 'lelutin/squeeze_my_cnf' into shared 2012-04-11 18:25:17 -04:00
Micah Anderson
27773b88a5 fix $use_munin hiera check to be use_munin instead of incorrect use_nagios
add $use_nagios hiera check
2012-04-11 12:59:33 -04:00
Gabriel Filion
8180e27115 Import my.cnf from Debian Squeeze
Lenny support was phased out and squeeze is the current stable.

There is an instruction in the file that originally came from lenny:
skip-bdb doesn't exist anymore in the mysql version in squeeze, so the
config file makes a fresh mysql install crash on Squeeze.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>
2012-04-09 02:26:23 -04:00
Micah Anderson
a55ce915da switch hiera to pull its variables in parameterized classes instead of inline in the manifest 2012-04-04 17:47:13 -04:00
Micah Anderson
32fdce32d9 add note to README about mysql::server::account_security removing accounts by default, and how to get around it 2012-04-04 13:42:11 -04:00
Micah Anderson
db8103b0ba Merge remote-tracking branch 'immerda/master'
Conflicts:
	manifests/server/munin/default.pp
	manifests/server/nagios.pp
2012-04-04 12:22:09 -04:00
Micah Anderson
cc65b6c584 Merge branch 'shared' 2012-03-07 15:11:06 -05:00
Micah Anderson
7f4f7e0052 fix typo in name of mysql_qcache_mem graph 2012-03-07 15:10:48 -05:00
Micah Anderson
56c0601953 Merge branch 'shared' 2012-03-07 11:52:15 -05:00
Micah Anderson
d66e58c325 Merge commit '356fdab8147f8a32a3f14514f2bb77f4f312c734' into shared
Conflicts:
	files/munin/mysql_connections
	manifests/server/base.pp
	manifests/server/munin/default.pp
2012-03-07 11:49:28 -05:00
Micah Anderson
084f0cfb67 Merge branch 'shared' 2012-03-07 11:05:49 -05:00
Micah Anderson
e9ff8b6a23 Merge remote-tracking branch 'lelutin/root_pw_no_arg' into shared
Conflicts:
	manifests/server/base.pp
2012-03-07 11:02:01 -05:00
Micah Anderson
196560b997 Merge remote-tracking branch 'lelutin/nagios_service' 2012-02-29 23:50:22 -05:00
Gabriel Filion
64bbe64b96 Use the right parameter to nagios::service::mysql
check_hostname is not defined. We need to use check_host instead
2012-02-23 04:52:55 -05:00
mh
64e116cfb5 correct key for trocla lookup 2012-02-10 15:04:40 +01:00
mh
4ac7369515 migrate everything to hiera/trocla and use the new scoping style 2012-02-10 14:35:19 +01:00
Micah Anderson
60bb96c9a1 add the trigger privilege to the list of potential MYSQL_USER_PRIVS 2012-01-22 23:48:33 -05:00
mh
a72ab41660 add innodb_file_per_table option by default
mysql has one innodb file per default and this can become very
huge, which can trigger various issues [1]. So in general it
is recommended to set the innodb_file_per_table option [2].

[1] http://forums.mysql.com/read.php?35,121880,121886
[2] http://dev.mysql.com/doc/refman/5.0/en/innodb-multiple-tablespaces.html
2012-01-17 22:31:44 +01:00
Micah Anderson
f2dd063f82 fix missing end 2012-01-16 15:48:31 -05:00
Micah Anderson
2fc3849818 Instead of doing an update table_priv, we need to do GRANT/REVOKE statements
When we are working with tables_priv we need to first get a downcased array of the currently set privileges, and a downcased array of the desired permissions.

Then we make a list of the permissions to revoke by subtracting the requested permissions from the currently set ones.

If the list of permissions to revoke is not empty, then we issue a REVOKE.

Then we make a list of the permissions to add by subtracting the requested permissions from the current set (no need to add select again if it is already there).

Then if the set of permissions to add is not empty, then we actually execute the statement.
2012-01-01 15:43:47 -05:00
Micah Anderson
7f10702c5a table privileges need to be handled with GRANT/REVOKE statements instead of inserts of Y/N values into the table.
To handle this, this comment removes the create_row for table_privs, it also selects the actual value of the Table_priv so its value can be used instead of the method that is used for Y/N value settings
2012-01-01 15:35:44 -05:00