Skip to content

Set provider on consul service#125

Merged
solarkennedy merged 1 commit intovoxpupuli:masterfrom
Interencheres:set_init_provider
Mar 10, 2016
Merged

Set provider on consul service#125
solarkennedy merged 1 commit intovoxpupuli:masterfrom
Interencheres:set_init_provider

Conversation

@albustax
Copy link

Hi,

If provider is not set, service management does not work on Debian 8, we have an error:

Error: /Stage[main]/Consul::Run_service/Service[consul]: Could not evaluate: Could not find init script for 'consul'

This PR set provider defined in consul::params

@solarkennedy
Copy link
Contributor

I'm hesitant, usually puppet figures out the provider correctly, we shouldn't have to be explicit here.

can you run sudo puppet resource service consul --debug and pastebin?

@albustax
Copy link
Author

@solarkennedy
Copy link
Contributor

Hmm, I'm pretty new to systemd, but is it more that the service name is simply "consul.service"? Per your pb there was a Service[consul.service] in there.

@solarkennedy
Copy link
Contributor

Sorry for the late review. This seems like a good thing, and init systems are hard, especially when there can be multiple providers per os. Thanks!

solarkennedy added a commit that referenced this pull request Mar 10, 2016
@solarkennedy solarkennedy merged commit 85de0cd into voxpupuli:master Mar 10, 2016
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