Dzahn has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/394096 )
Change subject: role::ci::slave::browsertests: Fix $redis_port by adding string
......................................................................
role::ci::slave::browsertests: Fix $redis_port by adding string
Puppet is failing on one of my instances with
Illegal title type at index 0. Expected String, got Integer at
/etc/puppet/modules/role/manifests/ci/slave/browsertests.pp:18:23 on
node jenkins-slave-01.git.eqiad.wmflabs
Change-Id: I8eb81b4e1a7a89e6b6baffc1e0aa51547c7846e2
---
M modules/role/manifests/ci/slave/browsertests.pp
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Hashar: Looks good to me, but someone else must approve
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/modules/role/manifests/ci/slave/browsertests.pp
b/modules/role/manifests/ci/slave/browsertests.pp
index f26b36a..1e5bf21 100644
--- a/modules/role/manifests/ci/slave/browsertests.pp
+++ b/modules/role/manifests/ci/slave/browsertests.pp
@@ -13,7 +13,7 @@
include contint::browsertests
# For CirrusSearch testing:
- $redis_port = 6379
+ $redis_port = '6379'
redis::instance { $redis_port:
settings => {
--
To view, visit https://gerrit.wikimedia.org/r/394096
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8eb81b4e1a7a89e6b6baffc1e0aa51547c7846e2
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox <thomasmulhall...@yahoo.com>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: Hashar <has...@free.fr>
Gerrit-Reviewer: Thiemo Mättig (WMDE) <thiemo.kr...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
