Rewrite to act like rest of roles, add common OS list

Signed-off-by: Jason Rothstein <fdragon@fdragon.org>

tasks/main.yml

@@ -1,5 +1,24 @@
 ---
 # tasks file for ensure_timezone
+- name: 'include variables'
+  when:
+    - ansible_facts["system"] == 'Linux'
+  include_vars:
+    file: '{{ lookup("first_found", findme ) }}'
+    name: 'ensure_timezone'
+  vars:
+    findme:
+      files:
+        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"] }}-{{ ansible_facts["architecture"] }}.yml'
+        - '{{ ansible_facts["distribution"] }}-{{ ansible_facts["distribution_major_version"] }}-default.yml'
+        - '{{ ansible_facts["distribution"] }}-default.yml'
+        - '{{ ansible_os_family }}-{{ ansible_facts["distribution_major_version"] }}-{{ ansible_facts["architecture"] }}.yml'
+        - '{{ ansible_os_family }}-{{ ansible_facts["distribution_major_version"] }}-default.yml'
+        - '{{ ansible_os_family }}-default.yml'
+        - 'default.yml'
+      paths:
+        - '../vars/'
+      errors: 'ignore'
 - name: 'package discovery'
   when:
     - ansible_facts["system"] == 'Linux'
@@ -14,50 +33,99 @@
   community.general.timezone:
     hwclock: '{{ timezone_hwclock }}'
     name: '{{ timezone_name }}'
-  register: 'results'
-- name: 'Restart if time zone changed'
+  notify:
+    - 'ensure_timezone.package_facts'
+    - 'ensure_timezone.service_facts'
+    - 'ensure_timezone.service_restart'
+- name: 'ensure packages'
   when:
     - ansible_facts["system"] == 'Linux'
-    - services is defined
-    - services[item.name] is defined
-    - results.changed
-  ansible.builtin.service:
-    enabled: '{{ item.enabled }}'
+    - ensure_timezone is defined
+    - ensure_timezone.package_list is defined
+    - ensure_timezone.package_list is iterable
+  ansible.builtin.package:
     name: '{{ item.name }}'
     state: '{{ item.state }}'
-  loop:
-    - enabled: 'yes'
-      name: 'crond'
-      state: 'restarted'
-    - enabled: 'yes'
-      name: 'crond.service'
-      state: 'restarted'
+  loop: '{{ ensure_timezone.package_list }}'
   loop_control:
     label: '{{ item.name }} will be {{ item.state }}'
   notify:
     - 'ensure_timezone.package_facts'
     - 'ensure_timezone.service_facts'
+- name: 'ensure configurations'
+  when:
+    - ansible_facts["system"] == 'Linux'
+    - ensure_timezone is defined
+    - ensure_timezone.template_list is defined
+    - ensure_timezone.template_list is iterable
+  ansible.builtin.template:
+    backup: 'no'
+    dest: '{{ item.dest }}'
+    group: '{{ item.group | default(omit) }}'
+    mode: '{{ item.mode | default(omit) }}'
+    owner: '{{ item.owner | default(omit) }}'
+    selevel: '{{ iteml.selevel | default(omit) }}'
+    serole: '{{ item.serole | default(omit) }}'
+    setype: '{{ item.setype | default(omit) }}'
+    seuser: '{{ item.seuser | default(omit) }}'
+    src: '{{ item.src }}'
+  loop: '{{ ensure_timezone.template_list }}'
+  loop_control:
+    label: '{{ item.dest }} will be ensured'
+  notify:
+    - 'ensure_timezone.package_facts'
+    - 'ensure_timezone.service_facts'
+    - 'ensure_timezone.service_reload'
+    - 'ensure_timezone.service_restart'
 - name: 'ensure services'
   when:
     - ansible_facts["system"] == 'Linux'
-    - services is defined
-    - services[item.name] is defined
+    - ensure_timezone is defined
+    - ensure_timezone.service_list is defined
+    - ensure_timezone.service_list is iterable
   ansible.builtin.service:
     enabled: '{{ item.enabled }}'
     name: '{{ item.name }}'
     state: '{{ item.state }}'
-  loop:
-    - enabled: 'yes'
-      name: 'crond'
-      state: 'started'
-    - enabled: 'yes'
-      name: 'crond.service'
-      state: 'started'
+  loop: '{{ ensure_timezone.service_list }}'
   loop_control:
     label: '{{ item.name }} will be {{ item.state }}'
   notify:
     - 'ensure_timezone.package_facts'
     - 'ensure_timezone.service_facts'
+- name: 'Ensure SELinux is configured'
+  when:
+    - ansible_facts["system"] == 'Linux'
+    - ensure_timezone is defined
+    - ensure_timezone.selinux_policy is defined
+    - ensure_timezone.selinux_state is defined
+  ansible.posix.selinux:
+    policy: '{{ ensure_timezone.selinux_policy }}'
+    state: '{{ ensure_timezone.selinux_state }}'
+  register: 'results'
+- name: 'Reboot if required'
+  when:
+    - ansible_facts["system"] == 'Linux'
+    - ensure_timezone is defined
+    - ensure_timezone.selinux_policy is defined
+    - ensure_timezone.selinux_state is defined
+    - results is defined
+    - results.reboot_required is defined
+    - results.reboot_required
+  reboot:
+- name: 'ensure seboolean'
+  when:
+    - ansible_facts["system"] == 'Linux'
+    - ensure_timezone is defined
+    - ensure_timezone.seboolean_list is defined
+    - ensure_timezone.seboolean_list is iterable
+  ansible.posix.seboolean:
+    name: '{{ item.name }}'
+    persistent: '{{ item.persistent }}'
+    state: '{{ item.state }}'
+  loop: '{{ ensure_timezone.seboolean_list }}'
+  loop_control:
+    label: '{{ item.name }} will be {{ item.state }}'
 - name: 'flush handlers'
   meta: 'flush_handlers'
 ...