Yuvipanda has submitted this change and it was merged.
Change subject: tools: Rever attempts to set $ssh_hba in puppet
......................................................................
tools: Rever attempts to set $ssh_hba in puppet
We need role based hiera for this, sadly. We can't assign to
top scope from inside a role
Change-Id: I78d7d112dd7ec53f2a68276bf14e4ed72fb6c2c1
---
M manifests/role/labstools.pp
1 file changed, 0 insertions(+), 2 deletions(-)
Approvals:
Yuvipanda: Verified; Looks good to me, approved
diff --git a/manifests/role/labstools.pp b/manifests/role/labstools.pp
index 7ee1b8d..63804b5 100644
--- a/manifests/role/labstools.pp
+++ b/manifests/role/labstools.pp
@@ -14,8 +14,6 @@
}
class compute inherits role::labs::tools::common {
- $::ssh_hba = 'yes' # Yes, it requires this
-
include toollabs::compute
system::role { 'role::labs::tools::compute': description => 'Tool Labs
compute node' }
--
To view, visit https://gerrit.wikimedia.org/r/207659
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I78d7d112dd7ec53f2a68276bf14e4ed72fb6c2c1
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Yuvipanda <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
