Juju Charms Collection
elasticsearch package

Code review comment for lp:~michael.nelson/charms/trusty/elasticsearch/ufw-for-peers-too

  1. Trusty Tahr (14.04)
  2. ufw-for-peers-too
  3. Merge into trunk
Revision history for this message
Matt Bruzek (mbruzek) wrote :

I tested this several different ways and found it to work as described. Thank you Michael for providing the pastebin outline for tests.

I did deploy 2 nodes of elastic search before the ufw fix verified they did not have ufw rules and then upgraded-charm to the latest version. The upgrade went well and worked as expected.

I also tested with an ELK relations. Adding and destroying units of elastic search.

This change looks good to me. +1

review: Approve

« Back to merge proposal