Dzahn has submitted this change and it was merged.

Change subject: ldap: top-scope variable, mini lint fix
......................................................................


ldap: top-scope variable, mini lint fix

Change-Id: I6be6ca079956c34ce747236885ccdb35ba536d12
---
M modules/ldap/manifests/role/server.pp
1 file changed, 2 insertions(+), 2 deletions(-)

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



diff --git a/modules/ldap/manifests/role/server.pp 
b/modules/ldap/manifests/role/server.pp
index 8b7c49f..669588c 100644
--- a/modules/ldap/manifests/role/server.pp
+++ b/modules/ldap/manifests/role/server.pp
@@ -36,7 +36,7 @@
 
     sslcert::certificate { $certificate: }
 
-    monitoring::service { "https-${certificate}":
+    monitoring::service { "SSL-${certificate}":
         description   => 'LDAP-SSL',
         check_command => "check_ssl_ldap!${certificate}",
     }
@@ -79,7 +79,7 @@
         first_master         => false,
     }
 
-    if $realm == 'labs' {
+    if $::realm == 'labs' {
         # server is on localhost
         file { '/var/opendj/.ldaprc':
             content => 'TLS_CHECKPEER   no TLS_REQCERT     never ',

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6be6ca079956c34ce747236885ccdb35ba536d12
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to