-
Notifications
You must be signed in to change notification settings - Fork 289
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
allow merging of hiera configs instead of only taking lowest in hierarchy #366
Comments
PR to follow (small change) |
@pdaugavietis have you made this work yet? I've just come across this, and merging of classes would be fantastic. |
i've got it working locally but not the tests :( i'm refactoring now to On Mon, Jun 15, 2015 at 11:38 AM, Robin Kearney [email protected]
|
okay - PR is passing now, and functioning as desired locally as well. thought i'd done this at least a week ago the first time, but now it's passing :) |
okay - this is now refactored to have 2 new options - $sensu::merge_{subscriptions,client_custom} that will merge all levels of hiera, instead of default taking lowest level as override(s) |
attacking this another way - closing ticket and pr |
specifically for subscriptions and client-custom fields.
The text was updated successfully, but these errors were encountered: