fix(roles): Default to root as user name
Always default to 'root' as a user_name if no variable has been set. This can easily happen as not every role sets the variable and instead we only set it once in the user role. Another way to possibly go about it in the future would be to inject the 'user_name' into each role that needs it as one of that role's default variables. If it is specified by the user somewhere it _should_ override those defaults, though I have to read up on the exact variable precedence.
This commit is contained in:
parent
a0ea10542d
commit
5dd160727a
5 changed files with 6 additions and 6 deletions
|
@ -138,6 +138,6 @@
|
|||
|
||||
- name: Put user in socklog group
|
||||
ansible.builtin.user:
|
||||
name: "{{ user_name }}"
|
||||
name: "{{ user_name | default('root') }}"
|
||||
groups: [socklog]
|
||||
append: true
|
||||
|
|
|
@ -14,6 +14,6 @@
|
|||
|
||||
- name: Put user in plugdev group
|
||||
ansible.builtin.user:
|
||||
name: "{{ user_name }}"
|
||||
name: "{{ user_name | default('root') }}"
|
||||
groups: [plugdev]
|
||||
append: true
|
||||
|
|
|
@ -6,7 +6,7 @@
|
|||
|
||||
- name: Put user in audio group
|
||||
ansible.builtin.user:
|
||||
name: "{{ user_name }}"
|
||||
name: "{{ user_name | default('root') }}"
|
||||
groups: "{{ audio_groups }}"
|
||||
append: true
|
||||
|
||||
|
|
|
@ -6,7 +6,7 @@
|
|||
|
||||
- name: Put user in power group
|
||||
ansible.builtin.user:
|
||||
name: "{{ user_name }}"
|
||||
name: "{{ user_name | default('root') }}"
|
||||
groups: [power]
|
||||
append: true
|
||||
|
||||
|
|
|
@ -16,11 +16,11 @@
|
|||
|
||||
- name: Add primary user
|
||||
ansible.builtin.user:
|
||||
name: "{{ user_name }}"
|
||||
name: "{{ user_name | default('root') }}"
|
||||
password: "{{ user_pass | password_hash('sha512', '{{ user_pass_salt}}') }}"
|
||||
create_home: true
|
||||
shell: "/bin/{{ user_shell }}"
|
||||
group: "{{ user_name }}"
|
||||
group: "{{ user_name | default('root') }}"
|
||||
groups: "{{ user_groups }}"
|
||||
generate_ssh_key: true
|
||||
tags:
|
||||
|
|
Loading…
Reference in a new issue