Skip to content

Commit

Permalink
Merge pull request #85 from ernestas-poskus/features/require_ansible_2_4
Browse files Browse the repository at this point in the history
Require ansible 2.4
  • Loading branch information
ernestas-poskus authored May 8, 2018
2 parents 87bbce5 + af62793 commit 9dd39a9
Show file tree
Hide file tree
Showing 7 changed files with 17 additions and 17 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Role Variables

```yaml
---
zookeeper_version: 3.4.11
zookeeper_version: 3.4.12
zookeeper_url: http://www.us.apache.org/dist/zookeeper/zookeeper-{{zookeeper_version}}/zookeeper-{{zookeeper_version}}.tar.gz

# Flag that selects if systemd or upstart will be used for the init service:
Expand Down
2 changes: 1 addition & 1 deletion defaults/main.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
zookeeper_version: 3.4.11
zookeeper_version: 3.4.12
zookeeper_url: http://www.us.apache.org/dist/zookeeper/zookeeper-{{zookeeper_version}}/zookeeper-{{zookeeper_version}}.tar.gz

# Flag that selects if systemd or upstart will be used for the init service:
Expand Down
6 changes: 3 additions & 3 deletions meta/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ galaxy_info:
description: Ansible Zookeeper Role
company: http://michaelhamrah.com
license: Apache 2
min_ansible_version: 1.6
min_ansible_version: 2.4
#
# Below are all platforms currently available. Just uncomment
# the ones that apply to your role. If you don't see your
# the ones that apply to your role. If you don't see your
# platform on this list, let us know and we'll get it added!
#
platforms:
Expand Down Expand Up @@ -113,4 +113,4 @@ dependencies: []
# dependencies available via galaxy should be listed here.
# Be sure to remove the '[]' above if you add dependencies
# to this list.

6 changes: 3 additions & 3 deletions tasks/Debian-tarball.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
---
- include: tarball.yml
- include_tasks: tarball.yml

- include: upstart.yml
- include_tasks: upstart.yml
when: not zookeeper_debian_systemd_enabled
tags: deploy

- include: systemd.yml
- include_tasks: systemd.yml
when: zookeeper_debian_systemd_enabled
tags: deploy
6 changes: 3 additions & 3 deletions tasks/RedHat.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@
with_items:
- libselinux-python

- include: tarball.yml
- include_tasks: tarball.yml

- include: upstart.yml
- include_tasks: upstart.yml
tags: deploy

- include: systemd.yml
- include_tasks: systemd.yml
tags: deploy
8 changes: 4 additions & 4 deletions tasks/main.yml
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
---

- include: Debian.yml
- include_tasks: Debian.yml
when: ansible_os_family == 'Debian' and zookeeper_debian_apt_install

- include: Debian-tarball.yml
- include_tasks: Debian-tarball.yml
when: ansible_os_family == 'Debian' and not zookeeper_debian_apt_install

- include: RedHat.yml
- include_tasks: RedHat.yml
when: ansible_os_family == 'RedHat'

- include: common-config.yml
- include_tasks: common-config.yml
4 changes: 2 additions & 2 deletions tests/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@
remote_user: root
roles:
- role: role_under_test
zookeeper_version: 3.4.11
zookeeper_version: 3.4.12
zookeeper_dir: /opt/zookeeper-{{zookeeper_version}}
zookeeper_hosts:
- host: "{{zookeeper_hosts_hostname}}" # the machine running
Expand All @@ -76,7 +76,7 @@
connection: local
remote_user: root
vars:
zookeeper_version: 3.4.11
zookeeper_version: 3.4.12
zookeeper_dir: /opt/zookeeper-{{zookeeper_version}}
zookeeper_cluster_ports: "2888:3888"
tasks:
Expand Down

0 comments on commit 9dd39a9

Please sign in to comment.