Skip to content

Conversation

@eladb
Copy link
Contributor

@eladb eladb commented Feb 9, 2021

poetry, which is needed when creating python bundles, has a dependency on the cryptography module.
As described in their changelog, starting version 3.4, they require the latest version of pip. Otherwise,
the installer will attempt to compile the module, and the Rust compiler will be required.

This fails in the amazon/aws-sam-cli-build-image-python3.6 image since it has an older version of pip installed.

To fix, simply add an instruction to the dockerfile to upgrade to the latest pip version before installing.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

…sion of pip

`poetry`, which is needed when creating python bundles, has a dependency on the `cryptography` module.
As described in their [changelog], starting version 3.4, they require the latest version of `pip`. Otherwise,
the installer will attempt to compile the module, and the Rust compiler will be required.

This fails in the amazon/aws-sam-cli-build-image-python3.6 image since it has an older version of pip installed.

To fix, simply add an instruction to the dockerfile to upgrade to the latest pip version before installing.

[changelog]: https://cryptography.io/en/3.4/changelog.html#v3-4
@gitpod-io
Copy link

gitpod-io bot commented Feb 9, 2021

@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 9, 2021
@eladb eladb added the pr-linter/exempt-test The PR linter will not require test changes label Feb 9, 2021
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 4f3608d
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify
Copy link
Contributor

mergify bot commented Feb 9, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit b68acf8 into master Feb 9, 2021
@mergify mergify bot deleted the benisrae/fix-lambda-python branch February 9, 2021 09:28
TLadd pushed a commit to TLadd/aws-cdk that referenced this pull request Feb 9, 2021
… version (aws#12934)

`poetry`, which is needed when creating python bundles, has a dependency on the `cryptography` module.
As described in their [changelog], starting version 3.4, they require the latest version of `pip`. Otherwise,
the installer will attempt to compile the module, and the Rust compiler will be required.

This fails in the amazon/aws-sam-cli-build-image-python3.6 image since it has an older version of pip installed.

To fix, simply add an instruction to the dockerfile to upgrade to the latest pip version before installing.

[changelog]: https://cryptography.io/en/3.4/changelog.html#v3-4


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
NovakGu pushed a commit to NovakGu/aws-cdk that referenced this pull request Feb 18, 2021
… version (aws#12934)

`poetry`, which is needed when creating python bundles, has a dependency on the `cryptography` module.
As described in their [changelog], starting version 3.4, they require the latest version of `pip`. Otherwise,
the installer will attempt to compile the module, and the Rust compiler will be required.

This fails in the amazon/aws-sam-cli-build-image-python3.6 image since it has an older version of pip installed.

To fix, simply add an instruction to the dockerfile to upgrade to the latest pip version before installing.

[changelog]: https://cryptography.io/en/3.4/changelog.html#v3-4


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@aws-cdk/aws-lambda-python contribution/core This is a PR that came from AWS. pr-linter/exempt-test The PR linter will not require test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants