Cannot differentiate string and numeric options

[joshuaspence]

I am using the following Puppet code:

  telegraf::input { 'statsd':
    options => {
      service_address          => '127.0.0.1:8125',
      allowed_pending_messages => 10000,
    },
  }

This results in the following Telegraf configuration:

[[inputs.statsd]]
  allowed_pending_messages = "10000"
  service_address = "127.0.0.1:8125"
  templates = ["logstash.* namespace.sender.measurement*"]

This configuration is invalid and Telegraf reports the following error:

Error parsing /etc/telegraf/telegraf.d/statsd.conf, line 2: statsd.Statsd.AllowedPendingMessages: `string' type is not assignable to `int' type

The problem here is that Puppet doesn't differentiate between numbers and strings (at least until Puppet 4, unless using the future parser). I was able to workaround this issue with the following Puppet code:

  telegraf::input { 'statsd':
    options => {
      service_address          => '127.0.0.1:8125',
      allowed_pending_messages => 0 + 10000,
    },
  }

Ideally though, the template used by telegraf::input would check if a configuration option is numeric and remove the quotes.