AnsibleShipyard / ansible-zookeeper

Ansible playbook for ZooKeeper
MIT License
95 stars 114 forks source link

Supporting RHEL 6 systems #5

Closed mhamrah closed 10 years ago

mhamrah commented 10 years ago

My first attempt was to follow the Mesosphere instructions displayed in #4 but that approach has some shortcomings. First it requires CDH repo which installs a lot of unnecessary dependencies, and the file layout and configuration override has some shortcomings when used with ansible.

Instead it's much easier to use the download-src approach and configure the init files by hand, which is what this repo is doing already. I just had to tweak some settings:

You can have:

[mesos_masters]
hostname1 zookeeper_id=1
hostname2 zookeeper_id=2

and in your role have:

 - { role: './librarian_roles/ansible-zookeeper', zookeeper_hosts: "{{ groups.mesos_masters}}", tags:['zookeeper'] }

No need to build/maintain a separate array.

Need to test on Ubuntu, but I think the su thing will work.

mhamrah commented 10 years ago

I also pulled zookeeper support from AnsibleShipyard/ansible-mesos#16 but haven't pushed.

mhamrah commented 10 years ago

This is ready-for-review, but could use some enhancements in certain areas.

mhamrah commented 10 years ago

I added the travis yml and tests from Marathon. so this should be good to merge.

JasonGiedymin commented 10 years ago

Taking a look

-Jason

On Oct 30, 2014, at 10:03 AM, Michael Hamrah notifications@github.com wrote:

I added the travis yml and tests from Marathon. so this should be good to merge.

— Reply to this email directly or view it on GitHub.

JasonGiedymin commented 10 years ago

+1