Dzahn has submitted this change and it was merged.

Change subject: stats.wm - re-enable loading of Apache SSL module in metrics.wm 
now that we don't do that with webserver::apache::site anymore. or there is a 
missing dependency on Webserver::Apache::Module[ssl]
......................................................................


stats.wm - re-enable loading of Apache SSL module in metrics.wm now that we
don't do that with webserver::apache::site anymore.
or there is a missing dependency on Webserver::Apache::Module[ssl]

Change-Id: Ie9eed88a0b0158861c6fb20a6fa6d496b3e7b533
---
M manifests/misc/statistics.pp
1 file changed, 1 insertion(+), 3 deletions(-)

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



diff --git a/manifests/misc/statistics.pp b/manifests/misc/statistics.pp
index af82a04..edc31f9 100644
--- a/manifests/misc/statistics.pp
+++ b/manifests/misc/statistics.pp
@@ -456,9 +456,7 @@
        include webserver::apache
        webserver::apache::module { "wsgi": }
        webserver::apache::module { "alias": }
-       # ssl module already loaded by webserver.pp
-       # when using webserver::apache::site with ssl => true
-       # webserver::apache::module { "ssl": }
+       webserver::apache::module { "ssl": }
 
        # install metrics.wikimedia.org SSL certificate
   install_certificate{ $site_name: }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie9eed88a0b0158861c6fb20a6fa6d496b3e7b533
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to