-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Python: Bump to python 3.8 #8778
Python: Bump to python 3.8 #8778
Conversation
Build failed.
|
Pausing work on this PR. Work will resume when the execution enviironment work will land. Removing the need for an ansible venv (and only having an awx one) |
recheck |
Build failed.
|
@Spredzy hey, this may impact our ui tests' use of awxkit. Let us know with @ui-qe when you're about 2 days away from merging this and we'll work to get it ready on our side |
@Spredzy is this PR still active (it's out of date given the k8s installer -> operator move)? Would it make sense to tackle this later, or do you think we'll update this PR in the near future? |
Adding @tiagodread |
e4b2340
to
380febc
Compare
Build failed.
|
380febc
to
8e958db
Compare
Build failed.
|
8e958db
to
74dd3ae
Compare
Build failed.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once Zuul is happy, one minor comment, otherwise, this looks about right to me 👍
74dd3ae
to
c2fe052
Compare
Build failed.
|
@Spredzy has this gone through the API and CLI tests? |
c2fe052
to
5515fb7
Compare
Build failed.
|
Build succeeded.
|
6b19557
to
9736734
Compare
Build succeeded.
|
9736734
to
0f06a73
Compare
Build succeeded.
|
0f06a73
to
442ed41
Compare
Build failed.
|
recheck |
1 similar comment
recheck |
Build failed.
|
Build succeeded.
|
@@ -83,6 +83,8 @@ defusedxml==0.6.0 | |||
# social-auth-core | |||
dictdiffer==0.8.1 | |||
# via openshift | |||
distro==1.5.0 | |||
# via -r /awx_devel/requirements/requirements.in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can/should regenerate this with 3.8. Some things will fall out (and you can remove the license files for them)
Or that can be a followup change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do in a subsequent PR yep
2280b59
to
6d8874d
Compare
Build failed.
|
recheck |
Build failed.
|
Build succeeded.
|
Build succeeded (gate pipeline).
|
The generated based venv from `python3.8 -m venv` vs. `virtualenv -p python38` is different. This changes aims to address the differences. It was introduced as part of the Python 3.8 migration. ansible#8778
The generated based venv from `python3.8 -m venv` vs. `virtualenv -p python38` is different. This changes aims to address the differences. It was introduced as part of the Python 3.8 migration. ansible#8778
The generated based venv from `python3.8 -m venv` vs. `virtualenv -p python38` is different. This changes aims to address the differences. It was introduced as part of the Python 3.8 migration. ansible#8778
Add wheel in venv creation The generated based venv from python3.8 -m venv vs. virtualenv -p python38 is different. This changes aims to address the differences. It was introduced as part of the Python 3.8 migration. #8778 Reviewed-by: Shane McDonald <me@shanemcd.com>
SUMMARY
Bumping default python used in awx and ansible venv to python 3.8
ISSUE TYPE
COMPONENT NAME
AWX VERSION
ADDITIONAL INFORMATION