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

allow overriding the init subsystem. #145

Merged
merged 1 commit into from
Sep 12, 2017

Conversation

majormoses
Copy link
Collaborator

relates to #141

We now allow using node['sumologic']['init_style'] with any valid chef provider. Default is nil and will defer to ohai unless overridden.

Note: it does not attempt an validation that you chose a valid provider.

Signed-off-by: Ben Abrams [email protected]

General

  • Remove any versioning you did yourself if applicable

  • Update Changelog following the conventions laid out on Keep A Changelog with all new changes under ## [Unreleased] and using a ### Added, Fixed, Changed, or Breaking Change sub-header.

  • Update README with any necessary changes

  • RuboCop passes

  • Foodcritic passes

  • Existing tests pass

Purpose

Allow overwriting the init subsystem used when starting the collector.

Known Compatibility Issues

If you choose an init subsystem that the collector does not support you will need to write your init script yourself. There are a limited number of providers, at the time of writing this: https://github.com/chef/chef/tree/v13.4.19/lib/chef/provider/service is a list of them.

relates to #141

We now allow using `node['sumologic']['init_style']` with any valid chef provider. Default is `nil` and will defer to ohai unless overridden.

Note: it does not attempt an validation that you chose a valid provider.

Signed-off-by: Ben Abrams <[email protected]>
@majormoses majormoses self-assigned this Sep 11, 2017
@majormoses
Copy link
Collaborator Author

@dannenberg sorry this completely fell off my radar I believe this should "solve" your problems, let me know if this does not work for you.

@dannenberg
Copy link
Contributor

@majormoses appears to do the trick. thank you!

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.

2 participants