syntax cleanup

defaults/main.yml

@@ -1,31 +1,30 @@
 ---
-ntp_pkg_state: installed
-ntp_service_state: started
-ntp_service_enabled: yes
+ntp_pkg_state: 'installed'
+ntp_service_state: 'started'
+ntp_service_enabled: 'yes'
 
-ntp_config_server: [0.pool.ntp.org, 1.pool.ntp.org, 2.pool.ntp.org, 3.pool.ntp.org]
+ntp_config_server: [ '0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org' ]
 ntp_config_restrict:
-- "-4 default kod notrap nomodify nopeer noquery"
-- "-6 default kod notrap nomodify nopeer noquery"
-- "127.0.0.1"
-- "::1"
+- '-4 default kod notrap nomodify nopeer noquery'
+- '-6 default kod notrap nomodify nopeer noquery'
+- '127.0.0.1'
+- '::1'
 
 ntp_config_listen:
-- 127.0.0.1
+- '127.0.0.1'
 
 ntp_config_filegen:
-- "loopstats file loopstats type day enable"
-- "peerstats file peerstats type day enable"
-- "clockstats file clockstats type day enable"
+- 'loopstats file loopstats type day enable'
+- 'peerstats file peerstats type day enable'
+- 'clockstats file clockstats type day enable'
 
-ntp_config_statistics: "loopstats peerstats clockstats"
-
-ntp_config_crypto: no
-ntp_config_includefile: no
-ntp_config_keys: no
-ntp_config_trustedkey: no
-ntp_config_requestkey: no
-ntp_config_controlkey: no
-ntp_config_broadcast: no 
-ntp_config_broadcastclient: no
-ntp_config_multicastclient: no
+ntp_config_statistics: 'loopstats peerstats clockstats'
+ntp_config_crypto: 'no'
+ntp_config_includefile: 'no'
+ntp_config_keys: 'no'
+ntp_config_trustedkey: 'no'
+ntp_config_requestkey: 'no'
+ntp_config_controlkey: 'no'
+ntp_config_broadcast: 'no'
+ntp_config_broadcastclient: 'no'
+ntp_config_multicastclient: 'no'

tasks/main.yml

@@ -1,26 +1,24 @@
 ---
 - name: Add the OS specific variables
-  include_vars: "{{ ansible_os_family }}.yml"
-  tags: ["configuration","packages","service","ntp"]
+  include_vars: '{{ ansible_os_family }}.yml'
+  tags: [ 'configuration', 'package', 'service', 'ntp' ]
 
 - name: Install the required packages in Redhat derivatives
-  yum: name={{ item }} state={{ ntp_pkg_state }}
-  with_items: ntp_pkgs
+  yum: name=ntp state={{ ntp_pkg_state }}
   when: ansible_os_family == 'RedHat'
-  tags: ["packages","ntp"]
+  tags: [ 'package', 'ntp' ]
 
 - name: Install the required packages in Debian derivatives
-  apt: name={{ item }} state={{ ntp_pkg_state }}
-  with_items: ntp_pkgs
+  apt: name=ntp state={{ ntp_pkg_state }}
   when: ansible_os_family == 'Debian'
-  tags: ["packages","ntp"]
+  tags: [ 'package', 'ntp' ]
 
 - name: Copy the ntp.conf template file
   template: src=ntp.conf.j2 dest=/etc/ntp.conf
   notify:
   - restart ntp
-  tags: ["configuration","packages","ntp"]
+  tags: [ 'configuration', 'package', 'ntp' ]
 
 - name: Start/stop ntp service
-  service: name={{ ntp_service_name }} state={{ ntp_service_state }} enabled={{ ntp_service_enabled }} pattern="/ntpd"
-  tags: ["service","ntp"]
+  service: name={{ ntp_service_name }} state={{ ntp_service_state }} enabled={{ ntp_service_enabled }} pattern='/ntpd'
+  tags: [ 'service', 'ntp' ]

vars/Debian.yml

@@ -1,7 +1,4 @@
 ---
-ntp_pkgs:
-- ntp
-
 ntp_service_name: ntp
 ntp_config_driftfile: /var/lib/ntp/ntp.drift
 

vars/RedHat.yml

@@ -1,6 +1,3 @@
 ---
-ntp_pkgs:
-- ntp
-
 ntp_service_name: ntpd
 ntp_config_driftfile: /var/lib/ntp/drift