Fix home creation

Author: jovial

ansible/roles/sssd/handlers/main.yml

@@ -16,3 +16,4 @@
   service:
     name: "{{ sssd_service }}"
     state: "restarted"
+

ansible/roles/sssd/meta/main.yml

@@ -50,3 +50,4 @@ galaxy_info:
 dependencies: []
   # List your role dependencies here, one per line. Be sure to remove the '[]' above,
   # if you add dependencies to this list.
+  

ansible/roles/sssd/tasks/configure.yml

@@ -9,16 +9,6 @@
   when: sssd_config is defined
   notify: "Restart sssd"
 
-- name: "Check if authconfig needs to be run to configure pam/nsswitch"
-  # FIXME(wszumski) --test is no longer supported. This probably isn't doing what we think it is doing
-  # Error: option --test is no longer supported and we cannot continue if it is set.
-  # authconfig is also deprecated. We should switch to authselect.
-  shell: "/usr/bin/test \"$(authconfig {% if sssd_enable_mkhomedir | bool %}--enablemkhomedir {% endif %}--enablesssd --enablesssdauth --test)\" = \"$(authconfig --test)\""
-  register: authconfig_result
-  changed_when: "authconfig_result.rc != 0"
-  check_mode: no
-  failed_when: "authconfig_result.rc >= 2"
-
+# TODO: Make idempotent
 - name: "Configure nsswitch and pam for SSSD via authconfig"
-  command: "authconfig {% if sssd_enable_mkhomedir | bool %}--enablemkhomedir {% endif %} --enablesssd --enablesssdauth --update"
-  when: "authconfig_result.rc != 0"
+  command: "authselect select sssd{% if sssd_enable_mkhomedir | bool %} with-mkhomedir{% endif %}"

ansible/roles/sssd/tasks/main.yml

@@ -5,3 +5,4 @@
 - include_tasks: "install.yml"
 
 - include_tasks: "runtime.yml"
+

ansible/roles/sssd/tasks/runtime.yml

@@ -4,4 +4,4 @@
 
 - include_tasks: "configure.yml"
 
-- include_tasks: "service.yml"
+- include_tasks: "service.yml"

ansible/roles/sssd/tasks/service.yml

@@ -4,3 +4,9 @@
     name: "{{ sssd_service }}"
     state: "{{ sssd_state }}"
     enabled: "{{ sssd_enabled }}"
+
+- name: "Ensure oddjob is started"
+  service:
+    name: oddjobd
+    state: started
+    enabled: "{{ sssd_enable_mkhomedir }}"

ansible/roles/sssd/tests/inventory

@@ -1 +1,2 @@
 localhost
+