Jcrespo has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/381491 )
Change subject: mariadb: backup user must be dump to match already in use mysql
account
......................................................................
mariadb: backup user must be dump to match already in use mysql account
Bug: T169516
Change-Id: If5fe7f4f3008c722286c30611a4c880fd0af6025
---
M modules/role/manifests/mariadb/backup_mydumper.pp
1 file changed, 10 insertions(+), 10 deletions(-)
Approvals:
jenkins-bot: Verified
Jcrespo: Looks good to me, approved
diff --git a/modules/role/manifests/mariadb/backup_mydumper.pp
b/modules/role/manifests/mariadb/backup_mydumper.pp
index 98e4e70..0855745 100644
--- a/modules/role/manifests/mariadb/backup_mydumper.pp
+++ b/modules/role/manifests/mariadb/backup_mydumper.pp
@@ -9,14 +9,14 @@
include ::profile::backup::host
include ::passwords::mysql::dump
- group { 'dumps':
+ group { 'dump':
ensure => present,
system => true,
}
- user { 'dumps':
+ user { 'dump':
ensure => present,
- gid => 'dumps',
+ gid => 'dump',
shell => '/bin/false',
home => '/srv/backups',
system => true,
@@ -25,23 +25,23 @@
file { '/srv/backups':
ensure => directory,
- owner => 'dumps',
- group => 'dumps',
+ owner => 'dump',
+ group => 'dump',
mode => '0600', # implicitly 0700 for dirs
}
file { '/usr/local/bin/dump_shards':
ensure => present,
- owner => 'dumps',
- group => 'dumps',
+ owner => 'dump',
+ group => 'dump',
mode => '0555',
source => 'puppet:///modules/role/mariadb/dump_shards.sh',
}
file { '/etc/mysql/dump_shards.cnf':
ensure => present,
- owner => 'dumps',
- group => 'dumps',
+ owner => 'dump',
+ group => 'dump',
mode => '0400',
content =>
"[client]\nuser=${passwords::mysql::dump::user}\npassword=${passwords::mysql::dump::pass}\n",
}
@@ -50,7 +50,7 @@
minute => 0,
hour => 22,
weekday => 2,
- user => 'dumps',
+ user => 'dump',
command => '/usr/local/bin/dump_shards >/dev/null 2>&1',
require => [File['/usr/local/bin/dump_shards'],
File['/srv/backups'],
--
To view, visit https://gerrit.wikimedia.org/r/381491
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If5fe7f4f3008c722286c30611a4c880fd0af6025
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Jcrespo <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Jcrespo <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
