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

Change subject: make the snapshot labs lvm stuff specific to a snapshot profile
......................................................................


make the snapshot labs lvm stuff specific to a snapshot profile

Didn't like the arrangement in previous commit much, this is clearer
Also use the standard mount point (the previous commit would have
required an additional symlink, no need for that)

Change-Id: Ibcb6eb56226321e57209380e17197dbf50bea59b
---
A modules/profile/manifests/dumps/generation/worker/labs_localdisk.pp
D modules/profile/manifests/labs/lvm/data.pp
2 files changed, 10 insertions(+), 8 deletions(-)

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



diff --git 
a/modules/profile/manifests/dumps/generation/worker/labs_localdisk.pp 
b/modules/profile/manifests/dumps/generation/worker/labs_localdisk.pp
new file mode 100644
index 0000000..447fb65
--- /dev/null
+++ b/modules/profile/manifests/dumps/generation/worker/labs_localdisk.pp
@@ -0,0 +1,10 @@
+# Provide space for writing dumps to local filesystem
+#
+# filtertags: labs-common
+class profile::dumps::generation::worker::labs_localdisk {
+    $xmldumpsmount = '/mnt/dumpsdata'
+
+    labs_lvm::volume { 'data-local-disk':
+        mountat => $xmldumpsmount,
+    }
+}
diff --git a/modules/profile/manifests/labs/lvm/data.pp 
b/modules/profile/manifests/labs/lvm/data.pp
deleted file mode 100644
index f1e04ff..0000000
--- a/modules/profile/manifests/labs/lvm/data.pp
+++ /dev/null
@@ -1,8 +0,0 @@
-# Allocate all of the instance's extra space as /data
-#
-# filtertags: labs-common
-class profile::labs::lvm::data {
-    labs_lvm::volume { 'data-local-disk':
-        mountat => '/data',
-    }
-}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibcb6eb56226321e57209380e17197dbf50bea59b
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: ArielGlenn <ar...@wikimedia.org>
Gerrit-Reviewer: ArielGlenn <ar...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to