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

Master broken on EL6 due to "Invalid service provider 'sysv'" #240

Closed
fatmcgav opened this issue Mar 15, 2016 · 0 comments
Closed

Master broken on EL6 due to "Invalid service provider 'sysv'" #240

fatmcgav opened this issue Mar 15, 2016 · 0 comments

Comments

@fatmcgav
Copy link
Contributor

When mistakenly using master, I observed that this module was broken on my CentOS 6 test box with the following error:

Invalid service provider 'sysv' at /modules/consul/manifests/run_service.pp:19

A quick browse through the code suggests that 85de0cd is the cause of this breakage, specifically whereby it's enforcing the provider param.
On <EL7, the default provider is init, not sysv.

Running through the history on params.pp, it looks like the this was first entered into params.pp in 24931c8, however as it was never passed through the service, it was never an issue previously...

PR incoming to correct default value on <EL7...

Cheers
Gav

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

No branches or pull requests

1 participant