Преглед на файлове

Fix lint errors and find replace issues.

Michael Porter преди 6 години
родител
ревизия
14b249ffd2

+ 2 - 1
tasks/Amazon/dashboard.yml

@@ -3,7 +3,8 @@
 # Specific to CentOS
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: dashboard
 
 - name: Ensure Uchiwa is installed

+ 2 - 1
tasks/Amazon/main.yml

@@ -3,8 +3,9 @@
 # This takes care of base prerequisites for Amazon Linux AMI
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
   tags: setup
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
 
 - name: Set epel_version override when AmazonLinux AMIv2
   tags: setup

+ 2 - 1
tasks/Amazon/rabbit.yml

@@ -3,7 +3,8 @@
 # Specific to Amazon Linux
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - name: Configure RabbitMQ/RabbitMQ-erlang GPG keys in the RPM keyring

+ 2 - 1
tasks/Amazon/redis.yml

@@ -3,7 +3,8 @@
 # Specific to Amazon Linux AMI
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: redis
 
 - name: Install EPEL repo

+ 3 - 4
tasks/CentOS/dashboard.yml

@@ -3,9 +3,8 @@
 # Specific to CentOS
 
 - name: Include ansible_distribution vars
-  include_varse ansible_distribution vars
-  include_varse ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: dashboard
 
 - name: Ensure Uchiwa is installed
@@ -36,7 +35,7 @@
   tags: dashboard
   template:
     src: sensu_enterprise_dashboard_config.json.j2
-    dest: "{{ sensu_config_path}}/dashboard.json"
+    dest: "{{ sensu_config_path }}/dashboard.json"
   when: se_enterprise
   notify:
     - restart sensu-enterprise-dashboard service

+ 2 - 2
tasks/CentOS/main.yml

@@ -3,8 +3,8 @@
 # This takes care of base prerequisites for CentOS
 
 - name: Include ansible_distribution vars
-  include_varse ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: setup
 
 - name: Ensure the Sensu Core Yum repo is present

+ 2 - 2
tasks/CentOS/rabbit.yml

@@ -3,8 +3,8 @@
 # Specific to CentOS
 
 - name: Include ansible_distribution vars
-  include_varse ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - name: Configure RabbitMQ GPG keys in the RPM keyring

+ 2 - 2
tasks/CentOS/redis.yml

@@ -3,8 +3,8 @@
 # Specific to CentOS
 
 - name: Include ansible_distribution vars
-  include_varse ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: redis
 
 - name: Install EPEL repo

+ 2 - 1
tasks/Debian/dashboard.yml

@@ -3,7 +3,8 @@
 # Specific to Debian
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: dashboard
 
 - name: Install uchiwa

+ 2 - 1
tasks/Debian/main.yml

@@ -3,7 +3,8 @@
 # This takes care of base prerequisites for Debian
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: setup
 
 - name: Ensure apt-transport-https is installed

+ 2 - 1
tasks/Debian/rabbit.yml

@@ -3,7 +3,8 @@
 # Specific to Debian
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - name: Ensure the RabbitMQ APT repo GPG key is present

+ 2 - 1
tasks/Debian/redis.yml

@@ -3,7 +3,8 @@
 # Specific to Debian
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: redis
 
 - name: Ensure redis is installed

+ 2 - 1
tasks/Fedora/rabbit.yml

@@ -3,7 +3,8 @@
 # Specific to Fedora
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - name: Configure RabbitMQ GPG keys in the RPM keyring

+ 2 - 1
tasks/Fedora/redis.yml

@@ -3,7 +3,8 @@
 # Specific to Fedora
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: redis
 
 - name: Ensure jemalloc is installed as a dependency of Redis

+ 2 - 1
tasks/FreeBSD/dashboard.yml

@@ -3,7 +3,8 @@
 # Specific to FreeBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: dashboard
 
 - name: Ensure Uchiwa (dashboard) dependencies are installed

+ 3 - 2
tasks/FreeBSD/main.yml

@@ -3,13 +3,14 @@
 # This takes care of base prerequisites for FreeBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: setup
 
 - name: Ensure the Sensu group is present
   tags: setup
   group:
-    name: {{ sensu_group_name }}
+    name: "{{ sensu_group_name }}"
     state: present
 
 - name: Ensure the Sensu user is present

+ 2 - 1
tasks/FreeBSD/rabbit.yml

@@ -3,7 +3,8 @@
 # Specific to FreeBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - name: Ensure RabbitMQ is installed

+ 2 - 1
tasks/FreeBSD/redis.yml

@@ -3,7 +3,8 @@
 # Specific to FreeBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: redis
 
 - name: Ensure redis is installed

+ 2 - 1
tasks/OpenBSD/dashboard.yml

@@ -3,7 +3,8 @@
 # Specific to OpenBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: dashboard
 
 - name: Ensure Uchiwa (dashboard) dependencies are installed

+ 3 - 2
tasks/OpenBSD/main.yml

@@ -3,7 +3,8 @@
 # This takes care of base prerequisites for OpenBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: setup
 
 - name: Ensure the Sensu group is present
@@ -45,7 +46,7 @@
   tags: setup
   gem:
     name: sensu
-    repository: "{{ sensu_gem_repository | default('https://api.rubygems.org/')}}"
+    repository: "{{ sensu_gem_repository | default('https://api.rubygems.org/') }}"
     user_install: no
     version: "{{ sensu_gem_version }}"
     executable: /usr/local/bin/gem23

+ 2 - 1
tasks/OpenBSD/rabbit.yml

@@ -3,7 +3,8 @@
 # Specific to OpenBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - name: Ensure RabbitMQ is installed

+ 2 - 1
tasks/OpenBSD/redis.yml

@@ -3,7 +3,8 @@
 # Specific to OpenBSD
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: redis
 
 - name: Ensure redis is installed

+ 2 - 1
tasks/SmartOS/client.yml

@@ -3,7 +3,8 @@
 # Specific to Joyent SmartOS
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: client
 
 - name: Deploy Sensu client service manifest

+ 3 - 2
tasks/SmartOS/dashboard.yml

@@ -3,7 +3,8 @@
 # Specific to Joyent SmartOS
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: dashboard
 
 - name: Ensure Uchiwa (dashboard) dependencies are installed
@@ -84,7 +85,7 @@
     src: uchiwa.smartos_smf_manifest.xml.j2
     owner: root
     group: root
-    mode: 644
+    mode: 0644
   notify: import uchiwa service
 
 - meta: flush_handlers

+ 2 - 1
tasks/SmartOS/main.yml

@@ -3,7 +3,8 @@
 # This takes care of base prerequisites for Joyent SmartOS
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: setup
 
 - name: Ensure the Sensu group is present

+ 2 - 1
tasks/SmartOS/redis.yml

@@ -3,7 +3,8 @@
 # Specific to Ubuntu
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: redis
 
 - name: Ensure redis is installed

+ 2 - 1
tasks/SmartOS/server.yml

@@ -4,7 +4,8 @@
 # Specific to SmartOS
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: server
 
 - name: Deploy Sensu server service manifest

+ 2 - 1
tasks/Ubuntu/dashboard.yml

@@ -3,7 +3,8 @@
 # Specific to Ubuntu
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: dashboard
 
 - name: Install Uchiwa

+ 2 - 1
tasks/Ubuntu/main.yml

@@ -3,7 +3,8 @@
 # This takes care of base prerequisites for Ubuntu
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: setup
 
 - name: Ensure that https transport is ready

+ 2 - 1
tasks/Ubuntu/rabbit.yml

@@ -3,7 +3,8 @@
 # Specific to Ubuntu
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - name: Ensure the RabbitMQ APT repo GPG key is present

+ 2 - 1
tasks/Ubuntu/redis.yml

@@ -3,7 +3,8 @@
 # Specific to Ubuntu
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
 
 - name: Ensure redis is installed
   apt:

+ 2 - 1
tasks/client.yml

@@ -2,7 +2,8 @@
 # tasks/client.yml: Deploy various client-side configurations for Sensu
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: client
 
 - name: Deploy Sensu client service configuration

+ 2 - 1
tasks/common.yml

@@ -2,7 +2,8 @@
 # tasks/common.yml: Deploy configurations common to client and server for Sensu
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
 
 - name: Ensure the Sensu config directory is present
   file:

+ 2 - 1
tasks/main.yml

@@ -2,7 +2,8 @@
 # tasks/main.yml: "Master" playbook for the sensu.sensu role
 
 - name: Include distribution specific variables
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
 
 - include_tasks: "{{ role_path }}/tasks/{{ ansible_distribution }}/main.yml"
   tags: setup

+ 4 - 3
tasks/plugins.yml

@@ -2,7 +2,8 @@
 # tasks/plugins.yml: Deploy available checks/plugins/handlers/filters/mutators
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
 
 - name: Ensure Sensu plugin directory exists
   file:
@@ -79,7 +80,7 @@
 - name: Deploy check/handler/filter/mutator definitions to the master
   template:
     src: "{{ item }}"
-    dest: "{{ sensu_config_path }}/conf.d/{{ item | basename | regex_replace('.j2', '')}}"
+    dest: "{{ sensu_config_path }}/conf.d/{{ item | basename | regex_replace('.j2', '') }}"
     owner: "{{ sensu_user_name }}"
     group: "{{ sensu_group_name }}"
   when: sensu_master
@@ -99,7 +100,7 @@
 - name: Deploy client definitions
   copy:
     src: "{{ static_data_store }}/sensu/client_definitions/{{ item }}/"
-    dest: "{{ sensu_config_path }}/conf.d/{{ item | basename | regex_replace('.j2', '')}}"
+    dest: "{{ sensu_config_path }}/conf.d/{{ item | basename | regex_replace('.j2', '') }}"
     owner: "{{ sensu_user_name }}"
     group: "{{ sensu_group_name }}"
   when:

+ 2 - 1
tasks/rabbit.yml

@@ -2,7 +2,8 @@
 # tasks/rabbit.yml: Deploy RabbitMQ and set-up vhost for Sensu messaging
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: rabbitmq
 
 - include_tasks: "{{ ansible_distribution }}/rabbit.yml"

+ 2 - 1
tasks/server.yml

@@ -2,7 +2,8 @@
 # tasks/server.yml: Deploy Sensu Server/API
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
   tags: server
 
 - name: Deploy Sensu server API configuration

+ 2 - 1
tasks/ssl.yml

@@ -2,7 +2,8 @@
 # tasks/ssl.yml: Deploy the client SSL cert/key to client systems
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
 
 - name: Ensure Sensu SSL directory exists
   file:

+ 2 - 1
tasks/ssl_generate.yml

@@ -3,7 +3,8 @@
 # data store for deployment to clients
 
 - name: Include ansible_distribution vars
-  include_vars: "{{ ansible_distribution }}.yml"
+  include_vars:
+    file: "{{ ansible_distribution }}.yml"
 
 - name: Ensure OpenSSL is installed
   package: