[MediaWiki-commits] [Gerrit] Break dependency cycle in osm - change (operations/puppet)

2014-03-02 Thread Alexandros Kosiaris (Code Review)
Alexandros Kosiaris has uploaded a new change for review.

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

Change subject: Break dependency cycle in osm
..

Break dependency cycle in osm

Error was:
(Augeas[hba_create-replication@-v4] = Exec[pgreload] =
Class[Postgresql::Server] = File[/etc/postgresql/9.1/main/master.conf]
= Class[Postgresql::Master] = Postgresql::User[replication@-v4] =
Augeas[hba_create-replication@-v4])

Fixing it by removing a dependency

Change-Id: I48dce7079ae5231b28ae2298af0e19c67dd08665
---
M manifests/role/osm.pp
1 file changed, 0 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/76/116276/1

diff --git a/manifests/role/osm.pp b/manifests/role/osm.pp
index d979051..0d899a3 100644
--- a/manifests/role/osm.pp
+++ b/manifests/role/osm.pp
@@ -19,7 +19,6 @@
 method   = 'md5',
 attrs= 'REPLICATION',
 database = 'replication',
-require  = Class['postgresql::master'],
 }
 }
 if $osm_slave_v6 {
@@ -32,7 +31,6 @@
 method   = 'md5',
 attrs= 'REPLICATION',
 database = 'replication',
-require  = Class['postgresql::master'],
 }
 }
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I48dce7079ae5231b28ae2298af0e19c67dd08665
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris akosia...@wikimedia.org

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits


[MediaWiki-commits] [Gerrit] Break dependency cycle in osm - change (operations/puppet)

2014-03-02 Thread Alexandros Kosiaris (Code Review)
Alexandros Kosiaris has submitted this change and it was merged.

Change subject: Break dependency cycle in osm
..


Break dependency cycle in osm

Error was:
(Augeas[hba_create-replication@-v4] = Exec[pgreload] =
Class[Postgresql::Server] = File[/etc/postgresql/9.1/main/master.conf]
= Class[Postgresql::Master] = Postgresql::User[replication@-v4] =
Augeas[hba_create-replication@-v4])

Fixing it by removing a dependency

Change-Id: I48dce7079ae5231b28ae2298af0e19c67dd08665
---
M manifests/role/osm.pp
1 file changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Alexandros Kosiaris: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/manifests/role/osm.pp b/manifests/role/osm.pp
index d979051..0d899a3 100644
--- a/manifests/role/osm.pp
+++ b/manifests/role/osm.pp
@@ -19,7 +19,6 @@
 method   = 'md5',
 attrs= 'REPLICATION',
 database = 'replication',
-require  = Class['postgresql::master'],
 }
 }
 if $osm_slave_v6 {
@@ -32,7 +31,6 @@
 method   = 'md5',
 attrs= 'REPLICATION',
 database = 'replication',
-require  = Class['postgresql::master'],
 }
 }
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I48dce7079ae5231b28ae2298af0e19c67dd08665
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris akosia...@wikimedia.org
Gerrit-Reviewer: Alexandros Kosiaris akosia...@wikimedia.org
Gerrit-Reviewer: jenkins-bot 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits