-
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
General dependency upgrades (awx venv only for now) #5407
General dependency upgrades (awx venv only for now) #5407
Conversation
0d4ae3e
to
7d7584c
Compare
Looks like the swagger failure was introduced by an upgrade of DRF in this between the two versions encode/django-rest-framework@3.9.4...3.10.0 and I know that it was introduced before 7762aaa90f92ff30e521931db2ec65c52f8da98e, still searching locally. |
passes with encode/django-rest-framework@9811a29 fails with encode/django-rest-framework@c3a1391 I think I'm confusing myself because they did branch merges, and I'm only interested in the 3.10 branch |
introduced in encode/django-rest-framework@37f210a
encode/django-rest-framework#6532
|
998598b
to
ecd9c29
Compare
Don't want to lose this |
ecd9c29
to
011be46
Compare
12f30a7
to
d30da5b
Compare
fc9e780
to
d62770b
Compare
Build failed.
|
Build failed.
|
Build succeeded.
|
39ddb50
to
11e198a
Compare
Build succeeded.
|
Versions selected to be pre-19 pip due to unresolved issues with the build systems Upgrade everything, party on document new process rotate license files fix Swagger schema generation target Remove --ignore-installed flag
this is in addition to pip and setuptools installs add in --ignore-installed and --no-build-isolation flags reasoning is that we are not installing needed packages for building
Remove build isolation flag Do not use --ignore-installed for any pip install commands Add Makefile comments do not use system site packages for awx venv Consolidate bootstrap pins Do another upgrade, properly document wheel
Remove poetry and wheel from bootstrap installs Cleanup some docs around them
11e198a
to
18bb910
Compare
Build failed.
|
recheck |
Build succeeded.
|
Build succeeded (gate pipeline).
|
Display host name for Associate Modal See: ansible#11256
Display host name for Associate Modal See: ansible#11256
Display host name for Associate Modal See: ansible#11256
Display host name for Associate Modal See: ansible#11256
Display host name for Associate Modal See: ansible#11256
Display host name for Associate Modal See: ansible#11256
SUMMARY
This implements a process change and does a general upgrade of everything in the AWX virtual environment which won't break things.
Like all dependency upgrades, most of the real work involves categorizing which things will break things on upgrade. To do this in a more maintainable way, I've written blurbs on those things we can't upgrade, and then unpinned the content in
requirements.in
for those that don't break things.ISSUE TYPE
COMPONENT NAME
AWX VERSION
ADDITIONAL INFORMATION
This needs some corresponding changes to unblock upgrade of pip & setuptools, but I have been testing it with temporary hacks for that and everything is going fine, so it looks like the ecosystem has worked out enough to not error here.