From afcea9439c078c9736eba6deabfb91c5d3bc6ed0 Mon Sep 17 00:00:00 2001
From: transcaffeine <transcaffeine@finally.coffee>
Date: Wed, 4 Feb 2026 11:53:40 +0100
Subject: [PATCH] fix: various ansible-lint failures (jinja2 spacing, yaml
 octal values)

---
 playbooks/docker_registry.yml           | 2 +-
 playbooks/lego_certificate.yml          | 2 +-
 roles/git/tasks/main.yml                | 2 +-
 roles/lego/vars/main.yml                | 4 ++--
 roles/minio/defaults/main/container.yml | 2 +-
 roles/minio/tasks/deploy-docker.yml     | 6 +++---
 roles/nginx/tasks/main.yml              | 2 +-
 roles/openldap/tasks/configure.yml      | 2 +-
 8 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/playbooks/docker_registry.yml b/playbooks/docker_registry.yml
index 2dd62cf..c8782e5 100644
--- a/playbooks/docker_registry.yml
+++ b/playbooks/docker_registry.yml
@@ -13,4 +13,4 @@
       loop: "{{ docker_registries | default([], true) }}"
       loop_control:
         loop_var: "docker_registry"
-        label: "{{ docker_registry.username}}@{{ docker_registry.registry }}"
+        label: "{{ docker_registry.username }}@{{ docker_registry.registry }}"
diff --git a/playbooks/lego_certificate.yml b/playbooks/lego_certificate.yml
index 7ae3aaa..325d0e0 100644
--- a/playbooks/lego_certificate.yml
+++ b/playbooks/lego_certificate.yml
@@ -6,7 +6,7 @@
   pre_tasks:
     - name: Build target dns records
       ansible.builtin.set_fact:
-        target_dns_records: "{{ target_dns_records + [ _dns_record ] }}"
+        target_dns_records: "{{ target_dns_records + [_dns_record] }}"
       vars:
         _dns_record:
           type: "CNAME"
diff --git a/roles/git/tasks/main.yml b/roles/git/tasks/main.yml
index beca778..d9269af 100644
--- a/roles/git/tasks/main.yml
+++ b/roles/git/tasks/main.yml
@@ -8,7 +8,7 @@
     state: present
     marker: "#{mark} ANSIBLE MANAGED BLOCK by finallycoffee.base.git"
     block: |+2
-      {% if git_config_user_name|default(false, true) and git_config_user_email|default(false, true) %}
+      {% if git_config_user_name | default(false, true) and git_config_user_email | default(false, true) %}
       [user]
         name = {{ git_config_user_name }}
         email = {{ git_config_user_email }}
diff --git a/roles/lego/vars/main.yml b/roles/lego/vars/main.yml
index c962c8b..7d232ba 100644
--- a/roles/lego/vars/main.yml
+++ b/roles/lego/vars/main.yml
@@ -1,11 +1,11 @@
 ---
 
-lego_domain_command_args: >-
+lego_domain_command_args: >-2
   {% for domain in lego_cert_domains %}
    --domains={{ domain }}
   {%- endfor %}
 
-lego_config_command_args: >-
+lego_config_command_args: >-2
   {% for key in lego_full_command_config %}
    --{{ key | replace("_", "-") }}
   {%- if lego_full_command_config[key] != None and lego_full_command_config[key] != '' -%}
diff --git a/roles/minio/defaults/main/container.yml b/roles/minio/defaults/main/container.yml
index a74823d..62d0289 100644
--- a/roles/minio/defaults/main/container.yml
+++ b/roles/minio/defaults/main/container.yml
@@ -23,7 +23,7 @@ minio_container_command:
   - "--console-address"
   - ":{{ minio_container_listen_port_console }}"
 minio_container_restart_policy: "unless-stopped"
-minio_container_image_force_source: "{{ (minio_container_image_tag == 'latest')|bool }}"
+minio_container_image_force_source: "{{ (minio_container_image_tag == 'latest') | bool }}"
 minio_container_state: >-2
   {{ (minio_state == 'present') | ternary('started', 'absent') }}
 
diff --git a/roles/minio/tasks/deploy-docker.yml b/roles/minio/tasks/deploy-docker.yml
index 04e86cc..662621c 100644
--- a/roles/minio/tasks/deploy-docker.yml
+++ b/roles/minio/tasks/deploy-docker.yml
@@ -3,15 +3,15 @@
   ansible.builtin.file:
     path: "{{ minio_data_path }}"
     state: directory
-    user: "{{ minio_user|default(omit, True) }}"
-    group: "{{ minio_user|default(omit, True) }}"
+    user: "{{ minio_user | default(omit, True) }}"
+    group: "{{ minio_user| default(omit, True) }}"
   when: minio_manage_host_filesystem
 
 - name: Ensure container image '{{ minio_container_image }}' is {{ minio_state }}
   community.docker.docker_image:
     name: "{{ minio_container_image }}"
     state: "{{ minio_state }}"
-    source: pull
+    source: "pull"
     force_source: "{{ minio_container_image_force_source }}"
 
 - name: Ensure container '{{ minio_container_name }}' is {{ minio_container_state }}
diff --git a/roles/nginx/tasks/main.yml b/roles/nginx/tasks/main.yml
index 52463a8..49fc216 100644
--- a/roles/nginx/tasks/main.yml
+++ b/roles/nginx/tasks/main.yml
@@ -30,7 +30,7 @@
   ansible.builtin.copy:
     dest: "{{ nginx_config_file }}"
     content: "{{ nginx_config }}"
-    mode: 0640
+    mode: "0640"
   notify:
     - restart-nginx
   when: nginx_state == 'present'
diff --git a/roles/openldap/tasks/configure.yml b/roles/openldap/tasks/configure.yml
index 4a944ea..491856a 100644
--- a/roles/openldap/tasks/configure.yml
+++ b/roles/openldap/tasks/configure.yml
@@ -13,7 +13,7 @@
 - name: Ensure config db attributes are configured
   community.general.ldap_attrs:
     dn: "{{ openldap_config_db_dn }}"
-    attributes: "{{ { entry.key : entry.value } }}"
+    attributes: "{{ { entry.key: entry.value } }}"
     state: exact
     server_uri: "{{ openldap_socket_url }}"
   loop: "{{ openldap_config_db_attributes | dict2items }}"