1
0
mirror of https://github.com/ansible/awx.git synced 2024-11-01 16:51:11 +03:00
awx/requirements
AlanCoding 0b9b8832a8
Get rid of hacks to simplify process
Remove poetry and wheel from bootstrap installs

Cleanup some docs around them
2020-01-07 17:14:34 -06:00
..
README.md Get rid of hacks to simplify process 2020-01-07 17:14:34 -06:00
requirements_ansible_git.txt unpin apache-libcloud now that a version is out without PyCrypto 2019-06-04 09:26:27 -04:00
requirements_ansible_uninstall.txt Don't use the rsa python lib. 2019-04-02 15:48:30 -04:00
requirements_ansible.in Get rid of hacks to simplify process 2020-01-07 17:14:34 -06:00
requirements_ansible.txt Get rid of hacks to simplify process 2020-01-07 17:14:34 -06:00
requirements_dev.txt add the ability to generate dot graphs for per-request profiling 2020-01-04 07:09:42 -05:00
requirements_git.txt Remove forks no longer needed from dependencies 2019-11-18 15:39:39 -05:00
requirements_setup_requires.txt Bootstrap venv creation with flit and poetry 2020-01-07 17:14:33 -06:00
requirements_tower_uninstall.txt Remove removal requirement that isn't actually in the requirements 2019-10-16 15:34:33 -04:00
requirements.in Get rid of hacks to simplify process 2020-01-07 17:14:34 -06:00
requirements.txt Get rid of hacks to simplify process 2020-01-07 17:14:34 -06:00
updater.sh Add some minimal sanity checking before running the updater script. 2019-07-11 12:12:22 -04:00

Dependency Management

The requirements.txt and requirements_ansible.txt files are generated from requirements.in and requirements_ansible.in, respectively, using pip-tools pip-compile.

How To Use

Commands should from inside ./requirements directory of the awx repository.

Make sure you have patch, awk, python3, python2, python3-venv, python2-virtualenv, pip2, pip3 installed. The development container image should have all these.

Upgrading or Adding Select Libraries

If you need to add or upgrade one targeted library, then modify requirements.in, then run the script:

./updater.sh

Upgrading Unpinned Dependency

If you require a new version of a dependency that does not have a pinned version for a fix or feature, pin a minimum version and run ./updater.sh. For example, replace the line asgi-amqp with asgi-amqp>=1.1.4, and consider leaving a note.

Then next time that a general upgrade is performed, the minimum version specifiers can be removed, because *.txt files are upgraded to latest.

Upgrading Dependencies

You can upgrade (pip-compile --upgrade) the dependencies by running

./updater.sh upgrade.

What The Script Does

This script will:

  • Update requirements.txt based on requirements.in
  • Update/generate requirements_ansible.txt based on requirements_ansible.in
    • including an automated patch that adds python_version < "3" for Python 2 backward compatibility
  • Removes the docutils dependency line from requirements.txt and requirements_ansible.txt

Licenses and Source Files

If any library has a change to its license with the upgrade, then the license for that library inside of docs/licenses needs to be updated.

For libraries that have source distribution requirements (LGPL as an example), a tarball of the library is kept along with the license. To download the PyPI tarball, you can run this command:

pip download <pypi library name> -d docs/licenses/ --no-binary :all: --no-deps

Make sure to delete the old tarball if it is an upgrade.

UPGRADE BLOCKERs

Anything pinned in *.in files involves additional manual work in order to upgrade. Some information related to that work is outlined here.

django

For any upgrade of Django, it must be confirmed that we don't regress on FIPS support before merging.

See internal integration test knowledge base article how_to_test_FIPS for instructions.

If operating in a FIPS environment, hashlib.md5() will raise a ValueError, but will support the usedforsecurity keyword on RHEL and Centos systems.

Keep an eye on https://code.djangoproject.com/ticket/28401

The override of names_digest could easily be broken in a future version. Check that the import remains the same in the desired version.

af5ec222cc/django/db/backends/base/schema.py (L7)

social-auth-app-django

django-social keeps a list of backends in memory that it gathers based on the value of settings.AUTHENTICATION_BACKENDS at import time: c1e2795b00/social_django/utils.py (L13)

Our settings.AUTHENTICATION_BACKENDS can change dynamically as Tower settings are changed (i.e., if somebody configures Github OAuth2 integration), so we need to overwrite this in-memory value at the top of every request so that we have the latest version see: https://github.com/ansible/tower/issues/1979

django-oauth-toolkit

Version 1.2.0 of this project has a bug that error when revoking tokens. This is fixed in the master branch but is not yet released.

When upgrading past 1.2.0 in the future, the 0025 migration needs to be edited, just like the old migration was edited in the project: 96538876d0 The field can simply have the validator method validate_uris removed.

azure-keyvault

Upgrading to 4.0.0 causes error because imports changed.

  File "/var/lib/awx/venv/awx/lib64/python3.6/site-packages/awx/main/credential_plugins/azure_kv.py", line 4, in <module>
  from azure.keyvault import KeyVaultClient, KeyVaultAuthentication
ImportError: cannot import name 'KeyVaultClient'

slackclient

Imports as used in awx/main/notifications/slack_backend.py changed in version 2.0. This plugin code will need to change and be re-tested as the upgrade takes place.

django-jsonfield

Instead of calling a loads() operation, the returned value is casted into a string in some cases, introduced in the change:

https://github.com/adamchainz/django-jsonfield/pull/14

This breaks a very large amount of AWX code that assumes these fields are returned as dicts. Upgrading this library will require a refactor to accomidate this change.

pip and setuptools

The offline installer needs to have functionality confirmed before upgrading these. Versions need to match the versions used in the pip bootstrapping step in the top-level Makefile.

Library Notes

celery

This is only used for the beat feature (running periodic tasks). This could be replaced, see: https://github.com/ansible/awx/pull/2530

requests-futures

This can be removed when a solution for the external log queuing is ready. https://github.com/ansible/awx/pull/5092

asgi-amqp

This library is not compatible with channels 2 and is not expected to become so. This drives other pins in the requirements file.