Merge lp:~bigdata-dev/charms/trusty/hdp-zookeeper/trunk into lp:charms/trusty/hdp-zookeeper
Proposed by
amir sanjar
Status: | Merged |
---|---|
Merged at revision: | 20 |
Proposed branch: | lp:~bigdata-dev/charms/trusty/hdp-zookeeper/trunk |
Merge into: | lp:charms/trusty/hdp-zookeeper |
Diff against target: |
145 lines (+31/-17) 7 files modified
README.md (+5/-0) bundles.yaml (+0/-10) config.yaml (+5/-0) hooks/bdutils.py (+4/-1) hooks/hooks.py (+5/-4) tests/10-deploy (+2/-2) tests/bundles.yaml (+10/-0) |
To merge this branch: | bzr merge lp:~bigdata-dev/charms/trusty/hdp-zookeeper/trunk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Charles Butler (community) | Approve | ||
charmers | Pending | ||
Review via email: mp+243358@code.launchpad.net |
Description of the change
adding single node support - having single node zookeeper is NOT RECOMMENDED for production
To post a comment you must log in.
Greetings Amir,
While this is a nice addition to the zookeeper charm, supporting single node deployment - it doesn't appear there was a test added to validate this configuration, nor was the readme updated to reflect the option of a single node deployment.
If you could get those added I'd be more than happy to re-review this branch for inclusion in the upstream charm.