From ed55c796bf1afd8de8ac64d46ee3860fa6387bdf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9mi=20-=20Le=20Filament?= <remi@le-filament.com>
Date: Wed, 5 Jul 2023 15:20:50 +0200
Subject: [PATCH] [UPD] ansible-lint

---
 .ansible-lint     |  7 ++++++
 .yamllint         | 39 +++++++++++++++++++++++++++++
 handlers/main.yml | 10 ++++----
 meta/main.yml     | 24 +++++++++---------
 tasks/main.yml    | 63 +++++++++++++++++++++++------------------------
 5 files changed, 94 insertions(+), 49 deletions(-)
 create mode 100644 .ansible-lint
 create mode 100644 .yamllint

diff --git a/.ansible-lint b/.ansible-lint
new file mode 100644
index 0000000..8d40d06
--- /dev/null
+++ b/.ansible-lint
@@ -0,0 +1,7 @@
+---
+warn_list:  # or 'skip_list' to silence them completely
+    - git-latest  # Git checkouts must contain explicit version
+    - ignore-errors  # Use failed_when and specify error conditions instead of using ignore_errors
+    - no-changed-when  # Commands should not change things if nothing needs doing
+    - no-handler  # Tasks that run when changed should likely be handlers
+    - package-latest  # Package installs should not use latest
diff --git a/.yamllint b/.yamllint
new file mode 100644
index 0000000..fbebdb8
--- /dev/null
+++ b/.yamllint
@@ -0,0 +1,39 @@
+---
+# Based on ansible-lint config
+extends: default
+
+rules:
+    braces:
+        max-spaces-inside: 1
+        level: error
+    brackets:
+        max-spaces-inside: 1
+        level: error
+    colons:
+        max-spaces-after: -1
+        level: error
+    commas:
+        max-spaces-after: -1
+        level: error
+    # comments enable
+    comments: enable
+    comments-indentation: enable
+    document-start: enable
+    empty-lines:
+        max: 3
+        level: error
+    hyphens:
+        level: error
+    indentation:
+        level: warning
+        indent-sequences: consistent
+        spaces: 4
+        check-multi-line-strings: true
+    key-duplicates: enable
+    line-length: disable
+    new-line-at-end-of-file: enable
+    new-lines:
+        type: unix
+    # trailing-spaces enable
+    trailing-spaces: enable
+    truthy: enable
diff --git a/handlers/main.yml b/handlers/main.yml
index 778b3fa..955d05c 100644
--- a/handlers/main.yml
+++ b/handlers/main.yml
@@ -1,10 +1,10 @@
 ---
 
-- name: start privatebin docker
-  docker_compose:
-    project_src: /home/docker/
-    files: privatebin.yaml
-    project_name: privatebin
+- name: Start privatebin docker
+  community.docker.docker_compose:
+      project_src: /home/docker/
+      files: privatebin.yaml
+      project_name: privatebin
   async: 120
   poll: 10
   when: not ansible_check_mode
diff --git a/meta/main.yml b/meta/main.yml
index efbf300..c1f57c4 100644
--- a/meta/main.yml
+++ b/meta/main.yml
@@ -1,14 +1,14 @@
 ---
 galaxy_info:
-  author: Rémi
-  description: Role for deploying Privatebin on Docker
-  company: Le Filament (https://le-filament.com)
-  license: AGPL-3.0-or-later
-  min_ansible_version: 2.1
-  platforms:
-    - name: Ubuntu
-      versions:
-        - bionic
-        - focal
-  galaxy_tags:
-    - framadate
+    author: lefilament
+    description: Role for deploying Privatebin on Docker
+    company: Le Filament (https://le-filament.com)
+    license: AGPL-3.0-or-later
+    min_ansible_version: "2.1"
+    platforms:
+        - name: Ubuntu
+          versions:
+              - bionic
+              - focal
+    galaxy_tags:
+        - framadate
diff --git a/tasks/main.yml b/tasks/main.yml
index 2a14b8e..abb7c76 100644
--- a/tasks/main.yml
+++ b/tasks/main.yml
@@ -2,47 +2,46 @@
 
 ## Install privatebin docker
 - name: Install privatebin docker
-  template:
-    src: privatebin.yaml.j2
-    dest: /home/docker/privatebin.yaml
-    owner: root
-    group: root
-    mode: '0400'
+  ansible.builtin.template:
+      src: privatebin.yaml.j2
+      dest: /home/docker/privatebin.yaml
+      owner: root
+      group: root
+      mode: '0400'
   notify:
-    - start privatebin docker
+      - Start privatebin docker
 
 ## Backup Privatebin section
 - name: Install privatebin backup compose file
-  template:
-    src: backup.yaml.j2
-    dest: /home/docker/backups/backup-privatebin.yaml
-    owner: root
-    group: root
-    mode: '0400'
+  ansible.builtin.template:
+      src: backup.yaml.j2
+      dest: /home/docker/backups/backup-privatebin.yaml
+      owner: root
+      group: root
+      mode: '0400'
   when: inventory_hostname in groups['maintenance_contract']
 
 - name: Install privatebin backup2 compose file
-  template:
-    src: backup2.yaml.j2
-    dest: /home/docker/backups/backup2-privatebin.yaml
-    owner: root
-    group: root
-    mode: '0400'
+  ansible.builtin.template:
+      src: backup2.yaml.j2
+      dest: /home/docker/backups/backup2-privatebin.yaml
+      owner: root
+      group: root
+      mode: '0400'
   when: inventory_hostname in groups['maintenance_contract']
 
-
-- name: add cron job to run backup every day
-  cron:
-    name: backup privatebin
-    minute: "25"
-    hour: "04"
-    job: /usr/bin/docker-compose -f /home/docker/backups/backup-privatebin.yaml run --rm backup_privatebin
+- name: Add cron job to run backup every day
+  ansible.builtin.cron:
+      name: backup privatebin
+      minute: "25"
+      hour: "04"
+      job: /usr/bin/docker-compose -f /home/docker/backups/backup-privatebin.yaml run --rm backup_privatebin
   when: inventory_hostname in groups['maintenance_contract']
 
-- name: add cron job to run backup2 every day
-  cron:
-    name: backup2 privatebin
-    minute: "35"
-    hour: "01"
-    job: /usr/bin/docker-compose -f /home/docker/backups/backup2-privatebin.yaml run --rm backup_privatebin
+- name: Add cron job to run backup2 every day
+  ansible.builtin.cron:
+      name: backup2 privatebin
+      minute: "35"
+      hour: "01"
+      job: /usr/bin/docker-compose -f /home/docker/backups/backup2-privatebin.yaml run --rm backup_privatebin
   when: inventory_hostname in groups['maintenance_contract']
-- 
GitLab