Merge pull request #335 from meysam81/issue-333-fix

feat: allow users to select their shell

Author: jdno

ansible/roles/dev-desktop/defaults/main.yml

@@ -8,3 +8,11 @@ vars_user_quota_gb: 150
 
 # Prototype user whose file system quota will be copied to new user accounts
 vars_user_quota_prototype_user: "quota-prototype"
+
+vars_user_config:
+  # - username: rylev
+  #   shell: /bin/sh
+  # - username: meysam
+  #   shell: /usr/bin/zsh
+  # - username: jdno
+  #   shell: /usr/bin/fish

ansible/roles/dev-desktop/tasks/dependencies.yml

@@ -57,6 +57,7 @@
       # Necessary for building rr
       - capnproto
       - libcapnp-dev
+      - zsh
     state: present
 
 - name: Uninstall valgrind from apt

ansible/roles/dev-desktop/tasks/main.yml

@@ -4,6 +4,9 @@
 - include_tasks: podman.yml
 - include_tasks: quota.yml
 - include_tasks: user_configuration.yml
+- include_tasks: usermod.yml
+  with_items: "{{ vars_user_config }}"
+  when: vars_user_config is not none and vars_user_config | length > 0
 - include_tasks: team_login.yml
 - include_tasks: github.yml
 - include_tasks: motd.yml

ansible/roles/dev-desktop/tasks/usermod.yml

@@ -0,0 +1,13 @@
+---
+- name: Check user {{ item.username }} exists
+  ansible.builtin.getent:
+    database: passwd
+    key: "{{ item.username }}"
+  register: user_exists
+  ignore_errors: true
+
+- name: Set the user shell to {{ item.shell }}
+  ansible.builtin.user:
+    name: "{{ item.username }}"
+    shell: "{{ item.shell }}"
+  when: user_exists is succeeded