[rqd] Change default RQD_BECOME_JOB_USER from True to False (#1659)

**Summarize your change.**
Change default RQD_BECOME_JOB_USER from True to False

**Reason for change**
I think most studios don't expect the render client to change to the
user of the render submitter, but have a "render" user dedicated for it.
Also, it's not very obvious why root is required to run RQD and that the
behavior can be turned off.

rqd/README.md

@@ -36,10 +36,10 @@ python3 setup.py install
 ### Create rqd.conf
 Example `rqd.conf` file :
 
-```toml
+```ini
 [Override]
-## Variable that decided if RQD should switch to the user of the job running. Requires root
-RQD_BECOME_JOB_USER = False 
+# Variable that decided if RQD should switch to the user of the job running. Requires root
+RQD_BECOME_JOB_USER = False
 
 # Log levels for RQD
 CONSOLE_LOG_LEVEL = INFO

rqd/rqd.example.conf

@@ -4,6 +4,8 @@
 # If this file is not found the defaults in rqconstants.py will be used.
 
 [Override]
+# Variable that decided if RQD should switch to the user of the job running. Requires root
+RQD_BECOME_JOB_USER = False
 RQD_USE_PATH_ENV_VAR=1
 RQD_USE_IP_AS_HOSTNAME=0
 OVERRIDE_IS_DESKTOP=True

rqd/rqd/rqconstants.py

@@ -75,7 +75,7 @@
 RQD_CUSTOM_HOME_PREFIX = None
 RQD_CUSTOM_MAIL_PREFIX = None
 
-RQD_BECOME_JOB_USER = True
+RQD_BECOME_JOB_USER = False
 RQD_CREATE_USER_IF_NOT_EXISTS = True
 SENTRY_DSN_PATH = None
 RQD_TAGS = ''

rqd/tests/rqcore_test.py

@@ -763,10 +763,7 @@ def test_runLinux(
                 "-p",
                 "-o",
                 jobTempPath + "rqd-stat-" + frameId + "-" + str(currentTime),
-                "/bin/su",
-                frameUsername,
-                "-c",
-                '"' + tempDir + "/rqd-cmd-" + frameId + "-" + str(currentTime) + '"',
+                tempDir + "/rqd-cmd-" + frameId + "-" + str(currentTime),
             ],
             env=mock.ANY,
             cwd=jobTempPath,