search

appsembler / configuration

a simple, but flexible, way for anyone to stand up an instance of the edX platform that is fully configured and ready-to-go
GNU Affero General Public License v3.0
15 stars 13 forks source link

s/sudo_user/become_user/ #389

Closed thraxil closed 2 years ago

thraxil commented 2 years ago

Ansible 1.9 deprecatd sudo_user in favor of become_user. As of Ansible 2.9, sudo_user isn't supported at all, so it's a blocker on us moving past 2.8.

I don't think we use the hotg role, but I figured it didn't hurt to fix it in that one too.

OmarIthawi commented 2 years ago

@thraxil we have more than what's mentioned in this PR, but changing part of them is very welcome:

$ git diff open-release/juniper.master..appsembler/juniper/master | grep '^+' | grep sudo_user
+      sudo_user: "{{ certs_user}}"
+      sudo_user: "{{ certs_user}}"
+      sudo_user: "{{ edxapp_user }}"
+      sudo_user: "{{ edxapp_user }}"
+      sudo_user: "{{ supervisor_service_user }}"
+      sudo_user: "{{ supervisor_service_user }}"
+  sudo_user: "{{ edxapp_user }}"