Skip to content
This repository has been archived by the owner on Oct 16, 2024. It is now read-only.

Fix creation of systemd service #8

Merged
merged 1 commit into from
Apr 12, 2018

Conversation

dmitry-g
Copy link

@dmitry-g dmitry-g commented Apr 12, 2018

  • Bump cookbook version to 0.3.0
  • Lock systemd version to ~> 3.2.2
  • Move description, after and condition_path_exists under unit section

This patch fixes the following issue:

05:34:25        NoMethodError
05:34:25        -------------
05:34:25        undefined method `description' for Custom resource systemd_service from cookbook systemd
05:34:25        
05:34:25        Relevant File Content:
05:34:25        ----------------------
05:34:25        /tmp/kitchen/cache/cookbooks/exabgp/recipes/default.rb:
05:34:25       
05:34:25         74:  systemd_service 'exabgp' do
05:34:25         75>>   description 'ExaBGP service'
05:34:25         76:    after node[:exabgp][:systemd][:after]
05:34:25         77:    condition_path_exists '/etc/exabgp/exabgp.conf'
05:34:25         78:  
05:34:25         79:    service do
05:34:25         80:      environment 'exabgp_daemon_daemonize' => 'false'
05:34:25         81:      exec_start '/usr/src/exabgp/sbin/exabgp /etc/exabgp/exabgp.conf'
05:34:25         82:      exec_reload '/bin/kill -s USR1 $MAINPID'
05:34:25         83:      user 'nobody'
05:34:25         84:    end

@ernestas-vinted
Copy link

maybe lock systemd version as well to prevent future failures

 - Bump cookbook version to `0.3.0`
 - Lock `systemd` version to `~> 3.2.2`
 - Move `description`, `after` and `condition_path_exists` under
   `unit` section
@dmitry-g dmitry-g force-pushed the fix/systemd_service_unit branch from 686b249 to b8c4513 Compare April 12, 2018 10:38
@dmitry-g dmitry-g merged commit 19e4919 into vinted:master Apr 12, 2018
@dmitry-g dmitry-g deleted the fix/systemd_service_unit branch April 12, 2018 10:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants