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

Support Newest Manylinux Version for dependencies #1083

Merged
merged 3 commits into from
Dec 1, 2021

Conversation

phil-airship
Copy link
Contributor

Zappa was hard coded to only allow Manylinux 1, 2010, and 2014.
Manylinux 2_24 is getting more traction, so dependencies are becoming incompatible with Zappa over time.
More information on new Manylinux versions can be found here - https://github.com/pypa/manylinux

An important example is Cryptography, where version 3.4.7 is the last one to support old-format linux wheels.
https://pypi.org/project/cryptography/3.4.7/#files
vs newest:
https://pypi.org/project/cryptography/#files

Tested on my env to make sure that manylinux 2_24 wheels would work without issue in Lambda

Zappa was hard coded to only allow Manylinux 1, 2010, and 2014.
Manylinux 2_24 is getting more traction, so dependencies are becoming incompatible with Zappa over time.
More information on new Manylinux versions can be found here - https://github.com/pypa/manylinux

An important example is Cryptography, where version 3.4.7 is the last one to support old-format linux wheels.
https://pypi.org/project/cryptography/3.4.7/#files
vs newest:
https://pypi.org/project/cryptography/#files

Tested on my env to make sure that manylinux 2_24 wheels would work without issue in Lambda
Copy link
Collaborator

@hellno hellno left a comment

Choose a reason for hiding this comment

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

Thanks for creating this PR, super helpful (and I wasn't aware of the new manylinux dependency format at all, so thanks for pointing that out)

@coveralls
Copy link

coveralls commented Nov 22, 2021

Coverage Status

Coverage remained the same at 73.527% when pulling 50381a6 on phil-airship:patch-2 into b5b80cf on zappa:master.

@phil-airship
Copy link
Contributor Author

Is there a reason it things the coverage decreased for some python version, but stayed the same for overall coverage and py3.7?

@hellno
Copy link
Collaborator

hellno commented Nov 22, 2021

We need to fix the flaky coverage tests separately, it's not related to this PR.
The +-0.0% coverage changes on different python versions shouldn't fail at all :)

@hellno
Copy link
Collaborator

hellno commented Nov 23, 2021

Updated from master to test new coveralls threshold, seems to work 👍🏼

@hellno hellno merged commit ba6c19a into zappa:master Dec 1, 2021
@hellno
Copy link
Collaborator

hellno commented Dec 1, 2021

Thanks for the PR @phil-airship, I went ahead and merged it

phil-airship added a commit to phil-airship/Zappa that referenced this pull request May 15, 2023
Had added support for Manylinux 2_24 in 2021:
zappa#1083

But that version is EoL and Python Cryptography has moved
to Manylinux 2_28.

Updating to support while maintaining backwards compatibility.

Tested on Lambda an encountered no issues
Ian288 pushed a commit to tackle-io/Zappa that referenced this pull request Jul 11, 2023
* Support Newest Manylinux Version for dependencies

Zappa was hard coded to only allow Manylinux 1, 2010, and 2014.
Manylinux 2_24 is getting more traction, so dependencies are becoming incompatible with Zappa over time.
More information on new Manylinux versions can be found here - https://github.com/pypa/manylinux

An important example is Cryptography, where version 3.4.7 is the last one to support old-format linux wheels.
https://pypi.org/project/cryptography/3.4.7/#files
vs newest:
https://pypi.org/project/cryptography/#files

Tested on my env to make sure that manylinux 2_24 wheels would work without issue in Lambda

* Update core.py

Co-authored-by: hellno <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants