linting

.ansible-lint

@@ -0,0 +1,5 @@
+---
+exclude_paths:
+  - ./.gitignore
+  - ./library/
+  - ./roles/

.yamllint

@@ -0,0 +1,11 @@
+---
+extends: default
+
+rules:
+  # 80 chars should be enough, but don't fail if a line is longer
+  line-length:
+    max: 80
+    level: warning
+    allow-non-breakable-words: true
+  comments:
+    min-spaces-from-content: 1

bootstrap.yml

@@ -4,6 +4,6 @@
     - name: "install python and archlinux-keyring"
       raw: pacman -Sy python archlinux-keyring --noconfirm
       when: not debian.stdout
-  gather_facts: no
+  gather_facts: false
   roles:
-  - ansible_bootstrap
+    - ansible_bootstrap

build.yml

@@ -1,17 +1,19 @@
 ---
 - hosts: build
   pre_tasks:
-  - name: install terminfo
-    pacman:
-      state: present # not required. choices: absent;latest;present. Desired state of the package.
-      name: 
-        - termite-terminfo # not required. Name or list of names of the packages to install, upgrade, or remove.
-    become: yes
+    - name: install terminfo
+      pacman:
+        state: present
+        name:
+          - termite-terminfo
+      become: true
   vars:
-      #    certbot_force: true
+  # certbot_force: true
   roles:
-  - docker
-  - {role: nginx, become: yes }  
-  - {role: ansible-role-postgresql, become: yes }
-  - php
-  - user_config
+    - docker
+    - role: nginx
+      become: true
+    - role: ansible-role-postgresql
+      become: true
+    - php
+    - user_config

debian.yml

@@ -2,5 +2,5 @@
 - hosts: debian
 
   roles:
-  - syncthing
-  - msmtp
+    - syncthing
+    - msmtp

music-player.yml

@@ -1,16 +1,15 @@
+---
 - hosts: music-player
   pre_tasks:
-  - name: install terminfo
-    pacman:
-      state: present # not required. choices: absent;latest;present. Desired state of the package.
-      name: 
-        - termite-terminfo # not required. Name or list of names of the packages to install, upgrade, or remove.
-    become: yes
+    - name: install terminfo
+      pacman:
+        state: present
+        name:
+          - termite-terminfo
+      become: true
 
   roles:
-  - user_config
-  - cronie
-  - hass-client-control
-  - mpd
-
-
+    - user_config
+    - cronie
+    - hass-client-control
+    - mpd

provisionning.yml

@@ -7,7 +7,7 @@
 - hosts: all
   remote_user: root
   vars:
-      ansible_password: "{{ vault_default_root }}"
+    ansible_password: "{{ vault_default_root }}"
   roles:
     - ansible_bootstrap
 

roles/requirements.yml

@@ -1,3 +1,4 @@
+---
 - src: gitea@git.ducamps.win:ansible-roles/JDownloader.git
   scm: git
 - src: gitea@git.ducamps.win:ansible-roles/ansible-arch-provissionning.git

server.yml

@@ -1,47 +1,55 @@
 ---
 - hosts: server
   pre_tasks:
-  - name: install terminfo
-    pacman:
-      state: present # not required. choices: absent;latest;present. Desired state of the package.
-      name: 
-        - termite-terminfo # not required. Name or list of names of the packages to install, upgrade, or remove.
-    become: yes
+    - name: install terminfo
+      pacman:
+        state: present
+        name:
+          - termite-terminfo
+      become: true
   vars:
-      #    certbot_force: true
+  # certbot_force: true
   roles:
-  - role: ansible-role-chisel
-    become: yes
-  - syncthing
-  - msmtp
-  - cronie
-  - rsyncd
-  - gandi-dyn-dns
-  - fail2ban
-  - ansible-pihole
-  - {role: ansible-role-certbot, become: yes} ##need to have a external connection on 80
-  - {role: nginx, become: yes }  
-  - {role: ansible-role-postgresql, become: yes }
-  - {role: radicale , become: yes }
-  - {role: ansible-drone ,become: yes}
-  - php
-  - nextcloud
-  - tt-rss
-  - gitea
-  - supysonic
-  - hass
-  - {role: ansible-bitwardenrs , become: yes }
-  - {role: ansible-role-samba , become: yes }
-  - chainetv
-  - user_config
+    - role: ansible-role-chisel
+      become: true
+    - syncthing
+    - msmtp
+    - cronie
+    - rsyncd
+    - gandi-dyn-dns
+    - fail2ban
+    - ansible-pihole
+    - role: ansible-role-certbot
+      become: true # need to have a external connection on 80
+    - role: nginx
+      become: true
+    - role: ansible-role-postgresql
+      become: true
+    - role: radicale
+      become: true
+    - role: ansible-drone
+      become: true
+    - php
+    - nextcloud
+    - tt-rss
+    - gitea
+    - supysonic
+    - hass
+    - role: ansible-bitwardenrs
+      become: true
+    - role: ansible-role-samba
+      become: true
+    - chainetv
+    - user_config
 
   tasks:
-  - name: create paccache job for arch mirror
-    cron:
-      job: "find {{arch_mirror_location}} -type d -exec paccache -v -r -k 2 -c {} \\;"
-      user: "root"
-      name: "paccache job" 
-      hour: "4" 
-      minute: "0"
-      state: present 
-    become: true
+    - name: create paccache job for arch mirror
+      cron:
+        job: "find {{ arch_mirror_location }} -type d
+        -exec paccache -v -r -k 2 -c {} \\;"
+        user: "root"
+        name: "paccache job"
+        hour: "4"
+        minute: "0"
+        state: present
+      become: true

test-role/JDownloader.yml

@@ -1,3 +1,4 @@
+---
 - hosts: server
   roles:
     - ../roles/JDownloader

test-role/bitwardenrs.yml

@@ -1,4 +1,5 @@
+---
 - hosts: server
-  become: yes 
+  become: true
   roles:
     - ../roles/ansible-bitwardenrs

test-role/chainetv.yml

@@ -4,5 +4,4 @@
     force_site_update: true
 
   roles:
-  - ../roles/chainetv
-
+    - ../roles/chainetv

test-role/chisel.yml

@@ -1,4 +1,5 @@
+---
 - hosts: server
-  become: yes 
+  become: true
   roles:
     - ../roles/ansible-role-chisel

test-role/drone.yml

@@ -1,6 +1,7 @@
+---
 - hosts:
     - server
     - build
-  become: yes 
+  become: true
   roles:
     - ../roles/ansible-drone

test-role/gitea.yml

@@ -1,3 +1,4 @@
+---
 - hosts: server
   roles:
     - ../roles/gitea

test-role/hass.yml

@@ -1,3 +1,4 @@
+---
 - hosts: server
   roles:
     - ../roles/hass

test-role/msmtp.yml

@@ -1,3 +1,4 @@
+---
 - hosts: server
   roles:
     - ../roles/msmtp

test-role/nextcloud.yml

@@ -1,14 +1,12 @@
 ---
 - hosts: server
   pre_tasks:
-  - name: install terminfo
-    pacman:
-      state: present # not required. choices: absent;latest;present. Desired state of the package.
-      name: 
-        - termite-terminfo # not required. Name or list of names of the packages to install, upgrade, or remove.
-    become: yes
-
-
+    - name: install terminfo
+      pacman:
+        state: present
+        name:
+          - termite-terminfo
+      become: true
 
   roles:
-  - ../roles/nextcloud
+    - ../roles/nextcloud

test-role/nginx.yml

@@ -1,4 +1,5 @@
+---
 - hosts: server
-  become: yes 
+  become: true
   roles:
     - ../roles/nginx

test-role/php.yml

@@ -4,5 +4,4 @@
     force_site_update: true
 
   roles:
-  - ../roles/php
-
+    - ../roles/php

test-role/pihole.yml

@@ -1,3 +1,4 @@
+---
 - hosts: server
   vars:
     pihole_update_container: true

test-role/postgres.yml

@@ -1,4 +1,5 @@
+---
 - hosts: server
-  become: yes 
+  become: true
   roles:
     - ../roles/ansible-role-postgresql

test-role/supysonic.yml

@@ -4,5 +4,4 @@
     supysonic_force_site_update: true
 
   roles:
-  - ../roles/supysonic
-
+    - ../roles/supysonic

test-role/system.yml

@@ -1,3 +1,4 @@
+---
 - hosts: all
   roles:
     - ../roles/system

test-role/tt-rss.yml

@@ -4,6 +4,4 @@
     force_site_update: true
 
   roles:
-  - ../roles/tt-rss
-
-
+    - ../roles/tt-rss

workstation.yml

@@ -1,21 +1,20 @@
 ---
 - hosts: workstation
   roles:
-  - role: ansible-role-chisel
-    become: yes
-  - syncthing
-  - user_config
-  - ansible-manager
-  - mpd
-  - virt
-  - hass-client-control
-  - workstation
-  - cups
-  - grub_themes
-  - nvidia
+    - role: ansible-role-chisel
+      become: true
+    - syncthing
+    - user_config
+    - ansible-manager
+    - mpd
+    - virt
+    - hass-client-control
+    - workstation
+    - cups
+    - grub_themes
+    - nvidia
 - hosts: laptop
   roles:
-  - laptop
-  - bluetooth
-
-#  - Windows_manager
+    - laptop
+    - bluetooth
+    # - Windows_manager