Merge lp:~mbruzek/charms/trusty/logstash/matt_fixes into lp:~lazypower/charms/trusty/logstash/trunk

Proposed by Matt Bruzek on 2014-08-18
Status: Merged
Merged at revision: 19
Proposed branch: lp:~mbruzek/charms/trusty/logstash/matt_fixes
Merge into: lp:~lazypower/charms/trusty/logstash/trunk
Diff against target: 20 lines (+3/-3)
1 file modified
config.yaml (+3/-3)
To merge this branch: bzr merge lp:~mbruzek/charms/trusty/logstash/matt_fixes
Reviewer Review Type Date Requested Status
Charles Butler 2014-08-18 Approve on 2014-08-19
Review via email:

Description of the change

Fixed some stuff for you in the config.yaml

To post a comment you must log in.
Charles Butler (lazypower) wrote :

+1 Thanks for the detail Matt!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config.yaml'
2--- config.yaml 2014-07-21 03:35:42 +0000
3+++ config.yaml 2014-08-18 20:10:02 +0000
4@@ -2,12 +2,12 @@
5 logstash-source:
6 type: string
7 default: ""
8- description: "A short description of the configuration option"
9+ description: The logstash binary file to install to this charm.
10 logstash-sum:
11 type: string
12 default: "d59ef579c7614c5df9bd69cfdce20ed371f728ff"
13- description: Logstash Checksum
14+ description: The checksum value for the logstash file.
15 extra-packages:
16 type: string
17 default: ''
18- description: Space separated list of extra apt packages to install. vim byobu
19\ No newline at end of file
20+ description: Space separated list of extra apt packages to install.


People subscribed via source and target branches

to all changes: