Commit c64535f2 authored by Maciej Delmanowski's avatar Maciej Delmanowski

Fix ansible-lint error E601 in DebOps roles

parent b8f99cc9
......@@ -50,7 +50,7 @@
extra_args: '--user --upgrade'
become: True
become_user: '{{ homeassistant__user }}'
when: homeassistant__virtualenv == False and homeassistant__register_git is changed
when: not homeassistant__virtualenv|bool and homeassistant__register_git is changed
notify: [ 'Restart Home Assistant' ]
- name: Install hass in virtualenv
......@@ -62,7 +62,7 @@
virtualenv_python: 'python3'
become: True
become_user: '{{ homeassistant__user }}'
when: homeassistant__virtualenv == True and homeassistant__register_git is changed
when: homeassistant__virtualenv|bool and homeassistant__register_git is changed
notify: [ 'Restart Home Assistant' ]
- name: Ensure Home Assistant config dir exists
......
......@@ -70,7 +70,7 @@
group: '{{ etherpad_group }}'
mode: '0755'
when: (etherpad_register_source is defined and etherpad_register_source is changed) or
(etherpad_register_directory is defined and etherpad_register_directory.stat.exists == False)
(etherpad_register_directory is defined and not etherpad_register_directory.stat.exists|bool)
tags: [ 'role::etherpad:source' ]
- name: Prepare Etherpad worktree
......@@ -81,7 +81,7 @@
group: '{{ etherpad_group }}'
mode: '0644'
when: (etherpad_register_source is defined and etherpad_register_source is changed) or
(etherpad_register_directory is defined and etherpad_register_directory.stat.exists == False)
(etherpad_register_directory is defined and not etherpad_register_directory.stat.exists|bool)
tags: [ 'role::etherpad:source' ]
- name: Checkout Etherpad
......@@ -95,7 +95,7 @@
register: etherpad_register_checkout
notify: [ 'Restart etherpad-lite' ]
when: (etherpad_register_source is defined and etherpad_register_source is changed) or
(etherpad_register_directory is defined and etherpad_register_directory.stat.exists == False)
(etherpad_register_directory is defined and not etherpad_register_directory.stat.exists|bool)
tags: [ 'role::etherpad:source' ]
## Not used anywhere but it should stay for now: See https://github.com/debops/ansible-etherpad/pull/13
......
......@@ -23,4 +23,4 @@
- '{{ gitusers_host_list }}'
when: ((item.name is defined and item.name) and
(item.state is undefined or (item.state is defined and item.state != 'absent')) and
(item.sshkeys is defined and item.sshkeys == False))
(item.sshkeys is defined and not item.sshkeys|bool))
......@@ -99,7 +99,7 @@
'--database-host={{ owncloud__database_map[owncloud__database].dbhost }}'
'--database-user={{ owncloud__database_map[owncloud__database].dbuser }}'
'--database-pass={{ owncloud__database_map[owncloud__database].dbpass }}'
{% if owncloud__admin_username != False %}
{% if owncloud__admin_username %}
'--admin-user={{ owncloud__admin_username }}'
'--admin-pass={{ owncloud__admin_password }}'
{% endif %}
......
......@@ -25,4 +25,4 @@
- '{{ sftpusers_host_list }}'
when: ((item.name is defined and item.name) and
(item.state is undefined or (item.state is defined and item.state != 'absent')) and
(item.forward is defined and item.forward == False))
(item.forward is defined and not item.forward|bool))
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment