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

fix(lambda-python): Pin poetry version to allow installing packages using Code Artifact #21865

Closed
wants to merge 2 commits into from

Conversation

setu4993
Copy link
Contributor

@setu4993 setu4993 commented Sep 1, 2022

The latest update to poetry, v1.2, doesn't support installing packages from private PyPI indices that only support MD5 hashes (python-poetry/poetry#6301).

AWS Code Artifact does not support SHA256 hashes, only MD5. Because of this, we cannot build any CDK stacks involving private packages that rely on Code Artifact.

Would appreciate a quick review + release because this is blocking production releases for us right now.


All Submissions:

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

@gitpod-io
Copy link

gitpod-io bot commented Sep 1, 2022

@github-actions github-actions bot added the p2 label Sep 1, 2022
@aws-cdk-automation aws-cdk-automation requested a review from a team September 1, 2022 00:18
@setu4993 setu4993 marked this pull request as draft September 1, 2022 00:43
@setu4993
Copy link
Contributor Author

setu4993 commented Sep 1, 2022

Looks like I was maybe hitting a different issue. Converting to draft while I investigate.

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: 0a56726
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

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

@TheRealAmazonKendra
Copy link
Contributor

Given the lack of updates here, it looks like this wasn't the issue? I'm going to go ahead and close this but feel free to tag me and let me know if you need it reopened.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants