Skip to content
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

Merged
merged 3 commits into from
Mar 24, 2021

Conversation

Spredzy
Copy link
Contributor

@Spredzy Spredzy commented Dec 7, 2020

SUMMARY

Bumping default python used in awx and ansible venv to python 3.8

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • API
AWX VERSION
  • devel
ADDITIONAL INFORMATION
  • N/A

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@Spredzy
Copy link
Contributor Author

Spredzy commented Dec 8, 2020

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)

@Spredzy
Copy link
Contributor Author

Spredzy commented Dec 16, 2020

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@unlikelyzero
Copy link

@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

@unlikelyzero
Copy link

@Spredzy , give ui-qe some time to fix our pipelines? I think @one-t is the most knowledgeable about how to test the changes.

@one-t do you mind taking this one?

@ryanpetrello
Copy link
Contributor

@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?

@unlikelyzero
Copy link

Adding @tiagodread

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

Makefile Outdated Show resolved Hide resolved
Copy link
Contributor

@ryanpetrello ryanpetrello left a 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 👍

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@unlikelyzero
Copy link

@Spredzy has this gone through the API and CLI tests?

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@Spredzy
Copy link
Contributor Author

Spredzy commented Mar 22, 2021

recheck

1 similar comment
@Spredzy
Copy link
Contributor Author

Spredzy commented Mar 22, 2021

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

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
Copy link
Contributor

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.

Copy link
Contributor Author

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

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@Spredzy
Copy link
Contributor Author

Spredzy commented Mar 24, 2021

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 2b965e5 into ansible:devel Mar 24, 2021
Spredzy added a commit to Spredzy/awx that referenced this pull request Apr 1, 2021
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
Spredzy added a commit to Spredzy/awx that referenced this pull request Apr 1, 2021
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
Spredzy added a commit to Spredzy/awx that referenced this pull request Apr 1, 2021
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
softwarefactory-project-zuul bot added a commit that referenced this pull request Apr 1, 2021
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants