Hashar has uploaded a new change for review.

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


Change subject: nginx_site now supports boolean values
......................................................................

nginx_site now supports boolean values

The nginx_site define is only handling string, hence passing boolean
true would not match.

Change-Id: Ie70b24f606486ff3789cf7fd47b7faf9f222c708
---
M manifests/generic-definitions.pp
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/83/62583/1

diff --git a/manifests/generic-definitions.pp b/manifests/generic-definitions.pp
index ae22901..ae315e9 100644
--- a/manifests/generic-definitions.pp
+++ b/manifests/generic-definitions.pp
@@ -130,7 +130,7 @@
        } else {
                $template_name = $template
        }
-       if ( $enable == "true" ) {
+       if ( $enable == "true" or $enable == true ) {
                file { "/etc/nginx/sites-enabled/${name}":
                        ensure => "/etc/nginx/sites-available/${name}",
                }
@@ -141,7 +141,7 @@
        }
 
        case $install {
-       "true": {
+       "true", true: {
                        file { "/etc/nginx/sites-available/${name}":
                                source => "puppet:///files/nginx/sites/${name}";
                        }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie70b24f606486ff3789cf7fd47b7faf9f222c708
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>

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

Reply via email to