Fix ansible-lint violations
Ansible Lint |Total|New|Outstanding|Fixed|Trend |:-:|:-:|:-:|:-:|:-: |2768|3|2765|12|:+1: Reference build: <a href="https://jenkins.evolix.org/job/gitea/job/ansible-roles/job/unstable/224//ansiblelint">Evolix » ansible-roles » unstable #224</a> Details
gitea/ansible-roles/pipeline/head This commit looks good Details

This commit is contained in:
Jérémy Lecour 2023-03-20 23:48:40 +01:00 committed by Jérémy Lecour
parent ee21973371
commit 70c93310f9
9 changed files with 13 additions and 16 deletions

View File

@ -27,4 +27,4 @@ galaxy_info:
allow_duplicates: yes allow_duplicates: yes
dependencies: [] dependencies: []

View File

@ -11,7 +11,7 @@
# - ansible_distribution_release == "bookworm" # - ansible_distribution_release == "bookworm"
# - mongodb_version is version('5.0', '<') # - mongodb_version is version('5.0', '<')
- name: Add MongoDB repository - name: Add MongoDB repository
ansible.builtin.template: ansible.builtin.template:
src: mongodb.sources.j2 src: mongodb.sources.j2
dest: /etc/apt/sources.list.d/mongodb.sources dest: /etc/apt/sources.list.d/mongodb.sources

View File

@ -3,8 +3,7 @@
- name: "Set php version to 8.2 (Debian 12)" - name: "Set php version to 8.2 (Debian 12)"
ansible.builtin.set_fact: ansible.builtin.set_fact:
php_version: "8.2" php_version: "8.2"
when: when: not (php_sury_enable | bool)
- php_sury_enable == false
check_mode: no check_mode: no
- name: "Set php config directories (Debian 12)" - name: "Set php config directories (Debian 12)"

View File

@ -3,8 +3,7 @@
- name: "Set php version to 7.4 if Sury repo is not enabled" - name: "Set php version to 7.4 if Sury repo is not enabled"
ansible.builtin.set_fact: ansible.builtin.set_fact:
php_version: "7.4" php_version: "7.4"
when: when: not (php_sury_enable | bool)
- php_sury_enable == False
check_mode: no check_mode: no
- name: "Set variables (Debian 11)" - name: "Set variables (Debian 11)"

View File

@ -7,8 +7,7 @@
ansible.builtin.set_fact: ansible.builtin.set_fact:
php_version: "7.3" php_version: "7.3"
check_mode: no check_mode: no
when: when: not (php_sury_enable | bool)
- not (php_sury_enable | bool)
- name: "Set variables (Debian 10)" - name: "Set variables (Debian 10)"
ansible.builtin.set_fact: ansible.builtin.set_fact:

View File

@ -25,8 +25,8 @@ galaxy_info:
# alphanumeric characters. Maximum 20 tags per role. # alphanumeric characters. Maximum 20 tags per role.
dependencies: dependencies:
- { role: evolix/ldap, ldap_schema: 'cn4evolix.ldif', when: postfix_packmail == True } - { role: evolix/ldap, ldap_schema: 'cn4evolix.ldif', when: postfix_packmail | bool }
- { role: evolix/spamassasin, when: postfix_packmail == True } - { role: evolix/spamassasin, when: postfix_packmail | bool }
- { role: evolix/clamav, when: postfix_packmail == True } - { role: evolix/clamav, when: postfix_packmail | bool }
- { role: evolix/opendkim, when: postfix_packmail == True } - { role: evolix/opendkim, when: postfix_packmail | bool }
- { role: evolix/dovecot, when: postfix_packmail == True } - { role: evolix/dovecot, when: postfix_packmail | bool }

View File

@ -21,7 +21,7 @@
- name: make sure a service Mailgraph is running - name: make sure a service Mailgraph is running
ansible.builtin.systemd: ansible.builtin.systemd:
name: mailgraph.service name: mailgraph.service
state: started state: started
enabled: true enabled: true
- name: create packmail main.cf - name: create packmail main.cf

View File

@ -137,7 +137,7 @@
dest: "{{ varnish_config_file }}" dest: "{{ varnish_config_file }}"
mode: "0644" mode: "0644"
force: yes force: yes
when: "{{ varnish_update_config }}" when: varnish_update_config | bool
loop: "{{ query('first_found', templates) }}" loop: "{{ query('first_found', templates) }}"
vars: vars:
templates: templates:

View File

@ -36,7 +36,7 @@
chdir: "{{ nextcloud_webroot }}" chdir: "{{ nextcloud_webroot }}"
creates: "{{ nextcloud_home }}/config/config.php" creates: "{{ nextcloud_home }}/config/config.php"
become_user: "{{ nextcloud_user }}" become_user: "{{ nextcloud_user }}"
when: (nc_status.stdout | from_json).installed == false when: not ((nc_status.stdout | from_json).installed | bool)
tags: tags:
- nextcloud - nextcloud