add TODO reminders to remove proot / venv code

* won't be needed once we move to container-based
  execution
This commit is contained in:
Jim Ladd 2020-11-03 13:00:39 -08:00 committed by Alan Rominger
parent 12cbc9756b
commit 277c47ba4e
No known key found for this signature in database
GPG Key ID: C2D7EAAA12B63559

View File

@ -2461,12 +2461,14 @@ class RunInventoryUpdate(BaseTask):
event_model = InventoryUpdateEvent
event_data_key = 'inventory_update_id'
# TODO: remove once inv updates run in containers
def should_use_proot(self, inventory_update):
'''
Return whether this task should use proot.
'''
return getattr(settings, 'AWX_PROOT_ENABLED', False)
# TODO: remove once inv updates run in containers
@property
def proot_show_paths(self):
return [settings.AWX_ANSIBLE_COLLECTIONS_PATHS]
@ -2503,6 +2505,7 @@ class RunInventoryUpdate(BaseTask):
private_data_files=private_data_files)
if private_data_files is None:
private_data_files = {}
# TODO: remove once containers replace custom venvs
self.add_ansible_venv(inventory_update.ansible_virtualenv_path, env, isolated=isolated)
# Legacy environment variables, were used as signal to awx-manage command