Alexandros Kosiaris has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/310811

Change subject: postgres: Fix recovery.conf dependency
......................................................................

postgres: Fix recovery.conf dependency

After the refactor in 7c84ece, the recovery.conf file no longer gets
reliably populated in the correct order. Fix that

Change-Id: Iecf8b97796e045f72e33f9100cf715ec18dba6a8
---
M modules/postgresql/manifests/slave.pp
1 file changed, 2 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/11/310811/1

diff --git a/modules/postgresql/manifests/slave.pp 
b/modules/postgresql/manifests/slave.pp
index 4b0c1be..8e72661 100644
--- a/modules/postgresql/manifests/slave.pp
+++ b/modules/postgresql/manifests/slave.pp
@@ -68,10 +68,8 @@
         group   => 'root',
         mode    => '0444',
         content => template('postgresql/recovery.conf.erb'),
-        require => [
-                    Class['postgresql::server'],
-                    Exec["pg_basebackup-${master_server}"],
-                ]
+        before  => Class['postgresql::server'],
+        require => Exec["pg_basebackup-${master_server}"],
     }
 
     # Let's sync once all our content from the master

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iecf8b97796e045f72e33f9100cf715ec18dba6a8
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris <[email protected]>

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

Reply via email to