Skip to content

Commit e457939

Browse files
mhjacksrichm
authored andcommitted
Always become user we are managing
1 parent fd9cdc7 commit e457939

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

tasks/get_systemd_units_info.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,7 @@
88
systemd_units:
99
user: "{{ __systemd_units_info.key }}"
1010
user_facts: "{{ ansible_facts['systemd_units_user'] | d({}) }}"
11-
become: "{{ __systemd_units_info.key != 'root' }}"
12-
become_user: "{{ __systemd_units_info.key if __systemd_units_info.key != 'root'
13-
else omit }}"
11+
become: true
12+
become_user: "{{ __systemd_units_info.key }}"
1413
environment:
1514
XDG_RUNTIME_DIR: "{{ __systemd_units_info.value['xdg_dir'] }}"

tasks/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@
114114
daemon_reload: true
115115
scope: "{{ 'system' if item == 'root' else 'user' }}"
116116
loop: "{{ __users_with_changes }}"
117-
become: "{{ item != 'root' }}"
118-
become_user: "{{ item if item != 'root' else omit }}"
117+
become: true
118+
become_user: "{{ item }}"
119119
environment:
120120
XDG_RUNTIME_DIR: "{{ __systemd_user_info[item]['xdg_dir'] }}"
121121
vars:

tasks/manage_units.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
masked: "{{ __systemd_masked | bool if __systemd_masked | length > 0 else omit }}"
3434
scope: "{{ 'system' if item.user == 'root' else 'user' }}"
3535
loop: "{{ __systemd_dict_list }}"
36-
become: "{{ item.user != 'root' }}"
37-
become_user: "{{ item.user if item.user != 'root' else omit }}"
36+
become: true
37+
become_user: "{{ item.user }}"
3838
environment:
3939
XDG_RUNTIME_DIR: "{{ item.xdg_dir }}"

0 commit comments

Comments
 (0)