multiple procstat sections failing

[ZyanKLee]

Hey,
I probably misunderstood something about the patches in #80, but I fail to implement multiple procstat sections.

What I did:

   procstat-nginx:
     - pattern: "nginx"
       pid_tag: true
       plugin_type: "procstat"
   procstat-php-fpm:
     - pattern: "php.*-fpm"
       pid_tag: true
       plugin_type: "procstat"

What I expected:

[[inputs.procstat]]
  pattern = "nginx"
  pid_tag = true
[[inputs.procstat]]
  pattern = "php.*-fpm"
  pid_tag = true

What I got instead:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Function Call, Failed to parse template telegraf/telegraf.conf.erb:
  Filepath: /etc/puppetlabs/code/environments/production/modules/telegraf/templates/telegraf.conf.erb
  Line: 35
  Detail: undefined method `each' for nil:NilClass
 at /etc/puppetlabs/code/environments/production/modules/telegraf/manifests/config.pp:12:18 on node $SERVERNAME
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

System Specs

• ubuntu 16.04 (all systems)
• puppet agent 4.10.9
• puppetserver version: 2.8.0
• yankcrime/telegraf 2.0.0

[nrdmn]

Hi,
please try

   procstat:
     - pattern: "nginx"
       pid_tag: true
     - pattern: "php.*-fpm"
       pid_tag: true

[ZyanKLee]

I had a look at the mentioned template file. line 35 is related to outputs, not inputs.
so I further looked at our output declaration:

    outputs  => {
        'influxdb' => [
            {
                'urls'     => [ "http://influxdb.ourdomain.exmpl:8086" ],
                'database' => 'telegraf',
                'username' => 'telegraf',
                'password' => 'metricsmetricsmetrics',
                'retention_policy'  => '',
                'write_consistency' => 'any',
                'timeout'           => '20s',
            }
        ]
    },

this seems rather basic to me. (we only added the last three parameters and do only use one influxdb url)

[ZyanKLee]

update:

after I added those squared brackets the issue at line 35 of the template was fixed, next issue is at line 55:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Function Call, Failed to parse template telegraf/telegraf.conf.erb:
  Filepath: /etc/puppetlabs/code/environments/production/modules/telegraf/templates/telegraf.conf.erb
  Line: 55
  Detail: undefined method `each' for nil:NilClass
 at /etc/puppetlabs/code/environments/production/modules/telegraf/manifests/config.pp:12:18 on node $SERVERNAME

after going through all the hieradata and profiles and adding that extra layer (as shown for the nginx input in the README.md) puppet ran without error. thanks