Skip to content

fix(docker): Hardcoding virtualenv version to keep Python 2.7 support#10479

Merged
dandhlee merged 3 commits into
mainfrom
strzelczyk/docker-py2.7-fix
Aug 1, 2023
Merged

fix(docker): Hardcoding virtualenv version to keep Python 2.7 support#10479
dandhlee merged 3 commits into
mainfrom
strzelczyk/docker-py2.7-fix

Conversation

@m-strzelczyk

Copy link
Copy Markdown
Contributor

Since version 20.22 virtualenv no longer supports Python 2.7. We hardcode virtualenv version to keep Python 2.7 while we work on deprecating it from our repositories.

@m-strzelczyk m-strzelczyk requested a review from a team as a code owner July 28, 2023 12:11
@product-auto-label product-auto-label Bot added the samples Issues that are directly related to samples. label Jul 28, 2023
@m-strzelczyk

Copy link
Copy Markdown
Contributor Author

Tests seem to be failing due to quota problems. Should be safe to merge.

Comment on lines +93 to +95
# Install virtualenv in version 20.21, since version 20.22 no longer supports Python 2.7
pip install --user -q virtualenv==20.21

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we split them for 2.7? There's a runtime specific condition above:

if [[ "${RUN_TESTS_SESSION}" == "py-3.8" ]]; then

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, done!

@dandhlee dandhlee merged commit 9df5b8f into main Aug 1, 2023
@dandhlee dandhlee deleted the strzelczyk/docker-py2.7-fix branch August 1, 2023 14:26
rsamborski pushed a commit that referenced this pull request Aug 2, 2023
…#10479)

* fix(docker): Hardcoding virtualenv version to keep Python 2.7 support

* Limiting the virtualenv installation to py-2.7 only
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants