Merge lp:~lazypower/charms/trusty/elasticsearch/open_status_port into lp:~charmers/charms/trusty/elasticsearch/trunk

Proposed by Charles Butler
Status: Merged
Merged at revision: 32
Proposed branch: lp:~lazypower/charms/trusty/elasticsearch/open_status_port
Merge into: lp:~charmers/charms/trusty/elasticsearch/trunk
Diff against target: 15 lines (+5/-0)
1 file modified
playbook.yaml (+5/-0)
To merge this branch: bzr merge lp:~lazypower/charms/trusty/elasticsearch/open_status_port
Reviewer Review Type Date Requested Status
Marco Ceppi (community) Approve
Review via email: mp+227086@code.launchpad.net

Description of the change

Adds port 9200 to the open-port block of the start hook for querying the cluster status when the ES cluster is exposed.

To post a comment you must log in.
Revision history for this message
Jorge Castro (jorge) wrote :

I've tested this and it works on AWS. (In progress of testing others)

Revision history for this message
Marco Ceppi (marcoceppi) wrote :

LGTM, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'playbook.yaml'
2--- playbook.yaml 2014-06-06 14:40:08 +0000
3+++ playbook.yaml 2014-07-16 18:39:50 +0000
4@@ -25,6 +25,11 @@
5 notify:
6 - Restart ElasticSearch
7
8+ - name: Open ES Port when exposed
9+ command: open-port 9200
10+ tags:
11+ - start
12+
13 - name: Start ElasticSearch
14 service: name=elasticsearch state=started
15 tags:

Subscribers

People subscribed via source and target branches