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

Service start is broken on Debian > 8.0 #232

Closed
albustax opened this issue Feb 24, 2016 · 5 comments
Closed

Service start is broken on Debian > 8.0 #232

albustax opened this issue Feb 24, 2016 · 5 comments

Comments

@albustax
Copy link
Contributor

Hello,

On Debian 8.2, (and 8.x globally), catalogue compilation is broken with this message:

Error: /Stage[main]/Consul::Run_service/Service[consul]: Could not evaluate: Could not find init script for 'consul'
Notice: /Stage[main]/Consul::Reload_service/Exec[reload consul service]: Dependency Service[consul] has failures: true
Warning: /Stage[main]/Consul::Reload_service/Exec[reload consul service]: Skipping because of failed dependencies
Notice: /Stage[main]/Consul/Anchor[consul_last]: Dependency Service[consul] has failures: true
Warning: /Stage[main]/Consul/Anchor[consul_last]: Skipping because of failed dependencies

Puppet try to launch the service with init because provideris not set on service declaration

This is a bug with the puppet provider, for more informations, see this bug

2 workarounds are possible:

  • Dirty method -> configure service globally
  Service { provider => 'systemd'}

  class { '::consul':
    config_hash => {
      'bootstrap_expect' => 1,
      'data_dir'         => '/opt/consul',
      'datacenter'       => 'eu-west-1',
      'log_level'        => 'INFO',
      'node_name'        => "${::fqdn}",
      'server'           => true,
    }
  }
  • Clean method -> change module behaviour

Add a switchcase on service declaration and set providerif necessary.

For informations:

puppet-client version: 3.8.6-1puppetlabs1
puppetmaster version: 3.8.6-1puppetlabs1
debian version: 8.2

@solarkennedy
Copy link
Contributor

Can you make a PR to make it work on the platform you use?
The code hint is probably here:
https://github.com/solarkennedy/puppet-consul/blob/3e43a74949df1ff9892f93ac5deb2a8fdc9d6362/manifests/params.pp#L52

@bdronneau
Copy link

Hey, this not the PR #125 ?

@albustax
Copy link
Contributor Author

👍

@solarkennedy
Copy link
Contributor

I've merge #125. Please try master.

@bdronneau
Copy link

with your last commit, puppet is able to start consul 👍

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

3 participants