Merge lp://staging/~ricardokirkner/charms/trusty/logstash/logstash2-extra-plugins into lp://staging/~canonical-is-sa/charms/trusty/logstash/logstash2

Proposed by Ricardo Kirkner
Status: Merged
Approved by: Alexandre Gomes
Approved revision: 75
Merged at revision: 69
Proposed branch: lp://staging/~ricardokirkner/charms/trusty/logstash/logstash2-extra-plugins
Merge into: lp://staging/~canonical-is-sa/charms/trusty/logstash/logstash2
Diff against target: 135 lines (+86/-3)
2 files modified
config.yaml (+10/-0)
hooks/config-changed (+76/-3)
To merge this branch: bzr merge lp://staging/~ricardokirkner/charms/trusty/logstash/logstash2-extra-plugins
Reviewer Review Type Date Requested Status
Alexandre Gomes Approve
Guillermo Gonzalez (community) Approve
Review via email: mp+336316@code.staging.launchpad.net

Commit message

added support for installing extra plugins

update logstash plugins during config-changed hook to allow for updating after installed

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

+1

review: Approve
Revision history for this message
Alexandre Gomes (alejdg) wrote :

+1

review: Approve
74. By Ricardo Kirkner

added workaround to avoid uninstall process from hanging

75. By Ricardo Kirkner

fixed typo

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches