openstack/openstack-ansible-os_designate
Code Issues Proposed changes

Merge "Ansible 2.x - Address deprecation warning of bare variables"

Browse Source
This commit is contained in:
Jenkins 2016-04-21 07:29:53 +00:00 committed by Gerrit Code Review
commit 38247df459
3 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
handlers/main.yml
View File

@ -47,6 +47,6 @@
name: "{{ item }}" name: "{{ item }}"
state: "restarted" state: "restarted"
pattern: "{{ item }}" pattern: "{{ item }}"
with_items: designate_service_names with_items: "{{ designate_service_names }}"
failed_when: false failed_when: false

8
tasks/designate_install.yml
View File

@ -73,8 +73,7 @@
until: install_packages|success until: install_packages|success
retries: 5 retries: 5
delay: 2 delay: 2
with_items: with_items: "{{ designate_requires_pip_packages }}"
- "{{ designate_requires_pip_packages }}"
tags: tags:
- designate-install - designate-install
- designate-pip-packages - designate-pip-packages
@ -192,7 +191,7 @@
until: install_packages|success until: install_packages|success
retries: 5 retries: 5
delay: 2 delay: 2
with_items: designate_pip_packages with_items: "{{ designate_pip_packages }}"
when: when:
- designate_venv_enabled | bool - designate_venv_enabled | bool
- designate_get_venv | failed or designate_developer_mode | bool - designate_get_venv | failed or designate_developer_mode | bool
@ -212,8 +211,7 @@
until: install_packages|success until: install_packages|success
retries: 5 retries: 5
delay: 2 delay: 2
with_items: with_items: "{{ designate_pip_packages }}"
- "{{ designate_pip_packages }}"
when: when:
- not designate_venv_enabled | bool - not designate_venv_enabled | bool
- not designate_developer_mode | bool - not designate_developer_mode | bool

2
tasks/designate_install_apt.yml
View File

@ -33,4 +33,4 @@
until: install_packages|success until: install_packages|success
retries: 5 retries: 5
delay: 2 delay: 2
with_items: designate_apt_packages with_items: "{{ designate_apt_packages }}"