sensu::checks failing when subscribers are specified

[devopsprosiva]

Hi,

I've the following code in hiera to define a check.

sensu::checks:
'check_cpu':
command: '/etc/sensu/plugins/check-cpu.rb -w 75 -c 90'
handlers: 'mail'
subscribers: 'all'

After I added the subscribers parameter, the puppet agent run is failing with this error.

Error: undefined method sort' for nil:NilClass Error: /Stage[main]/Sensu/Sensu::Check[check_cpu]/Sensu_check[check_cpu]/subscribers: change from to all failed: undefined methodsort' for nil:NilClass

Thanks,
Siva

[poolski]

I think it's to do with the fact that the module expects an array for both subscribers and handlers, even if it's just one item.

Check out my JSON structure in #382, make handlers and subscribers into arrays and try again?

[devopsprosiva]

@poolski I'm still getting this error.

Error: undefined method `sort' for nil:NilClass
Error: /Stage[main]/Sensu/Sensu::Check[check_http_wiki]/Sensu_check[check_http_wiki]/subscribers: change from  to all failed: undefined method `sort' for nil:NilClass

• This is my yaml file with arrays for handlers and subscribers.

sensu::checks:
  'check_http_wiki':
    command: '/etc/sensu/plugins/check-http.rb -u http://localhost -s -k -r'
    handlers:
      - 'mail'
    subscribers:
      - 'all'

[poolski]

It may be that the "all" keyword is reserved, or something? I don't really know the ins and outs of this module so I'm just throwing ideas out there to see what sticks!

I don't use "subscribers" at all where the check is common to all devices.

[jlambert121]

@devopsprosiva I believe this was fixed with #376. What version of the module are you using?

[devopsprosiva]

@jlambert121 I was using an older version. Pulled the latest code now and testing. Thanks for the help.

[jlambert121]

Awesome!