Merge pull request #424 from spk/timezone-config

Use common_timezone and fix idempotence

roles/common/tasks/main.yml

@@ -41,15 +41,22 @@
   tags:
     - dependencies
 
-- name: Set timezone to UTC
-  action: shell echo Etc/UTC > /etc/timezone
+- name: timezone - configure /etc/timezone
+  copy:
+    content: "{{ common_timezone | regex_replace('$', '\n') }}"
+    dest: /etc/timezone
+    owner: root
+    group: root
+    mode: 0644
+  register: common_timezone_config
 
-- name: Set localtime to UTC
+- name: timezone - Set localtime to UTC
   file: src=/usr/share/zoneinfo/Etc/UTC dest=/etc/localtime
+  when: common_timezone_config.changed
 
-- name: Reconfigure tzdata
-  action: command dpkg-reconfigure -f noninteractive tzdata
-  when: ansible_distribution == 'Debian' or ansible_distribution == 'Ubuntu'
+- name: timezone - reconfigure tzdata
+  command: dpkg-reconfigure --frontend noninteractive tzdata
+  when: common_timezone_config.changed
 
 - name: Apticron email configuration
   template: src=apticron.conf.j2 dest=/etc/apticron/apticron.conf

vars/defaults.yml

@@ -5,6 +5,7 @@
 ###############################################################################
 
 # # common
+common_timezone: 'Etc/UTC'
 # domain: (required)
 # main_user_name: (required)
 admin_email: "{{ main_user_name }}@{{ domain }}"

vars/testing.yml

@@ -5,6 +5,7 @@
 ###############################################################################
 
 # common
+common_timezone: 'Etc/UTC'
 domain: sovereign.local
 main_user_name: sovereign
 encfs_password: testPassword