Dzahn has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/81844


Change subject: sigh, quote 'true' because webserver.pp does treat them as 
strings
......................................................................

sigh, quote 'true' because webserver.pp does treat them as strings

..maybe this should be fixed in webserver.pp:

if $ssl in ["true", "only", "redirected"] {

because if you don't (like you should) you get:

'true' from left operand of 'in' expression is not a string
at /etc/puppet/manifests/webserver.pp:296

Change-Id: Id4977d0bee57dd3a57dcbfb4edd87af0049e25b1
---
M manifests/misc/statistics.pp
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/44/81844/1

diff --git a/manifests/misc/statistics.pp b/manifests/misc/statistics.pp
index 403cf34..c86053b 100644
--- a/manifests/misc/statistics.pp
+++ b/manifests/misc/statistics.pp
@@ -263,7 +263,7 @@
        webserver::apache::site { $site_name:
                require => [Class["webserver::apache"], 
Webserver::Apache::Module["rewrite"], File["/etc/apache2/htpasswd.stats"]],
                docroot => $docroot,
-               ssl => true,
+               ssl => 'true',
                certfile => '/etc/ssl/certs/stats.wikimedia.org.chained.pem',
                certkey =>  '/etc/ssl/private/stats.wikimedia.org.key',
                aliases   => ["stats.wikipedia.org"],

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id4977d0bee57dd3a57dcbfb4edd87af0049e25b1
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <dz...@wikimedia.org>

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

Reply via email to