Merge lp:~cprov/charms/trusty/logstash/ls-restart-config-changes into lp:~tanuki/charms/trusty/logstash/trunk

Proposed by Celso Providelo
Status: Merged
Merged at revision: 51
Proposed branch: lp:~cprov/charms/trusty/logstash/ls-restart-config-changes
Merge into: lp:~tanuki/charms/trusty/logstash/trunk
Diff against target: 14 lines (+4/-0)
1 file modified
hooks/config-changed (+4/-0)
To merge this branch: bzr merge lp:~cprov/charms/trusty/logstash/ls-restart-config-changes
Reviewer Review Type Date Requested Status
Guillermo Gonzalez Approve
Review via email: mp+268074@code.launchpad.net

Commit message

Restart services on configuration changes.

Description of the change

Restart services on configuration changes.

To post a comment you must log in.
Revision history for this message
Guillermo Gonzalez (verterok) wrote :

LGTM, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hooks/config-changed'
--- hooks/config-changed 2015-08-13 19:44:37 +0000
+++ hooks/config-changed 2015-08-14 13:09:21 +0000
@@ -45,6 +45,10 @@
45 # extracted45 # extracted
46 # subprocess.check_output(shlex.split('hooks/install'))46 # subprocess.check_output(shlex.split('hooks/install'))
4747
48 # Restart the service when configuration has changed.
49 import start
50 start.start()
51
4852
49def copy_config():53def copy_config():
50 files = os.listdir('templates')54 files = os.listdir('templates')

Subscribers

People subscribed via source and target branches

to all changes: