-
Notifications
You must be signed in to change notification settings - Fork 40
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
Repo installation is handled by uchiwa and sensu itself #46
Comments
Hmm the issue is that sensu and uchiwa cookbooks are meant to be decoupled and be able to run independently. I think that you probably should remove the repo.rb from your runlist if you are running both of them on the same instance. We might be able to inspect the collection of resources to conditionally assign it. |
I see that here we include the recipe so ya I see where you are coming from. I think what makes the most sense is to create custom resources that give you all of that functionality so you can decide how to implement it in your wrapper. |
or use 'includepkgs' to only pull in sensu/uchiwa from a particular repo |
@sspans if you want to submit a PR I am open to it, I am going to be working on moving to sensu-go in a few months and I have not had issues with my existing setup. If someone wants to submit a PR I am all ears but I am not likely gonna spend any time looking into improving this. |
Right now, the sensu repo is added by the sensu-chef and the uchiwa-chef cookbook. This creates chef warnings and leads to other funny problems.
I would recommend these steps:
The text was updated successfully, but these errors were encountered: