Filippo Giunchedi has submitted this change and it was merged.

Change subject: cassandra: don't run validate_bool on trickle_fsync
......................................................................


cassandra: don't run validate_bool on trickle_fsync

Makes puppet barf with the value from hiera:

Error: Could not retrieve catalog from remote server: Error 400 on SERVER:
"false" is not a boolean.  It looks to be a String at
/etc/puppet/modules/cassandra/manifests/init.pp:330 on node
restbase1007.eqiad.wmnet

Change-Id: Idc629cce27f793cde251e08fd2c54a256b5b6f33
---
M modules/cassandra/manifests/init.pp
1 file changed, 0 insertions(+), 1 deletion(-)

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



diff --git a/modules/cassandra/manifests/init.pp 
b/modules/cassandra/manifests/init.pp
index aeac296..846807d 100644
--- a/modules/cassandra/manifests/init.pp
+++ b/modules/cassandra/manifests/init.pp
@@ -327,7 +327,6 @@
     validate_re($internode_compression, '^(all|dc|none)$')
     validate_re($disk_failure_policy, '^(stop|best_effort|ignore)$')
 
-    validate_bool($trickle_fsync)
     validate_array($additional_jvm_opts)
     validate_array($extra_classpath)
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idc629cce27f793cde251e08fd2c54a256b5b6f33
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Filippo Giunchedi <fgiunch...@wikimedia.org>
Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@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