Merge lp:~canonical-is-sa/charms/trusty/kibana/trunk into lp:charms/trusty/kibana
Proposed by
Michael Foley
Status: | Rejected |
---|---|
Rejected by: | Michael Foley |
Proposed branch: | lp:~canonical-is-sa/charms/trusty/kibana/trunk |
Merge into: | lp:charms/trusty/kibana |
Diff against target: |
82 lines (+13/-10) 4 files modified
config.yaml (+4/-0) files/charm/config.js (+1/-1) hooks/install (+8/-0) hooks/web-relation-joined (+0/-9) |
To merge this branch: | bzr merge lp:~canonical-is-sa/charms/trusty/kibana/trunk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Foley (community) | superseded | Disapprove | |
Andrew McLeod (community) | Needs Fixing | ||
charmers | Pending | ||
Adam Israel | Pending | ||
Review via email: mp+276221@code.launchpad.net |
This proposal supersedes a proposal from 2015-05-25.
Description of the change
Added ability to change the elasticsearch url for kibana so that it can be set to https. This is so that kibana can be secured behind an apache2 proxy with openid.
Also added support for ubuntu basenode.
To post a comment you must log in.
Hi Michael,
I had a chance to review this merge proposal today. Thanks for your work on improving the kibana charm!
`charm proof` alerted me to 'hooks/ website- relation- joined' not being executable. That's not a blocker, as juju should do the right thing, but is an easy fix.
With a new feature such as this, I look for two things that I'm not seeing: some documentation about the new feature in the README, and a test to exercise it. If you could address both of those issues, I'll happily approve this merge.
Thanks once again for your work on this!