Alexandros Kosiaris has submitted this change and it was merged.

Change subject: deployment::target does not work in labs, skip it
......................................................................


deployment::target does not work in labs, skip it

Spotted while installing on beta a role::applicationserver. The common
sub class includes deployment::target which is causing a bunch of errors
in puppet.

Since we cant use salt on beta, lets skip the deployment::target
entirely unless we are in production realm.

Change-Id: I5702f3de95d4abdc23e6470461f0a2357afea122
---
M manifests/role/applicationserver.pp
1 file changed, 3 insertions(+), 1 deletion(-)

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



diff --git a/manifests/role/applicationserver.pp 
b/manifests/role/applicationserver.pp
index d3c7884..6dcfbc4 100644
--- a/manifests/role/applicationserver.pp
+++ b/manifests/role/applicationserver.pp
@@ -108,7 +108,9 @@
                        class { "lvs::realserver": realserver_ips => 
$lvs::configuration::lvs_service_ips[$::realm][$lvs_pool][$::site] }
                }
 
-               deployment::target { "mediawiki": }
+               if $::realm == 'production' {
+                       deployment::target { "mediawiki": }
+               }
        }
 
        # This class installs everything necessary for an apache webserver

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5702f3de95d4abdc23e6470461f0a2357afea122
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Ryan Lane <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to