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

Proposed by Celso Providelo on 2015-08-14
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 2015-08-14 Approve on 2015-08-14
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.
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
1=== modified file 'hooks/config-changed'
2--- hooks/config-changed 2015-08-13 19:44:37 +0000
3+++ hooks/config-changed 2015-08-14 13:09:21 +0000
4@@ -45,6 +45,10 @@
5 # extracted
6 # subprocess.check_output(shlex.split('hooks/install'))
7
8+ # Restart the service when configuration has changed.
9+ import start
10+ start.start()
11+
12
13 def copy_config():
14 files = os.listdir('templates')

Subscribers

People subscribed via source and target branches

to all changes: