-
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
Update dependencies to fix offline build #15732
Conversation
@@ -70,7 +70,7 @@ RECEPTOR_IMAGE ?= quay.io/ansible/receptor:devel | |||
SRC_ONLY_PKGS ?= cffi,pycparser,psycopg,twilio | |||
# These should be upgraded in the AWX and Ansible venv before attempting | |||
# to install the actual requirements | |||
VENV_BOOTSTRAP ?= pip==21.2.4 setuptools==69.0.2 setuptools_scm[toml]==8.0.4 wheel==0.42.0 cython==0.29.37 | |||
VENV_BOOTSTRAP ?= pip==21.2.4 setuptools==72.1.0 setuptools_scm[toml]==8.1.0 wheel==0.45.1 cython==3.0.11 |
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.
cross-referencing, this makes the values match what's in requirements.txt
, which is really our goof here. It would be good for us to add some form of check for this.
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.
I agree, last merge left things in an inconsistent state, and this fixes that
Created #15733 to address many of the loopholes that helped to create this situation. |
Quality Gate passedIssues Measures |
I put up simaishi#1 but I am struggling to test it. |
Closing in favor of #15740 |
SUMMARY
Follow-up for #15705 to fix offline build
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION
VENV_BOOTSTRAP
to match updates made in General upgrade of dependencies #15705