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

S3-Deployment-Assets : update python version on BucketDeployment handler #16669

Closed
aritranagnordcloud opened this issue Sep 27, 2021 · 3 comments
Assignees
Labels
effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p1

Comments

@aritranagnordcloud
Copy link

General Issue

Based on the description , We can see that Older Python version 3.6 is currently enabled which will be no longer supported soon.

Documentation
The custom resource is implemented in Python 3.6 in order to be able to leverage the AWS CLI for "aws s3 sync". The code is under lib/lambda and unit tests are under test/lambda.

This package requires Python 3.6 during build time in order to create the custom resource Lambda bundle and test it. It also relies on a few bash scripts, so might be tricky to build on Windows.

Resolution
Need to update the Lambda Python version having more than 3.6

Environment

CDK CLI Version: 1.119.0
Framework Version: 2.0.0rc18
Node.js Version: 16.6.1
OS: macOS Big Sur 11.4
Language (Version): Python 3.8.11

Other information

Lambda is configured with 3.6 version which has reached the EOF and will not longer be supported soon

@aritranagnordcloud aritranagnordcloud added guidance Question that needs advice or information. needs-triage This issue or PR still needs to be triaged. labels Sep 27, 2021
@github-actions github-actions bot added the @aws-cdk/aws-s3 Related to Amazon S3 label Sep 27, 2021
@LukvonStrom
Copy link
Contributor

Hi @aritranagnordcloud the issue is located here:

runtime: lambda.Runtime.PYTHON_3_6,

Changing this to a runtime: lambda.Runtime.PYTHON_3_9, in code should work.

@peterwoodworth peterwoodworth changed the title S3 Deployment Assets : Inbuild Lambda is build with older python version 3.6 S3-Deployment-Assets : update python version on BucketDeployment handler Sep 27, 2021
@peterwoodworth peterwoodworth added feature-request A feature should be added or improved. effort/small Small work item – less than a day of effort p1 and removed @aws-cdk/aws-s3 Related to Amazon S3 guidance Question that needs advice or information. needs-triage This issue or PR still needs to be triaged. labels Sep 27, 2021
@peterwoodworth
Copy link
Contributor

Converted this into a feature request. Once @otaviomacedo confirms what version we want to change it to (if we want to change it) I can create a PR for this

mergify bot pushed a commit that referenced this issue Oct 4, 2021
…ler (#16771)

fixes :#16669

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@github-actions
Copy link

github-actions bot commented Oct 7, 2021

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

njlynch pushed a commit that referenced this issue Oct 11, 2021
…ler (#16771)

fixes :#16669

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this issue Feb 21, 2022
…ler (aws#16771)

fixes :aws#16669

----

*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
effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants