From ded2bb2953f8e445aae0dcb92a5eb80288dd7272 Mon Sep 17 00:00:00 2001 From: Ken Dreyer Date: Fri, 21 Apr 2023 10:48:14 -0400 Subject: [PATCH] use spaces around variables for ansible-lint --- roles/cobbler/tasks/settings.yml | 2 +- roles/fog-server/tasks/main.yml | 2 +- roles/testnode/tasks/cpan.yml | 2 +- roles/testnode/tasks/ntp.yml | 2 +- roles/users/tasks/filter_users.yml | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/roles/cobbler/tasks/settings.yml b/roles/cobbler/tasks/settings.yml index 8623bc21..971e55f2 100644 --- a/roles/cobbler/tasks/settings.yml +++ b/roles/cobbler/tasks/settings.yml @@ -28,5 +28,5 @@ when: users_digest is changed or dynamic_settings is changed or server_value is changed - name: Update settings - command: cobbler setting edit --name={{ item.name }} --value={{ item.value}} + command: cobbler setting edit --name={{ item.name }} --value={{ item.value }} with_items: "{{ settings }}" diff --git a/roles/fog-server/tasks/main.yml b/roles/fog-server/tasks/main.yml index a647a31f..17c4de14 100644 --- a/roles/fog-server/tasks/main.yml +++ b/roles/fog-server/tasks/main.yml @@ -9,7 +9,7 @@ - name: Ensure a path for FOG file: - path: "/home/{{ fog_user}}/fog" + path: "/home/{{ fog_user }}/fog" owner: "{{ fog_user }}" state: directory diff --git a/roles/testnode/tasks/cpan.yml b/roles/testnode/tasks/cpan.yml index 05362460..2925c8d4 100644 --- a/roles/testnode/tasks/cpan.yml +++ b/roles/testnode/tasks/cpan.yml @@ -34,7 +34,7 @@ mode: 0755 - name: Ensure perl-doc and cpanminus is installed on apt systems. - apt: name={{item}} state=present + apt: name={{ item }} state=present with_items: - cpanminus - perl-doc diff --git a/roles/testnode/tasks/ntp.yml b/roles/testnode/tasks/ntp.yml index 9ec8f403..f93bef2e 100644 --- a/roles/testnode/tasks/ntp.yml +++ b/roles/testnode/tasks/ntp.yml @@ -49,7 +49,7 @@ - name: Make sure ntpd is running. service: - name: "{{ntp_service_name}}" + name: "{{ ntp_service_name }}" enabled: yes state: started # There's an issue with ansible<=2.9 and our custom built kernels (5.8 as of this commit) where the service and systemd modules don't have backwards compatibility with init scripts diff --git a/roles/users/tasks/filter_users.yml b/roles/users/tasks/filter_users.yml index 280f1ac9..8c7124dd 100644 --- a/roles/users/tasks/filter_users.yml +++ b/roles/users/tasks/filter_users.yml @@ -18,7 +18,7 @@ # in managed_admin_users managed_users: "[{% for lab_user in managed_users -%} - {% if not managed_admin_users|selectattr('name', 'equalto', lab_user.name)|list|length %}{{ lab_user}},{% endif %} + {% if not managed_admin_users|selectattr('name', 'equalto', lab_user.name)|list|length %}{{ lab_user }},{% endif %} {%- endfor %}]" when: extra_admin_users is defined and extra_admin_users|length > 0