]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
requirements.txt: Move the six dependency into the general requirements
authorFlorian Haas <florian@citynetwork.eu>
Fri, 12 Feb 2021 08:29:00 +0000 (09:29 +0100)
committerGuillaume Abrioux <gabrioux@redhat.com>
Wed, 3 Mar 2021 12:22:29 +0000 (13:22 +0100)
config_template.py depends on six, which isn't listed in the default
requirements.txt. This previously frequently wasn't a problem, because
six used to be a standard package being installed into a venv, and
lots of other projects depended on it.

It also does get installed for unit and integration tests via
tests/requirements.txt, so any broken dependency on six wouldn't be
detected by tox runs.

However, as other projects and distributions have phased out Python
2.7 support the dependency on six becomes less common. Thus, as long
as ceph-ansible does require it for config_template.py, add it to the
base requirements.

Signed-off-by: Florian Haas <florian@citynetwork.eu>
(cherry picked from commit d49ea9818b162a4b8fc0f5099f45acff4d22fe14)

requirements.txt
tests/requirements.txt

index 9dbf90ae2ca3fd88ffadfff5c7597c216c115842..3769437fd82a0c5e59c54e13cde70fb27f259db7 100644 (file)
@@ -1,3 +1,4 @@
 # These are Python requirements needed to run ceph-ansible master
 ansible>=2.9,<2.10,!=2.9.10
 netaddr
+six
index 5af56f78066d08f893b9b2ef80d045085fcdbf8c..5820c69e0b28faf00af5b393d07120467f8be6f4 100644 (file)
@@ -1,5 +1,4 @@
 # These are Python requirements needed to run the functional tests
-six==1.10.0
 testinfra>=3,<4
 pytest-xdist==1.28.0
 pytest>=4.6,<5.0