Merge lp:~evarlast/charms/trusty/elasticsearch/add-logs-relation into lp:charms/trusty/elasticsearch
Proposed by
Jay R. Wren
Status: | Merged |
---|---|
Merged at revision: | 40 |
Proposed branch: | lp:~evarlast/charms/trusty/elasticsearch/add-logs-relation |
Merge into: | lp:charms/trusty/elasticsearch |
Diff against target: |
39 lines (+11/-0) 3 files modified
hooks/hooks.py (+1/-0) metadata.yaml (+2/-0) playbook.yaml (+8/-0) |
To merge this branch: | bzr merge lp:~evarlast/charms/trusty/elasticsearch/add-logs-relation |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Nelson (community) | Approve | ||
charmers | Pending | ||
Review via email: mp+256299@code.launchpad.net |
Description of the change
So that this charm can be related to logstash or other logger for the purpose of logging its logs, announce the log file via the logs relation.
To post a comment you must log in.
Thanks Jay. Sorry for the delay, but just ran the existing tests to ensure everything works. Would be nice to add a test of the logging relation, but not necessary right now. Cheers.