Skip to content
This repository has been archived by the owner on Jul 11, 2020. It is now read-only.

Include default sensu recipe #8

Closed
wants to merge 3 commits into from
Closed

Include default sensu recipe #8

wants to merge 3 commits into from

Conversation

evverx
Copy link
Contributor

@evverx evverx commented Jan 21, 2015

From https://github.com/sensu/sensu-chef#sensudefault
This recipe must be included before any of the Sensu LWRP's can be used

Without it rubygems-monitoring::default failed on first run with:

Error executing action `create` on resource
'file[/etc/sensu/conf.d/handlers/default.json]'

From https://github.com/sensu/sensu-chef#sensudefault
This recipe must be included before any of the Sensu LWRP's can be used

Without it `rubygems-monitoring::default` failed on first run with:
```
Error executing action `create` on resource
'file[/etc/sensu/conf.d/handlers/default.json]'
```
Without it attributes have different values on client
(rubygems-sensu::default) and server (rubygems-sensu::server)
@skottler
Copy link
Member

Hey @evverx we're happy to accept this PR, but usually avoid using attributes except for setting defaults in the attributes directory. Can you remove the commits that change that and rebase off master?

@skottler
Copy link
Member

Closing this since there wasn't a response. Happy to re-open it if you want to finish it up, @evverx.

@skottler skottler closed this Mar 14, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants