Code review comment for lp:~canonical-is-sa/charms/trusty/kibana/trunk

Revision history for this message
Michael Nelson (michael.nelson) wrote :

If you mean that tests/11-scale-elastic.py fails, that would be a good thing IMO, as we updated our branch here to explicitly deny requests to :9200 on the kibana unit (as it's basically giving anyone with access to :80 on the kibana unit write access to the ES cluster).

Anyway, I've re-merged upstream, and updated 11-scale-elastic.py so it doesn't expect to be able to use :9200 on the kibana unit externally, in our kibana4 branch here:
https://code.launchpad.net/~canonical-is-sa/charms/trusty/kibana/kibana4/+merge/276944

I think this MP should be marked superseded. Michael F?

« Back to merge proposal