Merge lp:~thomir-deactivatedaccount/charms/trusty/logstash/trunk-expose-tcp into lp:~tanuki/charms/trusty/logstash/trunk

Proposed by Thomi Richards on 2015-09-01
Status: Merged
Approved by: Thomi Richards on 2015-09-01
Approved revision: 54
Merged at revision: 54
Proposed branch: lp:~thomir-deactivatedaccount/charms/trusty/logstash/trunk-expose-tcp
Merge into: lp:~tanuki/charms/trusty/logstash/trunk
Diff against target: 11 lines (+1/-0)
1 file modified
hooks/config-changed (+1/-0)
To merge this branch: bzr merge lp:~thomir-deactivatedaccount/charms/trusty/logstash/trunk-expose-tcp
Reviewer Review Type Date Requested Status
Guillermo Gonzalez 2015-09-01 Approve on 2015-09-01
Review via email: mp+269815@code.launchpad.net

Commit message

Expose tcp port 5959 along with the UDP port.

Description of the change

Expose tcp port 5959 along with the UDP port.

To post a comment you must log in.
Guillermo Gonzalez (verterok) wrote :

+1

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-14 17:52:22 +0000
3+++ hooks/config-changed 2015-09-01 22:04:39 +0000
4@@ -39,6 +39,7 @@
5 # This only actually opens the port if we've exposed the service in juju
6 hookenv.open_port(5043)
7 hookenv.open_port(5959, protocol='UDP')
8+ hookenv.open_port(5959, protocol='TCP')
9
10 # The install hook is idempotent, so re-run it.
11 # XXX: no, it's not idempotent. it fails if the tarball was already

Subscribers

People subscribed via source and target branches

to all changes: