Dzahn has submitted this change and it was merged.

Change subject: phragile: move role class to autoloader layout
......................................................................


phragile: move role class to autoloader layout

Move the role class to module role to a foo::bar
structure to put it in proper "autoloader layout".

In this case "labs" just makes sense for what it is.

Fix "class not documented" by adding a link to
mediawiki.org about Phragile.

Change-Id: Ifaf729d862d2f5fe59d297c75a1acc2086fa9d79
---
D manifests/role/phragile.pp
A modules/role/manifests/phragile/labs.pp
2 files changed, 7 insertions(+), 6 deletions(-)

Approvals:
  Jakob: Looks good to me, but someone else must approve
  JanZerebecki: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/manifests/role/phragile.pp b/manifests/role/phragile.pp
deleted file mode 100644
index bb240be..0000000
--- a/manifests/role/phragile.pp
+++ /dev/null
@@ -1,6 +0,0 @@
-class role::phragile {
-    # Only on labs!
-    requires_realm('labs')
-
-    include ::phragile
-}
diff --git a/modules/role/manifests/phragile/labs.pp 
b/modules/role/manifests/phragile/labs.pp
new file mode 100644
index 0000000..6f25ec1
--- /dev/null
+++ b/modules/role/manifests/phragile/labs.pp
@@ -0,0 +1,7 @@
+# https://www.mediawiki.org/wiki/Phragile
+class role::phragile::labs {
+    # Only on labs!
+    requires_realm('labs')
+
+    include ::phragile
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifaf729d862d2f5fe59d297c75a1acc2086fa9d79
Gerrit-PatchSet: 6
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Jakob <[email protected]>
Gerrit-Reviewer: JanZerebecki <[email protected]>
Gerrit-Reviewer: WMDE-leszek <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to