From 51aa672d18b6da44ba381219c42486a3f9e7ee6e Mon Sep 17 00:00:00 2001
From: jurgenhaas <juergen@paragon-es.de>
Date: Mon, 10 Jun 2019 12:51:08 +0200
Subject: [PATCH] ansible-playbooks/general#85 Linting

---
 defaults/main.yml   |  4 ++--
 tasks/main.yml      | 10 ++++++----
 tasks/register.yml  | 12 ++++++------
 tasks/spideroak.yml | 20 ++++++++++----------
 4 files changed, 24 insertions(+), 22 deletions(-)

diff --git a/defaults/main.yml b/defaults/main.yml
index 413c0cd..c8850f1 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -1,3 +1,3 @@
-spideroak_username: ''
-spideroak_password: ''
+spideroak_username:
+spideroak_password:
 spideroak_include: []
diff --git a/tasks/main.yml b/tasks/main.yml
index 85d5ea3..40d4725 100644
--- a/tasks/main.yml
+++ b/tasks/main.yml
@@ -1,12 +1,14 @@
 ---
 # file: roles/spideroak/tasks/main.yml
 
-- name: "SpiderOak Role"
-  set_fact: role_spideroak_started=true
-  tags: 'always'
+- name: SpiderOak Role
+  set_fact:
+    role_spideroak_started: yes
+  tags:
+    - always
 
 - block:
 
   - import_tasks: spideroak.yml
 
-  when: '"spideroak" not in excluded_roles and spideroak_username is defined and spideroak_username != ""'
+  when: not excluded_roles or "spideroak" not in excluded_roles and spideroak_username is defined and spideroak_username != ""
diff --git a/tasks/register.yml b/tasks/register.yml
index 6cae1a9..a73cc97 100644
--- a/tasks/register.yml
+++ b/tasks/register.yml
@@ -1,13 +1,13 @@
 ---
 # file: roles/spideroak/tasks/register.yml
 
-- name: "Stop SpiderOak"
+- name: Stop SpiderOak
   service:
     name='spideroak'
     state='stopped'
     pattern='SpiderOakONE'
 
-- name: "Transfer Files"
+- name: Transfer Files
   template:
     src='{{ item.template }}'
     dest='{{ item.dest }}'
@@ -15,15 +15,15 @@
     group='root'
     mode='755'
   with_items:
-    - { template: 'setup.json', dest: '/tmp/spideroak.setup.json'}
-    - { template: 'etc_init_d_spideroak', dest: '/etc/init.d/spideroak'}
+    - { template: setup.json', dest: '/tmp/spideroak.setup.json}
+    - { template: etc_init_d_spideroak', dest: '/etc/init.d/spideroak}
 
-- name: "Setup SpiderOak Host"
+- name: Setup SpiderOak Host
   shell: SpiderOakONE --setup=/tmp/spideroak.setup.json
   async: 9999999
   ignore_errors: true
 
-- name: 'Add SpiderOak to Boot-List'
+- name: Add SpiderOak to Boot-List
   command: update-rc.d spideroak defaults
 
 
diff --git a/tasks/spideroak.yml b/tasks/spideroak.yml
index 40b64be..65481f2 100644
--- a/tasks/spideroak.yml
+++ b/tasks/spideroak.yml
@@ -1,24 +1,24 @@
 ---
 # file: roles/spideroak/tasks/spideroak.yml
 
-- name: "Check SpiderOak Requirement"
+- name: Check SpiderOak Requirement
   shell: ls /usr/bin/SpiderOakONE
   register: spideroak_available
   failed_when: false
   changed_when: false
 
-- name: "Apt Key"
+- name: Apt Key
   apt_key:
     url='https://spideroak.com/dist/spideroak-apt-2016.asc'
     state=present
 
-- name: "Apt Repository"
+- name: Apt Repository
   apt_repository:
     repo='deb http://apt.spideroak.com/ubuntu-spideroak-hardy/ release restricted'
     state='present'
     mode='644'
 
-- name: "Install SpiderOak Agent"
+- name: Install SpiderOak Agent
   apt:
     pkg='spideroakone'
     state='present'
@@ -27,29 +27,29 @@
 - import_tasks: register.yml
   when: spideroak_reinstall|default(false) == 'true' or (spideroak_available is defined and spideroak_available.stdout != '/usr/bin/SpiderOakONE')
 
-- name: "Stop SpiderOak"
+- name: Stop SpiderOak
   service:
     name='spideroak'
     state='stopped'
     pattern='SpiderOakONE'
 
-- name: "Wait"
+- name: Wait
   shell: sleep 5
 
-- name: "Add Directories to SpiderOak"
+- name: Add Directories to SpiderOak
   shell: SpiderOakONE --include-dir={{ item }}
   with_items: '{{ spideroak_include|default([]) }}'
 
-- name: "Add Drupal Files Directory to SpiderOak"
+- name: Add Drupal Files Directory to SpiderOak
   shell: SpiderOakONE --include-dir={{ jailroot|default('/jails') }}/{{ item.jail.name }}/var/www/files
   with_items: '{{ drupal_settings|default([]) }}'
   when: item.jail is defined and item.mountpoints is not defined
 
-- name: "Add MySQL Backups to SpiderOak"
+- name: Add MySQL Backups to SpiderOak
   shell: SpiderOakONE --include-dir=/var/backups/mysql
   when: groups['dbserver_mysql'] is defined and inventory_hostname in groups['dbserver_mysql']
 
-- name: "Start SpiderOak"
+- name: Start SpiderOak
   service:
     name='spideroak'
     state='started'
-- 
GitLab