Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support custom zookeeper debian apt repository (to use v3.4.8 on Ubuntu 14.04 #58

Conversation

lhoss
Copy link

@lhoss lhoss commented Jan 5, 2017

by allowing to configure a custom (ppa) apt repos, we can use a community provided v3.4.8 deb-pkg based deployment (which is more secure than using 'tarball' , when later installing mesos which by default has 'zookeeper' in its dependencies).

Example config to make this work:

      zookeeper_debian_apt_repository: "ppa:ufscar/zookeeper"
      zookeeper_debian_apt_install: true
      zookeeper_version: 3.4.8

@ernestas-poskus
Copy link
Member

@lhoss please update README as well and you can include your example

@lhoss
Copy link
Author

lhoss commented Jan 6, 2017

@ernestas-poskus done including a more flexible way to define different repositories for different distribution versions (resp. this properly limits to add the repository only for the supported distro, as for ex. to deploy v3.4.8 on Ubuntu 16.04 does not require any additional repo)

@lhoss
Copy link
Author

lhoss commented Jan 6, 2017

ps: I also bumped (long time outdated) zookeeper_playbook_version , but I guess both following vars you can simply remove from the defaults ( and the README)

ansible_playbook_version: 0.1
zookeeper_playbook_version: "0.17.0"

@lhoss lhoss force-pushed the support_custom_zookeeper_debian_apt_repository branch from 966adbd to 37611ce Compare January 6, 2017 10:52
@lhoss lhoss force-pushed the support_custom_zookeeper_debian_apt_repository branch from 37611ce to 8dd152c Compare January 6, 2017 10:56
@ernestas-poskus
Copy link
Member

Will remove this zookeeper_playbook_version var

@ernestas-poskus
Copy link
Member

Thank you for contribution 👍

@ernestas-poskus ernestas-poskus merged commit 844e131 into AnsibleShipyard:master Jan 6, 2017
@lhoss lhoss deleted the support_custom_zookeeper_debian_apt_repository branch January 6, 2017 13:57
@lhoss lhoss restored the support_custom_zookeeper_debian_apt_repository branch January 6, 2017 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants