Juju Charms Collection
logstash package

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

  1. Trusty Tahr (14.04)
  2. trunk-expose-tcp
  3. Merge into trunk
Proposed by Thomi Richards
Status: Merged
Approved by: Thomi Richards
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 Approve
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.
Revision history for this message
Guillermo Gonzalez (verterok) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hooks/config-changed'
--- hooks/config-changed 2015-08-14 17:52:22 +0000
+++ hooks/config-changed 2015-09-01 22:04:39 +0000
@@ -39,6 +39,7 @@
39 # This only actually opens the port if we've exposed the service in juju39 # This only actually opens the port if we've exposed the service in juju
40 hookenv.open_port(5043)40 hookenv.open_port(5043)
41 hookenv.open_port(5959, protocol='UDP')41 hookenv.open_port(5959, protocol='UDP')
42 hookenv.open_port(5959, protocol='TCP')
4243
43 # The install hook is idempotent, so re-run it.44 # The install hook is idempotent, so re-run it.
44 # XXX: no, it's not idempotent. it fails if the tarball was already45 # XXX: no, it's not idempotent. it fails if the tarball was already

Subscribers

People subscribed via source and target branches

to all changes: