Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix subscribers parameter in sensu_check #376

Merged
merged 1 commit into from
Jun 24, 2015
Merged

fix subscribers parameter in sensu_check #376

merged 1 commit into from
Jun 24, 2015

Conversation

kam1kaze
Copy link
Contributor

Hi,

Today I faced with the issue when tried to define subscribers for existed check:

Error: /Stage[main]/Lmm_sensu::Checks/Lmm_sensu::Checks::Elasticsearch[***]/Sensu::Check[***]/Sensu_check[***]/subscribers: change from  to lmm failed: undefined method `sort' for nil:NilClass

it is related to such part of code:

lib/puppet/type/sensu_check.rb

   81   newproperty(:subscribers, :array_matching => :all) do
   82     desc "Who is subscribed to this check"
   83     def insync?(is)
>  84       is.sort == should.sort
   85     end
   86   end

@jlambert121
Copy link
Contributor

Thanks!

jlambert121 added a commit that referenced this pull request Jun 24, 2015
fix subscribers parameter in sensu_check
@jlambert121 jlambert121 merged commit e656c25 into sensu:master Jun 24, 2015
@benh57
Copy link

benh57 commented Dec 23, 2015

Just encountered this crash. Thanks for fixing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants