Jcrespo has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/327193 )

Change subject: mariadb: Remove all paralelism from misc backup generation
......................................................................


mariadb: Remove all paralelism from misc backup generation

With OTRS offloaded, backups do not really take so much time,
and dbstore1001 gets temporarilly overloaded every week.

We can wait and take backups more slowly. If it is not enough,
we can do the same to the core backups.

Bug: T134977
Change-Id: Iee203567dd04d93b65e4c9a96431420f5cf3ee9a
---
M modules/role/templates/mariadb/backups/dumps-misc.sh.erb
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Alexandros Kosiaris: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Jcrespo: Looks good to me, approved



diff --git a/modules/role/templates/mariadb/backups/dumps-misc.sh.erb 
b/modules/role/templates/mariadb/backups/dumps-misc.sh.erb
index c7c9786..12dd0e1 100644
--- a/modules/role/templates/mariadb/backups/dumps-misc.sh.erb
+++ b/modules/role/templates/mariadb/backups/dumps-misc.sh.erb
@@ -6,14 +6,14 @@
 dump="/usr/local/bin/mysqldump --single-transaction --quick --master-data=2 
--skip-ssl"
 
 $dump -h m1-slave --databases bacula etherpadlite librenms puppet racktables 
rt | \
-    pigz > /srv/backups/m1-$(date +%Y%m%d%H%M%S).sql.gz &
+    pigz > /srv/backups/m1-$(date +%Y%m%d%H%M%S).sql.gz
 
 $dump -h m2-slave --databases iegreview reviewdb scholarships | \
-    pigz > /srv/backups/m2-$(date +%Y%m%d%H%M%S).sql.gz &
+    pigz > /srv/backups/m2-$(date +%Y%m%d%H%M%S).sql.gz
 
 # OTRS backup, due to size are handled separatelly (T138915)
 #$dump -h m2-slave --max_allowed_packet=64M --databases otrs | \
-#    pigz > /srv/backups/m2-otrs-$(date +%Y%m%d%H%M%S).sql.gz &
+#    pigz > /srv/backups/m2-otrs-$(date +%Y%m%d%H%M%S).sql.gz
 
 sql="select schema_name from information_schema.schemata where schema_name 
regexp '^(phab|phlegal)'"
 
@@ -24,7 +24,7 @@
 
 $dump -h m5-slave --databases ceilometer glance keystone neutron nova \
     designate designate_pool_manager nodepooldb | \
-    pigz > /srv/backups/m5-$(date +%Y%m%d%H%M%S).sql.gz &
+    pigz > /srv/backups/m5-$(date +%Y%m%d%H%M%S).sql.gz
 
 sql="select schema_name from information_schema.schemata where schema_name 
regexp '^(wik|flowdb)'"
 

-- 
To view, visit https://gerrit.wikimedia.org/r/327193
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iee203567dd04d93b65e4c9a96431420f5cf3ee9a
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Jcrespo <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[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

Reply via email to