1
0
mirror of https://github.com/ansible/awx.git synced 2024-10-31 23:51:09 +03:00

Don't override ANSIBLE_SSH_ARGS for ad-hoc commands.

This reverts 8beed7f4, which says:
    Disable ControlPersist for ad hoc commands, should avoid any
    issues with proot or needing to clean up sockets afterwards.

Given we've switched to the much less finicky bwrap for process
isolation, along with runner-based process killing, this probably
isn't needed any more.
This commit is contained in:
Bill Nottingham 2020-04-22 20:36:48 -04:00
parent a34a63ec7f
commit 3f5820adf8

View File

@ -2737,9 +2737,12 @@ class RunAdHocCommand(BaseTask):
env['ANSIBLE_LOAD_CALLBACK_PLUGINS'] = '1' env['ANSIBLE_LOAD_CALLBACK_PLUGINS'] = '1'
env['ANSIBLE_SFTP_BATCH_MODE'] = 'False' env['ANSIBLE_SFTP_BATCH_MODE'] = 'False'
# Specify empty SSH args (should disable ControlPersist entirely for # Create a directory for ControlPath sockets that is unique to each
# ad hoc commands). # ad hoc command and visible inside the proot environment (when enabled).
env.setdefault('ANSIBLE_SSH_ARGS', '') cp_dir = os.path.join(private_data_dir, 'cp')
if not os.path.exists(cp_dir):
os.mkdir(cp_dir, 0o700)
env['ANSIBLE_SSH_CONTROL_PATH'] = cp_dir
return env return env