Jcrespo has uploaded a new change for review. ( 
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(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/91/381491/1

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: newchange
Gerrit-Change-Id: If5fe7f4f3008c722286c30611a4c880fd0af6025
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Jcrespo <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to