Skip to content

Commit d4d857f

Browse files
jdmajorgreys
authored andcommitted
ci: requires virtualenv < 20
There's a regression in virtualenv >= 20 that makes one of the ddtrace-run + Celery test to fails. virtualenv now adds a site.py which modifies sys.path and breaks the PYTHONPATH that is set by ddtrace-run.
1 parent 45a5c85 commit d4d857f

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.circleci/config.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -703,9 +703,9 @@ jobs:
703703
# list all executed test
704704
- run: jq -s ".[]|.testenvs|keys|.[]" /tmp/workspace/* | grep -v GLOB | sed 's/"//g' | sort | uniq | tee all_executed_tests
705705
# list all tests in tox.ini
706-
- run: tox -l | grep -v "^wait$" | sort > all_tests
706+
- run: tox -l | grep -v -E "^(wait$|\.)" | sort > all_tests
707707
# checks that all tests were executed
708-
- run: diff all_tests all_executed_tests
708+
- run: diff -u all_tests all_executed_tests
709709

710710

711711
requires_pre_test: &requires_pre_test

tox.ini

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
# versions.
44

55
[tox]
6+
requires = virtualenv<20
67
# Our various test environments. The py*-all tasks will run the core
78
# library tests and all contrib tests with the latest library versions.
89
# The others will test specific versions of libraries.

0 commit comments

Comments
 (0)