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

$check_notify does not load sensu::enterprise::service #495

Closed
confiq opened this issue Apr 12, 2016 · 1 comment · Fixed by #720
Closed

$check_notify does not load sensu::enterprise::service #495

confiq opened this issue Apr 12, 2016 · 1 comment · Fixed by #720
Assignees

Comments

@confiq
Copy link
Contributor

confiq commented Apr 12, 2016

When you set sensu::check, this module does not notify Service["sensu-enterprise"].

I've look at code, there is "ugly hack" but that hack does not work with sensu-enterprise.
If needed, I can make PR for this but just tell me if I'm on right direction.
How Service["sensu-enterprise"] should get notified when check is added?

EDIT:
Maybe I should be more descriptive, so if you have understand what is the problem, you might stop reading after this point.

When I add new check ex:

sensu::check { 'check_cpu':
    command     => 'check-cpu.rb -w 85 -c 95',
  }

and run puppet apply returns this

Notice: /Stage[main]/Fb_sensu::Checks::System/Sensu::Check[check_cpu]/Sensu_check[check_cpu]/ensure: created
Notice: /Stage[main]/Sensu::Client::Service/Service[sensu-server]: Triggered 'refresh' from 2 events

The problem here is that sensu-server is being refreshed and not sensu-enterprise

EDIT: relevant PR #388

@jeffmccune jeffmccune self-assigned this Jul 7, 2017
jeffmccune added a commit to jeffmccune/sensu-puppet that referenced this issue Jul 7, 2017
Without this patch The sensu-enterprise service is not notified automatically
when `sensu::check` resources are declared.  This patch addresses the problem
by checking the catalog for the service and automatically adding the notify
relationship.

resolves sensu#495
closes sensu#653
jeffmccune added a commit to jeffmccune/sensu-puppet that referenced this issue Jul 9, 2017
Without this patch The sensu-enterprise service is not notified automatically
when `sensu::check` resources are declared.  This patch addresses the problem
by checking the catalog for the service and automatically adding the notify
relationship.

resolves sensu#502
resolves sensu#495
closes sensu#653
ghoneycutt added a commit that referenced this issue Jul 10, 2017
…-enterprise

(#495) Notify Service[sensu-enterprise] from Sensu::Check resources
@ghoneycutt
Copy link
Collaborator

Released in v2.19.1

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