ArielGlenn has submitted this change and it was merged.
Change subject: Remove class role::dataset::publicdirs, noop
......................................................................
Remove class role::dataset::publicdirs, noop
It justs creates an empty directory and is applied only on fluorine
where it isn't used by anything. Remove it.
Change-Id: Ia8b6d0c1f214bd2439467316e077dea9dba355aa
---
M manifests/role/dataset.pp
M manifests/site.pp
2 files changed, 1 insertion(+), 11 deletions(-)
Approvals:
ArielGlenn: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/dataset.pp b/manifests/role/dataset.pp
index 08152fd..657c528 100644
--- a/manifests/role/dataset.pp
+++ b/manifests/role/dataset.pp
@@ -121,13 +121,3 @@
}
}
-class role::dataset::publicdirs {
-
- file { '/a/public-datasets':
- ensure => 'directory',
- owner => 'root',
- group => 'deployment',
- mode => '0775',
- }
-
-}
diff --git a/manifests/site.pp b/manifests/site.pp
index 79b4b75..bdbb032 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -959,7 +959,7 @@
}
node 'fluorine.eqiad.wmnet' {
- role xenon, dataset::publicdirs
+ role xenon
$cluster = 'misc'
include standard
--
To view, visit https://gerrit.wikimedia.org/r/246824
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia8b6d0c1f214bd2439467316e077dea9dba355aa
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: ArielGlenn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
