[MediaWiki-commits] [Gerrit] openstack: move instancersync define to own file - change (operations/puppet)

2016-06-26 Thread Dzahn (Code Review)
Dzahn has submitted this change and it was merged.

Change subject: openstack: move instancersync define to own file
..


openstack: move instancersync define to own file

Change-Id: Ibb86285d99123c3e04f5a60d04c98864d8ca4e26
---
M modules/openstack/manifests/nova/compute.pp
A modules/openstack/manifests/nova/instancersync.pp
2 files changed, 10 insertions(+), 10 deletions(-)

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



diff --git a/modules/openstack/manifests/nova/compute.pp 
b/modules/openstack/manifests/nova/compute.pp
index 80efe7c..ddbdbdb 100644
--- a/modules/openstack/manifests/nova/compute.pp
+++ b/modules/openstack/manifests/nova/compute.pp
@@ -207,13 +207,3 @@
 }
 }
 
-# defines an rsync server on an instance
-define instancersync (
-$hostname = undef) {
-
-rsync::server::module { "nova_instance_rsync_${hostname}":
-path=> '/var/lib/nova/instances',
-read_only   => 'no',
-hosts_allow => ["${hostname}.${::site}.wmnet"],
-}
-}
diff --git a/modules/openstack/manifests/nova/instancersync.pp 
b/modules/openstack/manifests/nova/instancersync.pp
new file mode 100644
index 000..9827fdd
--- /dev/null
+++ b/modules/openstack/manifests/nova/instancersync.pp
@@ -0,0 +1,10 @@
+# defines an rsync server on an instance
+define instancersync (
+$hostname = undef) {
+
+rsync::server::module { "nova_instance_rsync_${hostname}":
+path=> '/var/lib/nova/instances',
+read_only   => 'no',
+hosts_allow => ["${hostname}.${::site}.wmnet"],
+}
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibb86285d99123c3e04f5a60d04c98864d8ca4e26
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn 
Gerrit-Reviewer: Dzahn 
Gerrit-Reviewer: jenkins-bot <>

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


[MediaWiki-commits] [Gerrit] openstack: move instancersync define to own file - change (operations/puppet)

2016-06-25 Thread Dzahn (Code Review)
Dzahn has uploaded a new change for review.

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

Change subject: openstack: move instancersync define to own file
..

openstack: move instancersync define to own file

Change-Id: Ibb86285d99123c3e04f5a60d04c98864d8ca4e26
---
M modules/openstack/manifests/nova/compute.pp
1 file changed, 0 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/85/295985/1

diff --git a/modules/openstack/manifests/nova/compute.pp 
b/modules/openstack/manifests/nova/compute.pp
index 80efe7c..ddbdbdb 100644
--- a/modules/openstack/manifests/nova/compute.pp
+++ b/modules/openstack/manifests/nova/compute.pp
@@ -207,13 +207,3 @@
 }
 }
 
-# defines an rsync server on an instance
-define instancersync (
-$hostname = undef) {
-
-rsync::server::module { "nova_instance_rsync_${hostname}":
-path=> '/var/lib/nova/instances',
-read_only   => 'no',
-hosts_allow => ["${hostname}.${::site}.wmnet"],
-}
-}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibb86285d99123c3e04f5a60d04c98864d8ca4e26
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn 

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